From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Simmons Date: Tue, 26 Jun 2018 02:13:22 +0100 (BST) Subject: [lustre-devel] [PATCH 00/24] lustre - more cleanups including module reduction. In-Reply-To: <87tvprah32.fsf@notabene.neil.brown.name> References: <152904663333.10587.10934053155404014785.stgit@noble> <3FD4D051-9C95-449D-8A23-D42B271E55B8@dilger.ca> <87tvprah32.fsf@notabene.neil.brown.name> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lustre-devel@lists.lustre.org > > Ah, thanks Andreas. Perhaps not coincidentally, Lustre 2.4 is the first release I worked on. > > > > Neil, I am really unenamored of the idea of the shared ptlrpc ldlm module being named ptlrpc... > > Can you say why? Is it the choice of name that bothers you, or the > combining of two things into the one kernel module, or something else? > Currently in git://git.hpdd.intel.com/fs/lustre-release.git > (or the git tree that was until recently at the above address), > the module named "ptlrpc" contains ptlrpc code, ldlm code, and target > code. I wonder what "target" means in this context. target is the server code built into ptlrpc which is for the "Unified Target". Its the shared code that is used by both the MDS and OSS code. Andreas or Oleg can add more details than this. BTW also the nodemap code also get built into ptlrpc module as well for the server side. > > Also, Lustre currently has a bunch of module parameters which are used for configuration. Thoughts on that? > > Yes, the module parameters are an interesting part of the story. > libcfs has a bunch of module parameters that are symlinked from debugfs. > It seems that user-space largely uses the debugfs links to access them, > so they can become part of the "lnet" module with minimal pain. > Other modules have parameters that I haven't yet looked in to. > > There probably will need to be user-space changes to support reduction > in the number of modules. The test suite will need some changes since it loads the modules. We need to handle the case if its built into the kernel as well.