Re: [PATCH v5 09/12] cifs: Cut over to using netfslib

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Feb 19, 2024 at 03:10:44PM +0000, David Howells wrote:
> Simon Horman <horms@xxxxxxxxxx> wrote:
> 
> > >  /* Functions related to files and directories */
> > > +extern const struct netfs_request_ops cifs_req_ops;
> > >  extern const struct file_operations cifs_file_ops;
> > >  extern const struct file_operations cifs_file_direct_ops; /* if directio mnt */
> > >  extern const struct file_operations cifs_file_strict_ops; /* if strictio mnt */
> > 
> > Nit: this hunk would probably be better placed in the
> >      patch at adds cifs_req_ops to fs/smb/client/file.c
> 
> I'm not sure I understand what you mean.  Is there a bit missing between "at"
> and "adds" in that?

Sorry, "patch that adds".

What I meant is, the declaration of cifs_req_ops and it's definition
seem to appear in different patches of this series. And it might
make sense if they were both in the same patch. But given that
both are present by the end of the series it is more cosmetic
than anything else.





[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [NTFS 3]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [NTFS 3]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux