[ANNOUNCE] xfs-linux: for-next updated to 9ee85f235efe

[ANNOUNCE] xfs-linux: for-next updated to 9ee85f235efe

[Chandan Babu R]

Hi folks,

The for-next branch of the xfs-linux repository at:

	https://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git

has just been updated.

Patches often get missed, so please check if your outstanding patches
were in this update. If they have not been in this update, please
resubmit them to linux-xfs@vger.kernel.org so they can be picked up in
the next update.

The new head of the for-next branch is commit:

9ee85f235efe xfs: ensure submit buffers on LSN boundaries in error handlers

35 new commits:

Christoph Hellwig (17):
      [f23e079e024c] mm: move mapping_set_update out of <linux/swap.h>
      [604ee858a8c8] shmem: move shmem_mapping out of line
      [8481cd645af6] shmem: set a_ops earlier in shmem_symlink
      [9b4ec2cf0154] shmem: move the shmem_mapping assert into shmem_get_folio_gfp
      [36e3263c623a] shmem: export shmem_get_folio
      [74f6fd19195a] shmem: export shmem_kernel_file_setup
      [eb84b86441e3] shmem: document how to "persist" data when using shmem_*file_setup
      [7712a852890f] xfs: remove xfile_stat
      [c6fcb46eb7ae] xfs: remove the xfile_pread/pwrite APIs
      [8672b3a8172c] xfs: don't try to handle non-update pages in xfile_obj_load
      [7391e9df8844] xfs: shmem_file_setup can't return NULL
      [2ac41afb75c3] xfs: don't modify file and inode flags for shmem files
      [6d8b3f1209e4] xfs: don't allow highmem pages in xfile mappings
      [509f2f9bb384] xfs: use shmem_get_folio in xfile_obj_store
      [ae2bb8c31c36] xfs: use shmem_get_folio in in xfile_load
      [167db3241ef0] xfs: remove xfarray_sortinfo.page_kaddr
      [97456c5eb256] xfs: fix a comment in xfarray.c

Darrick J. Wong (4):
      [513911e94428] xfs: add file_{get,put}_folio
      [cca31e401b4a] xfs: convert xfarray_pagesort to deal with large folios
      [42461a423bfe] xfs: remove xfile_{get,put}_page
      [6ef2978cfed8] xfs: disable sparse inode chunk alignment check when there is no alignment

Dave Chinner (12):
      [10634530f7ba] xfs: convert kmem_zalloc() to kzalloc()
      [f078d4ea8276] xfs: convert kmem_alloc() to kmalloc()
      [afdc115559c5] xfs: move kmem_to_page()
      [49292576136f] xfs: convert kmem_free() for kvmalloc users to kvfree()
      [d4c75a1b40cd] xfs: convert remaining kmem_free() to kfree()
      [178231af2bdc] xfs: use an empty transaction for fstrim
      [94a69db2367e] xfs: use __GFP_NOLOCKDEP instead of GFP_NOFS
      [0b3a76e955eb] xfs: use GFP_KERNEL in pure transaction contexts
      [2c1e31ed5c88] xfs: place intent recovery under NOFS allocation context
      [c704ecb2410e] xfs: place the CIL under nofs allocation context
      [204fae32d5f7] xfs: clean up remaining GFP_NOFS users
      [57b98393b812] xfs: use xfs_defer_alloc a bit more

Long Li (1):
      [9ee85f235efe] xfs: ensure submit buffers on LSN boundaries in error handlers

Shrikanth Hegde (1):
      [4d6933c1c21a] xfs: remove duplicate ifdefs

Code Diffstat:

 .../filesystems/xfs/xfs-online-fsck-design.rst    |  25 +-
 fs/xfs/Makefile                                   |   3 +-
 fs/xfs/kmem.c                                     |  30 --
 fs/xfs/kmem.h                                     |  83 -----
 fs/xfs/libxfs/xfs_ag.c                            |  10 +-
 fs/xfs/libxfs/xfs_attr.c                          |   3 +-
 fs/xfs/libxfs/xfs_attr_leaf.c                     |  18 +-
 fs/xfs/libxfs/xfs_bmap.c                          |   2 +-
 fs/xfs/libxfs/xfs_btree.c                         |   2 +-
 fs/xfs/libxfs/xfs_btree.h                         |   4 +-
 fs/xfs/libxfs/xfs_btree_staging.c                 |  10 +-
 fs/xfs/libxfs/xfs_da_btree.c                      |  22 +-
 fs/xfs/libxfs/xfs_defer.c                         |  23 +-
 fs/xfs/libxfs/xfs_dir2.c                          |  48 ++-
 fs/xfs/libxfs/xfs_dir2_block.c                    |   6 +-
 fs/xfs/libxfs/xfs_dir2_sf.c                       |  16 +-
 fs/xfs/libxfs/xfs_iext_tree.c                     |  26 +-
 fs/xfs/libxfs/xfs_inode_fork.c                    |  29 +-
 fs/xfs/libxfs/xfs_refcount.c                      |   2 +-
 fs/xfs/libxfs/xfs_rmap.c                          |   2 +-
 fs/xfs/scrub/cow_repair.c                         |   2 +-
 fs/xfs/scrub/ialloc_repair.c                      |   2 +-
 fs/xfs/scrub/rtsummary.c                          |   6 +-
 fs/xfs/scrub/trace.h                              |  81 +++--
 fs/xfs/scrub/xfarray.c                            | 234 ++++++-------
 fs/xfs/scrub/xfarray.h                            |  11 +-
 fs/xfs/scrub/xfile.c                              | 345 +++++++------------
 fs/xfs/scrub/xfile.h                              |  62 +---
 fs/xfs/xfs_acl.c                                  |   4 +-
 fs/xfs/xfs_attr_item.c                            |  14 +-
 fs/xfs/xfs_attr_list.c                            |   6 +-
 fs/xfs/xfs_bmap_item.c                            |   7 +-
 fs/xfs/xfs_bmap_util.c                            |   2 +-
 fs/xfs/xfs_buf.c                                  |  48 +--
 fs/xfs/xfs_buf_item.c                             |   8 +-
 fs/xfs/xfs_buf_item_recover.c                     |   8 +-
 fs/xfs/xfs_discard.c                              |  17 +-
 fs/xfs/xfs_dquot.c                                |   2 +-
 fs/xfs/xfs_error.c                                |   8 +-
 fs/xfs/xfs_extent_busy.c                          |   5 +-
 fs/xfs/xfs_extfree_item.c                         |   8 +-
 fs/xfs/xfs_filestream.c                           |   6 +-
 fs/xfs/xfs_icache.c                               |   5 +-
 fs/xfs/xfs_icreate_item.c                         |   2 +-
 fs/xfs/xfs_inode.c                                |   4 +-
 fs/xfs/xfs_inode_item.c                           |   2 +-
 fs/xfs/xfs_inode_item_recover.c                   |   5 +-
 fs/xfs/xfs_ioctl.c                                |   8 +-
 fs/xfs/xfs_iops.c                                 |   2 +-
 fs/xfs/xfs_itable.c                               |  12 +-
 fs/xfs/xfs_iwalk.c                                |   9 +-
 fs/xfs/xfs_linux.h                                |  14 +-
 fs/xfs/xfs_log.c                                  |  20 +-
 fs/xfs/xfs_log_cil.c                              |  31 +-
 fs/xfs/xfs_log_recover.c                          | 101 ++++--
 fs/xfs/xfs_mount.c                                |   2 +-
 fs/xfs/xfs_mru_cache.c                            |  17 +-
 fs/xfs/xfs_qm.c                                   |  18 +-
 fs/xfs/xfs_refcount_item.c                        |  12 +-
 fs/xfs/xfs_rmap_item.c                            |  11 +-
 fs/xfs/xfs_rtalloc.c                              |  10 +-
 fs/xfs/xfs_super.c                                |   4 +-
 fs/xfs/xfs_sysfs.c                                |   4 -
 fs/xfs/xfs_trace.h                                |  25 --
 fs/xfs/xfs_trans_ail.c                            |   7 +-
 include/linux/shmem_fs.h                          |   6 +-
 include/linux/swap.h                              |  10 -
 mm/filemap.c                                      |   9 +
 mm/internal.h                                     |   4 +
 mm/shmem.c                                        |  37 +-
 mm/workingset.c                                   |   1 +
 71 files changed, 722 insertions(+), 920 deletions(-)
 delete mode 100644 fs/xfs/kmem.c
 delete mode 100644 fs/xfs/kmem.h

shmem patches headsup: Re: [ANNOUNCE] xfs-linux: for-next updated to 9ee85f235efe

[Christoph Hellwig]

On Wed, Feb 14, 2024 at 12:18:41PM +0530, Chandan Babu R wrote:
> The for-next branch of the xfs-linux repository at:
> 
> 	https://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git
> 
> has just been updated.

<snip>

> Christoph Hellwig (17):
>       [f23e079e024c] mm: move mapping_set_update out of <linux/swap.h>
>       [604ee858a8c8] shmem: move shmem_mapping out of line
>       [8481cd645af6] shmem: set a_ops earlier in shmem_symlink
>       [9b4ec2cf0154] shmem: move the shmem_mapping assert into shmem_get_folio_gfp
>       [36e3263c623a] shmem: export shmem_get_folio
>       [74f6fd19195a] shmem: export shmem_kernel_file_setup
>       [eb84b86441e3] shmem: document how to "persist" data when using shmem_*file_setup

I would have prefer an ACK or even a shared branch in the MM tree
for these.  But as it's been impossible to get any feedback from
the shmem and mm maintainer maybe this is the right thing to do.

Andrew, Hugh: can you commet if this is ok?

Re: shmem patches headsup: Re: [ANNOUNCE] xfs-linux: for-next updated to 9ee85f235efe

[Hugh Dickins]

On Wed, 14 Feb 2024, Christoph Hellwig wrote:
> On Wed, Feb 14, 2024 at 12:18:41PM +0530, Chandan Babu R wrote:
> > The for-next branch of the xfs-linux repository at:
> > 
> > 	https://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git
> > 
> > has just been updated.
> 
> <snip>
> 
> > Christoph Hellwig (17):
> >       [f23e079e024c] mm: move mapping_set_update out of <linux/swap.h>
> >       [604ee858a8c8] shmem: move shmem_mapping out of line
> >       [8481cd645af6] shmem: set a_ops earlier in shmem_symlink
> >       [9b4ec2cf0154] shmem: move the shmem_mapping assert into shmem_get_folio_gfp
> >       [36e3263c623a] shmem: export shmem_get_folio
> >       [74f6fd19195a] shmem: export shmem_kernel_file_setup
> >       [eb84b86441e3] shmem: document how to "persist" data when using shmem_*file_setup
> 
> I would have prefer an ACK or even a shared branch in the MM tree
> for these.  But as it's been impossible to get any feedback from
> the shmem and mm maintainer maybe this is the right thing to do.
> 
> Andrew, Hugh: can you commet if this is ok?

Each day I hope to reach looking at it.  I sincerely believed that
I would get to it yesterday, but no.  Later on today?  We shall see.

Hugh

Re: shmem patches headsup: Re: [ANNOUNCE] xfs-linux: for-next updated to 9ee85f235efe

[Chandan Babu R]

On Wed, Feb 14, 2024 at 09:03:05 AM +0100, Christoph Hellwig wrote:
> On Wed, Feb 14, 2024 at 12:18:41PM +0530, Chandan Babu R wrote:
>> The for-next branch of the xfs-linux repository at:
>> 
>> 	https://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git
>> 
>> has just been updated.
>
> <snip>
>
>> Christoph Hellwig (17):
>>       [f23e079e024c] mm: move mapping_set_update out of <linux/swap.h>
>>       [604ee858a8c8] shmem: move shmem_mapping out of line
>>       [8481cd645af6] shmem: set a_ops earlier in shmem_symlink
>>       [9b4ec2cf0154] shmem: move the shmem_mapping assert into shmem_get_folio_gfp
>>       [36e3263c623a] shmem: export shmem_get_folio
>>       [74f6fd19195a] shmem: export shmem_kernel_file_setup
>>       [eb84b86441e3] shmem: document how to "persist" data when using shmem_*file_setup
>
> I would have prefer an ACK or even a shared branch in the MM tree
> for these.  But as it's been impossible to get any feedback from
> the shmem and mm maintainer maybe this is the right thing to do.
>

I am sorry. I completely forgot about the requirement for an ack from the MM
maintainers. Thanks for bringing it to notice.

-- 
Chandan

Re: shmem patches headsup: Re: [ANNOUNCE] xfs-linux: for-next updated to 9ee85f235efe

[Darrick J. Wong]

On Wed, Feb 14, 2024 at 01:51:42PM +0530, Chandan Babu R wrote:
> On Wed, Feb 14, 2024 at 09:03:05 AM +0100, Christoph Hellwig wrote:
> > On Wed, Feb 14, 2024 at 12:18:41PM +0530, Chandan Babu R wrote:
> >> The for-next branch of the xfs-linux repository at:
> >> 
> >> 	https://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git
> >> 
> >> has just been updated.
> >
> > <snip>
> >
> >> Christoph Hellwig (17):
> >>       [f23e079e024c] mm: move mapping_set_update out of <linux/swap.h>
> >>       [604ee858a8c8] shmem: move shmem_mapping out of line
> >>       [8481cd645af6] shmem: set a_ops earlier in shmem_symlink
> >>       [9b4ec2cf0154] shmem: move the shmem_mapping assert into shmem_get_folio_gfp
> >>       [36e3263c623a] shmem: export shmem_get_folio
> >>       [74f6fd19195a] shmem: export shmem_kernel_file_setup
> >>       [eb84b86441e3] shmem: document how to "persist" data when using shmem_*file_setup
> >
> > I would have prefer an ACK or even a shared branch in the MM tree
> > for these.  But as it's been impossible to get any feedback from
> > the shmem and mm maintainer maybe this is the right thing to do.
> >
> 
> I am sorry. I completely forgot about the requirement for an ack from the MM
> maintainers. Thanks for bringing it to notice.

These seven patches have been out for review for nineteen days.

Patches 4, 5, and 7 have been out for review for FORTY TWO DAYS.

willy reviewed them after I asked him (thank you willy!), but this kind
of lead time for fairly minor patches is unworkable.

If you two are so overworked that you cannot provide feedback in under
six weeks, then I really need you to ask your manager for more help
hiring staff so that you can delegate tasks and unburden yourselves.
Stalling everyone else is a shitty thing to do.  Long feedback cycles
are destructive to developing things together -- look at what XFS has
become.

Or just let the patches go in and hch and I will deal with the
regression reports.  Maybe we'll even learn a few things in the process.
Spreading knowledge around the community and decentralizing to reduce
bus factor are two key points of free software, right?

In the meantime, this is blocking me from preparing online repair pull
requests for XFS for 6.9 because those patches need the stuff at the end
of the diet-v3 series.

--D

> -- 
> Chandan
> 

Re: shmem patches headsup: Re: [ANNOUNCE] xfs-linux: for-next updated to 9ee85f235efe

[Hugh Dickins]

On Wed, 14 Feb 2024, Hugh Dickins wrote:
> On Wed, 14 Feb 2024, Christoph Hellwig wrote:
> > On Wed, Feb 14, 2024 at 12:18:41PM +0530, Chandan Babu R wrote:
> > > The for-next branch of the xfs-linux repository at:
> > > 
> > > 	https://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git
> > > 
> > > has just been updated.
> > 
> > <snip>
> > 
> > > Christoph Hellwig (17):
> > >       [f23e079e024c] mm: move mapping_set_update out of <linux/swap.h>
> > >       [604ee858a8c8] shmem: move shmem_mapping out of line
> > >       [8481cd645af6] shmem: set a_ops earlier in shmem_symlink
> > >       [9b4ec2cf0154] shmem: move the shmem_mapping assert into shmem_get_folio_gfp
> > >       [36e3263c623a] shmem: export shmem_get_folio
> > >       [74f6fd19195a] shmem: export shmem_kernel_file_setup
> > >       [eb84b86441e3] shmem: document how to "persist" data when using shmem_*file_setup
> > 
> > I would have prefer an ACK or even a shared branch in the MM tree
> > for these.  But as it's been impossible to get any feedback from
> > the shmem and mm maintainer maybe this is the right thing to do.
> > 
> > Andrew, Hugh: can you commet if this is ok?
> 
> Each day I hope to reach looking at it.  I sincerely believed that
> I would get to it yesterday, but no.  Later on today?  We shall see.

Christoph, I'd better give you a progress report, to avoid another mail
as foul as was sent after yours.

I wonder what the fuss is: Matthew (thank you) has been giving excellent
reviews, and mm is a cooperative not a dickinstatorship (but I was upset
that he caught that GFP_HIGHUSER, which I had been eager to point out).

I haven't finished yet, but made good progress and it does look good:
a few very minor comments so far, nothing that couldn't be patched up
later if that suits you best.

mm.git contains no updates to mm/shmem.c yet this cycle, so I expect
that Andrew will be fine with the series going in via the XFS tree,
unless some awkward conflict in one of the mm files appears.

Tomorrow,
Hugh

Re: shmem patches headsup: Re: [ANNOUNCE] xfs-linux: for-next updated to 9ee85f235efe

[Matthew Wilcox]

On Wed, Feb 14, 2024 at 11:39:06PM -0800, Hugh Dickins wrote:
> I wonder what the fuss is: Matthew (thank you) has been giving excellent
> reviews, and mm is a cooperative not a dickinstatorship (but I was upset
> that he caught that GFP_HIGHUSER, which I had been eager to point out).

Sorry; it's always frustrating when somebody else says the same thing
you were going to say ;-)