2020-03-11 16:28:47

by Jaegeuk Kim

[permalink] [raw]
Subject: [GIT PULL] f2fs for 5.6-rc6

Hi Linus,

Sorry for late pull request. Could you please consider this?

Thanks,

The following changes since commit b19e8c68470385dd2c5440876591fddb02c8c402:

Merge tag 'arm64-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux (2020-02-13 14:36:57 -0800)

are available in the Git repository at:

git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-5.6-rc6

for you to fetch changes up to 2536ac6872e5265b4d9d263122cee02e3d5cae1d:

f2fs: allow to clear F2FS_COMPR_FL flag (2020-03-11 08:25:38 -0700)

----------------------------------------------------------------
f2fs-for-5.6-rc6

Some major bug fixes wrt compression:
- compressed block count
- memory access and leak
- remove obsolete fields
- flag controls

And some others:
- replace rwsem with spinlock
- potential deadlock

----------------------------------------------------------------
Chao Yu (14):
f2fs: fix to wait all node page writeback
f2fs: fix to avoid NULL pointer dereference
f2fs: recycle unused compress_data.chksum feild
f2fs: add missing function name in kernel message
f2fs: fix to avoid potential deadlock
f2fs: fix to check i_compr_blocks correctly
f2fs: cover last_disk_size update with spinlock
f2fs: remove i_sem lock coverage in f2fs_setxattr()
f2fs: fix inconsistent comments
f2fs: fix to avoid using uninitialized variable
f2fs: fix to avoid use-after-free in f2fs_write_multi_pages()
f2fs: fix to account compressed inode correctly
f2fs: fix to check dirty pages during compressed inode conversion
f2fs: allow to clear F2FS_COMPR_FL flag

Eric Biggers (1):
f2fs: fix leaking uninitialized memory in compressed clusters

Jaegeuk Kim (1):
f2fs: fix wrong check on F2FS_IOC_FSSETXATTR

Sahitya Tummala (1):
f2fs: fix the panic in do_checkpoint()

fs/f2fs/checkpoint.c | 34 +++++++++++-----------------------
fs/f2fs/compress.c | 24 ++++++++++++++----------
fs/f2fs/data.c | 23 ++++++++---------------
fs/f2fs/f2fs.h | 18 ++++++++++--------
fs/f2fs/file.c | 40 +++++++++++++++++++++++++---------------
fs/f2fs/gc.c | 5 ++++-
fs/f2fs/inode.c | 25 +++++++++++++++++++++----
fs/f2fs/namei.c | 2 +-
fs/f2fs/node.c | 18 ++++++++----------
fs/f2fs/shrinker.c | 2 +-
fs/f2fs/super.c | 7 ++++---
fs/f2fs/xattr.c | 3 ---
12 files changed, 107 insertions(+), 94 deletions(-)


2020-03-11 17:21:52

by Linus Torvalds

[permalink] [raw]
Subject: Re: [GIT PULL] f2fs for 5.6-rc6

On Wed, Mar 11, 2020 at 9:27 AM Jaegeuk Kim <[email protected]> wrote:
>
> Sorry for late pull request. Could you please consider this?

I pulled this, and then immediately unpulled.

Most (all?) of the commits have been committed basically minutes
before you sent the pull request.

That's simply not acceptable. Not when we're in late rc, and with
hundreds of lines of changes, and when there is no explanation for a
late pull request that was very very recently generated.

Linus

2020-03-11 17:36:51

by Jaegeuk Kim

[permalink] [raw]
Subject: Re: [GIT PULL] f2fs for 5.6-rc6

On 03/11, Linus Torvalds wrote:
> On Wed, Mar 11, 2020 at 9:27 AM Jaegeuk Kim <[email protected]> wrote:
> >
> > Sorry for late pull request. Could you please consider this?
>
> I pulled this, and then immediately unpulled.
>
> Most (all?) of the commits have been committed basically minutes
> before you sent the pull request.
>
> That's simply not acceptable. Not when we're in late rc, and with
> hundreds of lines of changes, and when there is no explanation for a
> late pull request that was very very recently generated.

I actually merged the last three patches which were introduced yesterday.
I thought that it'd be fine to pull in, since they are quite trivial several
lines of code changes.

Others were merged over a week ago, and I've tested in the mean time.
It seems the commit times were modified as recent date, when I reorganized
there-in commits aligned to other branch, dev-test. Probably, I had to keep
the commit date in somehow.

Thanks,

>
> Linus

2020-03-11 17:42:17

by Linus Torvalds

[permalink] [raw]
Subject: Re: [GIT PULL] f2fs for 5.6-rc6

On Wed, Mar 11, 2020 at 10:35 AM Jaegeuk Kim <[email protected]> wrote:
>
> I actually merged the last three patches which were introduced yesterday.
>
> Others were merged over a week ago.

No, three were done just before you sent the pull request, and then
seven others were done yesterday.

Yes, the rest were a week ago.

But basically, by rc6, I want to get the warm and fuzzies that the
code has been tested, seen review, and isn't some last-minute
addition.

Linus

2020-03-11 17:50:10

by Jaegeuk Kim

[permalink] [raw]
Subject: Re: [GIT PULL] f2fs for 5.6-rc6

On 03/11, Linus Torvalds wrote:
> On Wed, Mar 11, 2020 at 10:35 AM Jaegeuk Kim <[email protected]> wrote:
> >
> > I actually merged the last three patches which were introduced yesterday.
> >
> > Others were merged over a week ago.
>
> No, three were done just before you sent the pull request, and then
> seven others were done yesterday.

Due to my rebase work. :(

>
> Yes, the rest were a week ago.
>
> But basically, by rc6, I want to get the warm and fuzzies that the
> code has been tested, seen review, and isn't some last-minute
> addition.

Ok, I see. Let me prepare them for next merge window.

Thanks,

2020-04-07 21:09:13

by pr-tracker-bot

[permalink] [raw]
Subject: Re: [GIT PULL] f2fs for 5.6-rc6

The pull request you sent on Wed, 11 Mar 2020 09:27:35 -0700:

> git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git tags/f2fs-for-5.6-rc6

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/c6d5789bea5187c2716959f8954a1e00b8a7835d

Thank you!

--
Deet-doot-dot, I am a bot.
https://korg.wiki.kernel.org/userdoc/prtracker