[GIT PULL] f2fs fixes for v3.8-rc7

[GIT PULL] f2fs fixes for v3.8-rc7

[Jaegeuk Kim]

[-- Attachment #1: Type: text/plain, Size: 3347 bytes --]

Hi Linus,

Here are four patches which are critical bug fixes on f2fs, three
enhancement patches, and a number of trivial patches.
Please pull the following tag. Sorry for the late request.

Thanks,

The following changes since commit
6abb7c25775b7fb2225ad0508236d63ca710e65f:

  Merge tag 'regulator-3.8-rc5' of
git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator
(2013-01-28 22:44:53 -0800)

are available in the git repository at:


  git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git
tags/f2fs-for-v3.8

for you to fetch changes up to 1efc6d3277f59b764384781c0f8dfc821f229380:

  f2fs: add compat_ioctl to provide backward compatability (2013-02-06
17:38:59 +0900)

----------------------------------------------------------------
f2fs fixes for v3.8

[Major bug fixes]
o Store device file information correctly
o Fix -EIO handling with respect to power-off-recovery
o Allocate blocks with global locks
o Fix wrong calculation of the SSR cost

[Enhancement]
o Support (un)freeze_fs
o Enhance the f2fs_gc flow
o Support 32-bit binary execution on 64-bit kernel

----------------------------------------------------------------
Alejandro Martinez Ruiz (1):
      f2fs: fix disable_ext_identify option spelling

Changman Lee (5):
      f2fs: save device node number into f2fs_inode
      f2fs: add un/freeze_fs into super_operations
      f2fs: stop repeated checking if cp is needed
      f2fs: remove repeated F2FS_SET_SB_DIRT call
      f2fs: remove unnecessary gc option check and balance_fs

Jaegeuk Kim (6):
      f2fs: prevent checkpoint once any IO failure is detected
      f2fs: cover global locks for reserve_new_block
      f2fs: remove the use of page_cache_release
      f2fs: avoid balanc_fs during evict_inode
      f2fs: clarify and enhance the f2fs_gc flow
      f2fs: fix calculation of max. gc cost in the SSR case

Namjae Jeon (8):
      f2fs: avoid redundant call to has_not_enough_free_secs in f2fs_gc
      f2fs: reorganize code for ra_node_page
      f2fs: fix typo mistake for data_version description
      f2fs: name gc task as per the block device
      f2fs: mark gc_thread as NULL when thread creation is failed
      f2fs: make an accessor to get sections for particular block type
      f2fs: optimize the return condition for has_not_enough_free_secs
      f2fs: add compat_ioctl to provide backward compatability

majianpeng (4):
      f2fs: clean up the add_orphan_inode func
      f2fs: add device name in debugfs
      f2fs: use F2FS_BLKSIZE to judge bloksize and page_cache_size
      f2fs: when check superblock failed, try to check another
superblock

 fs/f2fs/checkpoint.c |  63 +++++++++++---------------
 fs/f2fs/debug.c      |   4 +-
 fs/f2fs/f2fs.h       |  32 ++++++++++---
 fs/f2fs/file.c       |  35 ++++++++++++---
 fs/f2fs/gc.c         | 124
++++++++++++++++++++++-----------------------------
 fs/f2fs/gc.h         |  21 ---------
 fs/f2fs/inode.c      |  53 +++++++++-------------
 fs/f2fs/node.c       |  14 +++---
 fs/f2fs/recovery.c   |   4 +-
 fs/f2fs/segment.c    |  29 ++++++++----
 fs/f2fs/segment.h    |  23 +++-------
 fs/f2fs/super.c      |  92 +++++++++++++++++++++++++++++---------
 12 files changed, 262 insertions(+), 232 deletions(-)

-- 
Jaegeuk Kim
Samsung

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [GIT PULL] f2fs fixes for v3.8-rc7

[Linus Torvalds]

No.

You guys need to realize that I'm not talking crap like this this late.

This is not major bugfixes. I already looked away once just because
it's a new filesystem, but enough is enough. This is way way WAY too
late to start sendign "enhancements". Seriously.

Send them for the next merge window. Not just before rc7.

                   Linus

On Thu, Feb 7, 2013 at 11:21 AM, Jaegeuk Kim <jaegeuk.kim@samsung.com> wrote:
> Hi Linus,
>
> Here are four patches which are critical bug fixes on f2fs, three
> enhancement patches, and a number of trivial patches.
> Please pull the following tag. Sorry for the late request.
>
> Thanks,
>
> The following changes since commit
> 6abb7c25775b7fb2225ad0508236d63ca710e65f:
>
>   Merge tag 'regulator-3.8-rc5' of
> git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator
> (2013-01-28 22:44:53 -0800)
>
> are available in the git repository at:
>
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git
> tags/f2fs-for-v3.8
>
> for you to fetch changes up to 1efc6d3277f59b764384781c0f8dfc821f229380:
>
>   f2fs: add compat_ioctl to provide backward compatability (2013-02-06
> 17:38:59 +0900)
>
> ----------------------------------------------------------------
> f2fs fixes for v3.8
>
> [Major bug fixes]
> o Store device file information correctly
> o Fix -EIO handling with respect to power-off-recovery
> o Allocate blocks with global locks
> o Fix wrong calculation of the SSR cost
>
> [Enhancement]
> o Support (un)freeze_fs
> o Enhance the f2fs_gc flow
> o Support 32-bit binary execution on 64-bit kernel
>
> ----------------------------------------------------------------
> Alejandro Martinez Ruiz (1):
>       f2fs: fix disable_ext_identify option spelling
>
> Changman Lee (5):
>       f2fs: save device node number into f2fs_inode
>       f2fs: add un/freeze_fs into super_operations
>       f2fs: stop repeated checking if cp is needed
>       f2fs: remove repeated F2FS_SET_SB_DIRT call
>       f2fs: remove unnecessary gc option check and balance_fs
>
> Jaegeuk Kim (6):
>       f2fs: prevent checkpoint once any IO failure is detected
>       f2fs: cover global locks for reserve_new_block
>       f2fs: remove the use of page_cache_release
>       f2fs: avoid balanc_fs during evict_inode
>       f2fs: clarify and enhance the f2fs_gc flow
>       f2fs: fix calculation of max. gc cost in the SSR case
>
> Namjae Jeon (8):
>       f2fs: avoid redundant call to has_not_enough_free_secs in f2fs_gc
>       f2fs: reorganize code for ra_node_page
>       f2fs: fix typo mistake for data_version description
>       f2fs: name gc task as per the block device
>       f2fs: mark gc_thread as NULL when thread creation is failed
>       f2fs: make an accessor to get sections for particular block type
>       f2fs: optimize the return condition for has_not_enough_free_secs
>       f2fs: add compat_ioctl to provide backward compatability
>
> majianpeng (4):
>       f2fs: clean up the add_orphan_inode func
>       f2fs: add device name in debugfs
>       f2fs: use F2FS_BLKSIZE to judge bloksize and page_cache_size
>       f2fs: when check superblock failed, try to check another
> superblock
>
>  fs/f2fs/checkpoint.c |  63 +++++++++++---------------
>  fs/f2fs/debug.c      |   4 +-
>  fs/f2fs/f2fs.h       |  32 ++++++++++---
>  fs/f2fs/file.c       |  35 ++++++++++++---
>  fs/f2fs/gc.c         | 124
> ++++++++++++++++++++++-----------------------------
>  fs/f2fs/gc.h         |  21 ---------
>  fs/f2fs/inode.c      |  53 +++++++++-------------
>  fs/f2fs/node.c       |  14 +++---
>  fs/f2fs/recovery.c   |   4 +-
>  fs/f2fs/segment.c    |  29 ++++++++----
>  fs/f2fs/segment.h    |  23 +++-------
>  fs/f2fs/super.c      |  92 +++++++++++++++++++++++++++++---------
>  12 files changed, 262 insertions(+), 232 deletions(-)
>
> --
> Jaegeuk Kim
> Samsung

Re: [GIT PULL] f2fs fixes for v3.8-rc7

[Al Viro]

On Fri, Feb 08, 2013 at 12:09:34PM +1100, Linus Torvalds wrote:
> No.
> 
> You guys need to realize that I'm not talking crap like this this late.
> 
> This is not major bugfixes. I already looked away once just because
> it's a new filesystem, but enough is enough. This is way way WAY too
> late to start sendign "enhancements". Seriously.
> 
> Send them for the next merge window. Not just before rc7.

BTW, for the next merge window I've got several patches to that sucker;
commits f3d9d7e..10e4e72 in vfs.git.  I can send them to f2fs tree or
have them go in vfs one, but this stuff needs to be dealt with.  Fake
struct dentry on stack is a bad thing with capital Hell No...

Re: [GIT PULL] f2fs fixes for v3.8-rc7

[Jaegeuk Kim]

[-- Attachment #1: Type: text/plain, Size: 934 bytes --]

2013-02-08 (금), 03:10 +0000, Al Viro:
> On Fri, Feb 08, 2013 at 12:09:34PM +1100, Linus Torvalds wrote:
> > No.
> > 
> > You guys need to realize that I'm not talking crap like this this late.
> > 
> > This is not major bugfixes. I already looked away once just because
> > it's a new filesystem, but enough is enough. This is way way WAY too
> > late to start sendign "enhancements". Seriously.
> > 
> > Send them for the next merge window. Not just before rc7.
> 
> BTW, for the next merge window I've got several patches to that sucker;
> commits f3d9d7e..10e4e72 in vfs.git.  I can send them to f2fs tree or
> have them go in vfs one, but this stuff needs to be dealt with.  Fake
> struct dentry on stack is a bad thing with capital Hell No...

Hi Al,

Could you send the patches to me?
I'll merge them in my tree and submit them in the next merge window.
Is it OK?

Thanks,

-- 
Jaegeuk Kim
Samsung

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [GIT PULL] f2fs fixes for v3.8-rc7

[Al Viro]

On Fri, Feb 08, 2013 at 01:20:31PM +0900, Jaegeuk Kim wrote:

> Hi Al,
> 
> Could you send the patches to me?
> I'll merge them in my tree and submit them in the next merge window.
> Is it OK?

Feel free to fetch/cherry-pick or simply pull from

git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs.git f2fs

- that branchlet grows directly off the mainline and it's not going to
be rebased...

Shortlog:
Al Viro (4):
      f2fs: init_dent_inode() should take qstr
      f2fs: switch new_inode_page() from dentry to qstr
      f2fs: switch init_inode_metadata() to passing parent and name separately
      f2fs: get rid of fake on-stack dentries

Diffstat:
 fs/f2fs/dir.c      |   29 +++++++++++++----------------
 fs/f2fs/f2fs.h     |   12 +++++++++---
 fs/f2fs/node.c     |    4 ++--
 fs/f2fs/recovery.c |   12 +++++-------
 4 files changed, 29 insertions(+), 28 deletions(-)

Re: [GIT PULL] f2fs fixes for v3.8-rc7

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 646 bytes --]

Hi,

On Fri, 8 Feb 2013 08:01:48 +0000 Al Viro <viro@ZenIV.linux.org.uk> wrote:
>
> On Fri, Feb 08, 2013 at 01:20:31PM +0900, Jaegeuk Kim wrote:
> 
> > Could you send the patches to me?
> > I'll merge them in my tree and submit them in the next merge window.
> > Is it OK?
> 
> Feel free to fetch/cherry-pick or simply pull from
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs.git f2fs
> 
> - that branchlet grows directly off the mainline and it's not going to
> be rebased...

Please pull/merge it directly, do not cherry-pick the patches.

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

Re: [GIT PULL] f2fs fixes for v3.8-rc7

[Jaegeuk Kim]

[-- Attachment #1: Type: text/plain, Size: 4624 bytes --]

Ok, I understand.
I'll prepare and send patches for the next merge window with these
patches together.
Thanks,

2013-02-08 (금), 12:09 +1100, Linus Torvalds:
> No.
> 
> You guys need to realize that I'm not talking crap like this this late.
> 
> This is not major bugfixes. I already looked away once just because
> it's a new filesystem, but enough is enough. This is way way WAY too
> late to start sendign "enhancements". Seriously.
> 
> Send them for the next merge window. Not just before rc7.
> 
>                    Linus
> 
> On Thu, Feb 7, 2013 at 11:21 AM, Jaegeuk Kim <jaegeuk.kim@samsung.com> wrote:
> > Hi Linus,
> >
> > Here are four patches which are critical bug fixes on f2fs, three
> > enhancement patches, and a number of trivial patches.
> > Please pull the following tag. Sorry for the late request.
> >
> > Thanks,
> >
> > The following changes since commit
> > 6abb7c25775b7fb2225ad0508236d63ca710e65f:
> >
> >   Merge tag 'regulator-3.8-rc5' of
> > git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator
> > (2013-01-28 22:44:53 -0800)
> >
> > are available in the git repository at:
> >
> >
> >   git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git
> > tags/f2fs-for-v3.8
> >
> > for you to fetch changes up to 1efc6d3277f59b764384781c0f8dfc821f229380:
> >
> >   f2fs: add compat_ioctl to provide backward compatability (2013-02-06
> > 17:38:59 +0900)
> >
> > ----------------------------------------------------------------
> > f2fs fixes for v3.8
> >
> > [Major bug fixes]
> > o Store device file information correctly
> > o Fix -EIO handling with respect to power-off-recovery
> > o Allocate blocks with global locks
> > o Fix wrong calculation of the SSR cost
> >
> > [Enhancement]
> > o Support (un)freeze_fs
> > o Enhance the f2fs_gc flow
> > o Support 32-bit binary execution on 64-bit kernel
> >
> > ----------------------------------------------------------------
> > Alejandro Martinez Ruiz (1):
> >       f2fs: fix disable_ext_identify option spelling
> >
> > Changman Lee (5):
> >       f2fs: save device node number into f2fs_inode
> >       f2fs: add un/freeze_fs into super_operations
> >       f2fs: stop repeated checking if cp is needed
> >       f2fs: remove repeated F2FS_SET_SB_DIRT call
> >       f2fs: remove unnecessary gc option check and balance_fs
> >
> > Jaegeuk Kim (6):
> >       f2fs: prevent checkpoint once any IO failure is detected
> >       f2fs: cover global locks for reserve_new_block
> >       f2fs: remove the use of page_cache_release
> >       f2fs: avoid balanc_fs during evict_inode
> >       f2fs: clarify and enhance the f2fs_gc flow
> >       f2fs: fix calculation of max. gc cost in the SSR case
> >
> > Namjae Jeon (8):
> >       f2fs: avoid redundant call to has_not_enough_free_secs in f2fs_gc
> >       f2fs: reorganize code for ra_node_page
> >       f2fs: fix typo mistake for data_version description
> >       f2fs: name gc task as per the block device
> >       f2fs: mark gc_thread as NULL when thread creation is failed
> >       f2fs: make an accessor to get sections for particular block type
> >       f2fs: optimize the return condition for has_not_enough_free_secs
> >       f2fs: add compat_ioctl to provide backward compatability
> >
> > majianpeng (4):
> >       f2fs: clean up the add_orphan_inode func
> >       f2fs: add device name in debugfs
> >       f2fs: use F2FS_BLKSIZE to judge bloksize and page_cache_size
> >       f2fs: when check superblock failed, try to check another
> > superblock
> >
> >  fs/f2fs/checkpoint.c |  63 +++++++++++---------------
> >  fs/f2fs/debug.c      |   4 +-
> >  fs/f2fs/f2fs.h       |  32 ++++++++++---
> >  fs/f2fs/file.c       |  35 ++++++++++++---
> >  fs/f2fs/gc.c         | 124
> > ++++++++++++++++++++++-----------------------------
> >  fs/f2fs/gc.h         |  21 ---------
> >  fs/f2fs/inode.c      |  53 +++++++++-------------
> >  fs/f2fs/node.c       |  14 +++---
> >  fs/f2fs/recovery.c   |   4 +-
> >  fs/f2fs/segment.c    |  29 ++++++++----
> >  fs/f2fs/segment.h    |  23 +++-------
> >  fs/f2fs/super.c      |  92 +++++++++++++++++++++++++++++---------
> >  12 files changed, 262 insertions(+), 232 deletions(-)
> >
> > --
> > Jaegeuk Kim
> > Samsung
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

-- 
Jaegeuk Kim
Samsung

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]