Skip to content

RFC making order in file2alias

Alessandro Rubini lkml 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 RFC Latest1 patch 6 repliesIn review
  2. v12 patches 7 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/1modpost: use table-lookup to build module aliasesscripts/mod/file2alias.c+60 -1061 reply

Reviews trailers per patch

The patch has a Reviewed-by or Acked-by.

PatchSamuel I. GonsalvezManohar VangaGreg Kroah-Hartman
1/1A-b A-b A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v2 thread

Cover letter [RFC PATCH V2 0/1] making order in file2alias

(this message is the RFC, the patch itself is expected to be fine)

This is a repost of what I've sent on Nov 4th. Since Rusty asked to only do the first step, here it is. I rebased on next-20111201 and it still works fine.

As for the previous 2/2 (see https://lkml.org/lkml/2011/11/4/127 ), I still think it makes sense. And as Dave Martin suggested, we can do without the array.

Thus, I may prepare three smaller steps, if that's acceptable (greg?)

step 1: create the ELF section so ENTRY() lines can leave the array
(and change name accordingly).

step 2: each ENTRY() line can be moved just after the associated code
(this means that a new bus is just a single hunk, not 2 of them)

step 3: I create the headers needed to move code and ENTRY in separate
files. This is some movement around, not trivial so it may deserve a patch in itself.

step 4 and later ones: Individual busses may reach their own external file,
conditionally compiled per Kconfig rules.

If that's something worth evaluating, I can do that over the weekend.

thanks
/alessandro

Alessandro Rubini (1):
modpost: use table-lookup to build module aliases

 scripts/mod/file2alias.c |  166 +++++++++++++++++-----------------------------
 1 files changed, 60 insertions(+), 106 deletions(-)
Signature · 1 line
1.7.7.2