Message ID | 20240508151951.1445074-1-andriy.shevchenko@linux.intel.com (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | [v1,1/1] lockd: Use *-y instead of *-objs in Makefile | expand |
On 8 May 2024, at 11:19, Andy Shevchenko wrote: > *-objs suffix is reserved rather for (user-space) host programs while > usually *-y suffix is used for kernel drivers (although *-objs works > for that purpose for now). > > Let's correct the old usages of *-objs in Makefiles. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Reviewed-by: Benjamin Coddington <bcodding@redhat.com> Ben
On Wed, May 08, 2024 at 06:19:38PM +0300, Andy Shevchenko wrote: > *-objs suffix is reserved rather for (user-space) host programs while > usually *-y suffix is used for kernel drivers (although *-objs works > for that purpose for now). > > Let's correct the old usages of *-objs in Makefiles. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > --- > > Note, the original approach is weirdest from the existing. > Only a few drivers use this (-objs-y) one most likely by mistake. > > fs/lockd/Makefile | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/fs/lockd/Makefile b/fs/lockd/Makefile > index ac9f9d84510e..fe3e23dd29c3 100644 > --- a/fs/lockd/Makefile > +++ b/fs/lockd/Makefile > @@ -7,8 +7,7 @@ ccflags-y += -I$(src) # needed for trace events > > obj-$(CONFIG_LOCKD) += lockd.o > > -lockd-objs-y += clntlock.o clntproc.o clntxdr.o host.o svc.o svclock.o \ > - svcshare.o svcproc.o svcsubs.o mon.o trace.o xdr.o > -lockd-objs-$(CONFIG_LOCKD_V4) += clnt4xdr.o xdr4.o svc4proc.o > -lockd-objs-$(CONFIG_PROC_FS) += procfs.o > -lockd-objs := $(lockd-objs-y) > +lockd-y := clntlock.o clntproc.o clntxdr.o host.o svc.o svclock.o \ > + svcshare.o svcproc.o svcsubs.o mon.o trace.o xdr.o > +lockd-$(CONFIG_LOCKD_V4) += clnt4xdr.o xdr4.o svc4proc.o > +lockd-$(CONFIG_PROC_FS) += procfs.o > -- > 2.43.0.rc1.1336.g36b5255a03ac > Acked-by: Chuck Lever <chuck.lever@oracle.com> Unless, of course, you'd like me to take this through the nfsd tree.
diff --git a/fs/lockd/Makefile b/fs/lockd/Makefile index ac9f9d84510e..fe3e23dd29c3 100644 --- a/fs/lockd/Makefile +++ b/fs/lockd/Makefile @@ -7,8 +7,7 @@ ccflags-y += -I$(src) # needed for trace events obj-$(CONFIG_LOCKD) += lockd.o -lockd-objs-y += clntlock.o clntproc.o clntxdr.o host.o svc.o svclock.o \ - svcshare.o svcproc.o svcsubs.o mon.o trace.o xdr.o -lockd-objs-$(CONFIG_LOCKD_V4) += clnt4xdr.o xdr4.o svc4proc.o -lockd-objs-$(CONFIG_PROC_FS) += procfs.o -lockd-objs := $(lockd-objs-y) +lockd-y := clntlock.o clntproc.o clntxdr.o host.o svc.o svclock.o \ + svcshare.o svcproc.o svcsubs.o mon.o trace.o xdr.o +lockd-$(CONFIG_LOCKD_V4) += clnt4xdr.o xdr4.o svc4proc.o +lockd-$(CONFIG_PROC_FS) += procfs.o
*-objs suffix is reserved rather for (user-space) host programs while usually *-y suffix is used for kernel drivers (although *-objs works for that purpose for now). Let's correct the old usages of *-objs in Makefiles. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- Note, the original approach is weirdest from the existing. Only a few drivers use this (-objs-y) one most likely by mistake. fs/lockd/Makefile | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-)