Received-SPF: pass (google.com: domain of linux-kernel+bounces-176759-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
From: Kris Van Hees <kris.van.hees@oracle.com>
To: linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org,
        linux-modules@vger.kernel.org, linux-trace-kernel@vger.kernel.org
Cc: Kris Van Hees <kris.van.hees@oracle.com>,
        Steven Rostedt <rostedt@goodmis.org>,
        Luis Chamberlain <mcgrof@kernel.org>,
        Masami Hiramatsu <mhiramat@kernel.org>,
        Nick Desaulniers <ndesaulniers@google.com>,
        Jiri Olsa <olsajiri@gmail.com>,
        Elena Zannoni <elena.zannoni@oracle.com>
Subject: [PATCH v2 0/6] Generate address range data for built-in modules
Date: Sat, 11 May 2024 18:40:29 -0400
Message-ID: <20240511224035.27775-1-kris.van.hees@oracle.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: 
	=?us-ascii?Q?oEE3HYbeDhTt7ZjUaWYqRl3wDCFWmaZUP99rLc+hLMDBwxm2NBlGPzHdjshD?=
 =?us-ascii?Q?Hc3+AVA6IiEqScdtBU0dLcxY6T9tgEkHmC8nw0fsI7HhrGVjry3GCexpazBP?=
 =?us-ascii?Q?VoUDbQLz14bErT9c9mWzJm1BOfNDfB4qKfJv2eO1aFxxm/GveWiubECubLzd?=
 =?us-ascii?Q?7eLiLQg9Pt6YvUzP5GynmCz2sZasMRRUeKdKQ0naOen6cV/UaLP9QMWzoMWA?=
 =?us-ascii?Q?NaTEHs7l/ZL3i73eLhf8m5sez4PIAROjaXxir8pD1hGs0S3NrKpIQFe/tYrO?=
 =?us-ascii?Q?SIVIMI/a1QULS2wRX904pb44Idxw8z88++/2cpRAZhriyxtATyaBW4N39N2c?=
 =?us-ascii?Q?D7T2Jd5ov2YUyPNVwfm7hisNGH6By+SgZg0F8V9qjJ1WU6FUikOcXC9RZSMs?=
 =?us-ascii?Q?P0DgjYnx/9lsBEb5uO1k3Q7rgXdl0pYRbUqZKaVP5imdK7om5ktAKN9H34eN?=
 =?us-ascii?Q?sqNnfJ/LSU/o4Veh1aVMQq7cHdrXB3+awnG4JG1l2VIdJdq32Br/8Gz6cafk?=
 =?us-ascii?Q?kERf76336YHVPPGeg83nMBaMxyISkhTZHjItzd7va27ekcgiT+w1h4SQ+/aL?=
 =?us-ascii?Q?c9l+S+Wiuh0LcajA4FhQQLx8HVygybWbaB2bhExMbVD5hHRSEoMB1NYEw7jg?=
 =?us-ascii?Q?4C51dbw0YUtdgIXuTp/0pK4w1mML8vDXjM3nDMJYrSQlFTQ5ry0Ocp8x7oV2?=
 =?us-ascii?Q?VlqLYJomXvK0pPvCEdLekwHVXpTSYhi1wv189IUAwz5Kk//vGeeny/Qjd3GF?=
 =?us-ascii?Q?MjbiUwbpDu0objQqaGdT/HJJAH/DEXXYd8ouxh7d3Znr+7DR3WG29ATSibb5?=
 =?us-ascii?Q?NN+0Q+apkw0YgtUIphGpE3CP14vzrumKJMhmK5YE6xm6pDKHx6lgU9XrijOR?=
 =?us-ascii?Q?yihHjOk6/6EJMN5iOGNXucxbCyIugskH6IGH0UA1DPSIztPfjTKmk+4lQSD3?=
 =?us-ascii?Q?dXVqxiVfbxxahTOns6EXqLh7SYwhpRcjFwFWku9v6i2FDNMjds51G+zjRnRf?=
 =?us-ascii?Q?J3fE9PMlnnKjbb9T0I5xjDL5WsPYXTO0ZqbZcSSpg5oJFxJJwNKZoxx78rJO?=
 =?us-ascii?Q?IQC6UEGdfDdD7d2PhMvUnDwx5oorV8NKkx2Iw7IrnbYdiSYQ26XHOszQRFwu?=
 =?us-ascii?Q?G8YLRsr0BAcqR4YwsmNr69Qh2J3TTOCKqHzBSU6a3JsO4xPpuHBaL7ADXws/?=
 =?us-ascii?Q?1RaN4lkGnrFGsyQG6JBc2UGdtoHb1/JR1BZ67i8Jviy9tGnskczxl7t/U7dR?=
 =?us-ascii?Q?InLWBlJ5FH5vRWbX5mKDtFJWMe98kuD2C7GzMcUQQ6kEQN3tbmir6u8ryd72?=
 =?us-ascii?Q?ykZLaUrD3dB0GghWMI6T2WwhBp85ZntDMMaXjQq9f7Cu7ykB9GzXwhmFDvcG?=
 =?us-ascii?Q?q6ZbjZvz8Kt94hyKu+5v3/bF+aU0R2CJbD3tLjUUZTmgQoRENuh0jeLhb+rO?=
 =?us-ascii?Q?cp3CcxRymRn1jc7i1I+LfVvwhsE/21kXt1m8cAzlUuEj+Qdf8lZ1oL6MbYuZ?=
 =?us-ascii?Q?5/ZnAlPMU4V5EmczXX9HgExp33JbS98VjaLrPvgz7RcoYqiKMQ3C6rMqpxrN?=
 =?us-ascii?Q?OhA0rOUZNLETWBpkR/IlJJV19g9N/j4PauVfsisBrd2EFx0rJ521zEZhEgr2?=
 =?us-ascii?Q?Rw=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 
	XEMO6zTZPX5ZodirH87HquMhC2/ANb2weGPldOdSA3GHv048lK1iH7qmoJoOjn36nUFNch1Fj4Sm5OdpnVSlIOw9wup2apnbtHfrCt3VQCp2RS3hqow/MVVD+000DNxiNoh7RPKs6lkcr+EAHYtxDoonYX1+qmF3Dq0OPWMoKmq4H8qHjgGoyVbBKKsPpXaaVhJ/l8UtFaKMeZropti3DVHSsL4RzUcXr3q93e53s8MvfCCHlompqeZEwUeFfVLqELIXQyLKz50muL+BsGDXxp71LUB8IY9LN8HSUjey9cG7wCcCHwrtg/O3B+jei+idkR101qXEfnwWPuw0eeCuoqdNSViQ58lzY67YCcQ2lzfZ5dtPA5R4W90gaMB1hVIq8fx+RoB7h4uxMvnMd3wgKMVXmiNE8zeuien5H7X5Cbywn4hrhPLaf779XyTzrtCbEDWZowLIIZP+wjeg2yEHHnjxRkB9bCyE+RqnlF/AAV6YVPosk5F2uqXpgzI9EWp0HGWet/azRgwuZZRNeE+9JflZ+4Z6uH6SznQtHvupTWxfTpaqHHXZ8Ga7YLplvwNL/7U7c8hNgVyRJhDp4YQZPSFSQZJHNw4rCPVBMHBmDxM=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: db3b2e8e-630d-44aa-29f1-08dc720b78eb
X-MS-Exchange-CrossTenant-AuthSource: SN7PR10MB6287.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 May 2024 22:41:17.9362
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: +X1T1OfzIziAltqFLtSHGqOsma5D+fRCjzr3YgzCllLpsYefxs463l/rH74hY5FIAHpxpxHtUDze22msvrtB200PbbH2gTnfPjVh54YQaQ0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR10MB5184
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.293,Aquarius:18.0.1039,Hydra:6.0.650,FMLib:17.11.176.26
 definitions=2024-05-11_06,2024-05-10_02,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 adultscore=0
 mlxlogscore=999 phishscore=0 malwarescore=0 suspectscore=0 mlxscore=0
 spamscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2405010000 definitions=main-2405110171
X-Proofpoint-GUID: 77_nOWvCApU5DkrqUM0Wi-5lurJTH5Bm
X-Proofpoint-ORIG-GUID: 77_nOWvCApU5DkrqUM0Wi-5lurJTH5Bm

Especially for tracing applications, it is convenient to be able to
refer to a symbol using a <module name, symbol name> pair and to be able
to translate an address into a <nodule mname, symbol name> pair.  But
that does not work if the module is built into the kernel because the
object files that comprise the built-in module implementation are simply
linked into the kernel image along with all other kernel object files.

This is especially visible when providing tracing scripts for support
purposes, where the developer of the script targets a particular kernel
version, but does not have control over whether the target system has
a particular module as loadable module or built-in module.  When tracing
symbols within a module, referring them by <module name, symbol name>
pairs is both convenient and aids symbol lookup.  But that naming will
not work if the module name information is lost if the module is built
into the kernel on the target system.

Earlier work addressing this loss of information for built-in modules
involved adding module name information to the kallsyms data, but that
required more invasive code in the kernel proper.  This work never did
get merged into the kernel tree.

All that is really needed is knowing whether a given address belongs to
a particular module (or multiple modules if they share an object file).
Or in other words, whether that address falls within an address range
that is associated with one or more modules.

This patch series is baaed on Luis Chamberlain's patch to generate
modules.builtin.objs, associating built-in modules with their object
files.  Using this data, vmlinux.o.map and vmlinux.map can be parsed in
a single pass to generate a modules.buitin.ranges file with offset range
information (relative to the base address of the associated section) for
built-in modules.  The file gets installed along with the other
modules.builtin.* files.

The impact on the kernel build is minimal because everything is done
using a single-pass AWK script.  The generated data size is minimal as
well, (depending on the exact kernel configuration) usually in the range
of 500-700 lines, with a file size of 20-40KB.

Changes since v1:
 - Renamed CONFIG_BUILTIN_RANGES to CONFIG_BUILTIN_MODULE_RANGES
 - Moved the config option to the tracers section
 - 2nd arg to generate_builtin_ranges.awk should be vmlinux.map

Kris Van Hees (5):
  trace: add CONFIG_BUILTIN_MODULE_RANGES option
  kbuild: generate a linker map for vmlinux.o
  module: script to generate offset ranges for builtin modules
  kbuild: generate modules.builtin.ranges when linking the kernel
  module: add install target for modules.builtin.ranges

Luis Chamberlain (1):
  kbuild: add modules.builtin.objs

 .gitignore                          |   2 +-
 Documentation/dontdiff              |   2 +-
 Documentation/kbuild/kbuild.rst     |   5 ++
 Makefile                            |   8 +-
 include/linux/module.h              |   4 +-
 kernel/trace/Kconfig                |  17 ++++
 scripts/Makefile.lib                |   5 +-
 scripts/Makefile.modinst            |  11 ++-
 scripts/Makefile.vmlinux            |  17 ++++
 scripts/Makefile.vmlinux_o          |  18 ++++-
 scripts/generate_builtin_ranges.awk | 149 ++++++++++++++++++++++++++++++++++++
 11 files changed, 228 insertions(+), 10 deletions(-)
 create mode 100755 scripts/generate_builtin_ranges.awk


base-commit: dd5a440a31fae6e459c0d6271dddd62825505361
-- 
2.42.0