Re: [PATCH RFC v2 4/6] inode: port __I_NEW to var event

[Christian Brauner <brauner@kernel.org>]

On Fri, Aug 23, 2024 at 10:20:31AM GMT, Christian Brauner wrote:
> On Fri, Aug 23, 2024 at 10:31:55AM GMT, NeilBrown wrote:
> > On Thu, 22 Aug 2024, Christian Brauner wrote:
> > > Port the __I_NEW mechanism to use the new var event mechanism.
> > > 
> > > Signed-off-by: Christian Brauner <brauner@kernel.org>
> > > ---
> > >  fs/bcachefs/fs.c          | 10 ++++++----
> > >  fs/dcache.c               |  3 +--
> > >  fs/inode.c                | 18 ++++++++----------
> > >  include/linux/writeback.h |  3 ++-
> > >  4 files changed, 17 insertions(+), 17 deletions(-)
> > > 
> > > diff --git a/fs/bcachefs/fs.c b/fs/bcachefs/fs.c
> > > index 94c392abef65..c0900c0c0f8a 100644
> > > --- a/fs/bcachefs/fs.c
> > > +++ b/fs/bcachefs/fs.c
> > > @@ -1644,14 +1644,16 @@ void bch2_evict_subvolume_inodes(struct bch_fs *c, snapshot_id_list *s)
> > >  				break;
> > >  			}
> > >  		} else if (clean_pass && this_pass_clean) {
> > > -			wait_queue_head_t *wq = bit_waitqueue(&inode->v.i_state, __I_NEW);
> > > -			DEFINE_WAIT_BIT(wait, &inode->v.i_state, __I_NEW);
> > > +			struct wait_bit_queue_entry wqe;
> > > +			struct wait_queue_head *wq_head;
> > >  
> > > -			prepare_to_wait(wq, &wait.wq_entry, TASK_UNINTERRUPTIBLE);
> > > +			wq_head = inode_bit_waitqueue(&wqe, &inode->v, __I_NEW);
> > 
> > I don't think you EXPORT inode_bit_waitqueue() so you cannot use it in
> > this module.
> 
> I had already fixed that in-tree because I got a build failure on one of
> my test machines.
> 
> > 
> > And maybe it would be good to not export it so that this code can get
> > cleaned up.
> 
> Maybe but I prefer this just to be a follow-up. So that we get hung up

*don't

> on ever more details.