[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230118074047.GA27385@lst.de>
Date: Wed, 18 Jan 2023 08:40:47 +0100
From: Christoph Hellwig <hch@....de>
To: Song Liu <song@...nel.org>
Cc: Thomas Gleixner <tglx@...utronix.de>,
Luis Chamberlain <mcgrof@...nel.org>,
Christophe Leroy <christophe.leroy@...roup.eu>,
songliubraving@...com, Peter Zijlstra <peterz@...radead.org>,
Christoph Hellwig <hch@....de>, linux-modules@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH/RFC] module: replace module_layout with module_memory
On Tue, Jan 17, 2023 at 10:50:55AM -0800, Song Liu wrote:
> Hi Thomas and Luis,
>
> Could you please share your comments on this? Specifically, is this on
> the right direction? And, what's your preference with Christophe's
> suggestions?
>
> "I dislike how it looks with enums, things like
> mod->mod_mem[MOD_MEM_TYPE_INIT
> _TEXT] are odd and don't read nicely.
> Could we have something nicer like mod->mod_mem_init_text ?
> I know it will complicate your for_each_mod_mem_type() but it would look
> nicer."
FYI, I don't particularly like the array either. But if it makes
the code much simpler I can live with it.
Powered by blists - more mailing lists