[lustre-devel] [PATCH 00/24] lustre - more cleanups including module reduction.

NeilBrown neilb at suse.com
Sun Jun 24 16:58:04 PDT 2018


On Sun, Jun 24 2018, James Simmons wrote:

>> >> Here are a bunch more cleanups that will appear in my lustre-testing
>> >> tree shortly.
>> >> 
>> >> The interesting stuff is some enhancements to kbuild to let us
>> >> build modules from multiple directories.
>> >> I've used that to clean up the build of ptlrpc+ldlm and to
>> >> merge libcfs and lnet into the one module - I've left the lnds
>> >> separate.
>> >
>> > That looks pretty good. It doesn't apply to the stable lustre tree but I
>> > did try it out on lustre-testing. The module building does seem to 
>> > function well. Only thing not clear is if do a
>> >
>> > make SUBDIRS=drivers/staging/lustre/lustre/lmv modules
>> 
>> Why would you do that?  SUBDIRS= is for building external modules.
>> What are you trying to achieve?
>> 
>> >
>> > do lmv.a and the other *.a files get relinked int lustre.ko ?
>> 
>> It isn't "lmv.a", it is ".../lustre/lmv/mod.a".
>
> Sorry I forgot the mod.a naming. I was asking to get a feel of how the
> work flow would change. It is common for lustre developers to build
> in tree and test without installing anything. The many module approach
> might of been a way for people to rebuild a module, unload module and
> then reload module quickly to test a change. I don't work this way so
> I can't say. I was looking to see if that work flow could still be
> done.

This general approach should still work.
You wouldn't do
  make SUBDIRS=drivers/staging/lustre/lustre/lmv modules

unless there was a module built in that directory, but you might

  make SUBDIRS=drivers/staging/lustre modules

to just build whatever lustre modules there are.

NeilBrown

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <http://lists.lustre.org/pipermail/lustre-devel-lustre.org/attachments/20180625/d09e4f75/attachment.sig>


More information about the lustre-devel mailing list