[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