Le 03/03/2022 à 18:54, Christoph Hellwig a écrit :

On Thu, Mar 03, 2022 at 02:59:49PM +0000, Daniel Thompson wrote:

quoted

One of the best ways that we can common up code might be to dust
off some code I wrote a while back to display seq_files from
kdb.

The basic idea worked well enough but it often needs special
start/stop operatings to ensure the start meeds kdb's rather
odd locking restrictions. If there is a willingness for
something like the below to be included in the module code then we
could replace kdb_lsmod() with something that reused the code to
  format /proc/modules.

Displaying seq_files sounds nice to have, but in the short term I'm
just thinking of something like this:

Well .... The idea at the first place was to get rid of the #ifdef 
CONFIG_KGDB_KDB in modules.

Here you propose it the other way round. Why not, in that case that 
would mean a dedicated file in kernel/module/ as part of the series 
https://patchwork.kernel.org/project/linux-modules/list/?series=618917&state=*

Christophe