Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 22 Feb 2022 13:37:08 +0000
From:   Aaron Tomlin <atomlin@atomlin.com>
To:     Christophe Leroy <christophe.leroy@csgroup.eu>
Cc:     Aaron Tomlin <atomlin@redhat.com>,
        "mcgrof@kernel.org" <mcgrof@kernel.org>,
        "cl@linux.com" <cl@linux.com>,
        "pmladek@suse.com" <pmladek@suse.com>,
        "mbenes@suse.cz" <mbenes@suse.cz>,
        "akpm@linux-foundation.org" <akpm@linux-foundation.org>,
        "jeyu@kernel.org" <jeyu@kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-modules@vger.kernel.org" <linux-modules@vger.kernel.org>,
        "void@manifault.com" <void@manifault.com>,
        "allen.lkml@gmail.com" <allen.lkml@gmail.com>,
        "joe@perches.com" <joe@perches.com>,
        "msuchanek@suse.de" <msuchanek@suse.de>,
        "oleksandr@natalenko.name" <oleksandr@natalenko.name>
Subject: Re: [PATCH v7 05/13] module: Move latched RB-tree support to a
 separate file
Message-ID: <20220222133708.tew4wxhkf5hpkg2u@ava.usersys.com>
References: <20220222130911.1348513-1-atomlin@redhat.com>
 <20220222130911.1348513-6-atomlin@redhat.com>
 <8ed19f41-625a-154f-096c-ae7ea19a9649@csgroup.eu>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <8ed19f41-625a-154f-096c-ae7ea19a9649@csgroup.eu>
Precedence: bulk

On Tue 2022-02-22 13:32 +0000, Christophe Leroy wrote:
> Le 22/02/2022 à 14:09, Aaron Tomlin a écrit :
> > No functional change.
> > 
> > This patch migrates module latched RB-tree support
> > (e.g. see __module_address()) from core module code
> > into kernel/module/tree_lookup.c.
> > 
> > Signed-off-by: Aaron Tomlin <atomlin@redhat.com>
> > ---
> >   kernel/module/Makefile      |   3 +
> >   kernel/module/internal.h    |  33 +++++++++
> >   kernel/module/main.c        | 130 ++----------------------------------
> >   kernel/module/tree_lookup.c | 109 ++++++++++++++++++++++++++++++
> >   4 files changed, 149 insertions(+), 126 deletions(-)
> >   create mode 100644 kernel/module/tree_lookup.c
> > 
> > diff --git a/kernel/module/Makefile b/kernel/module/Makefile
> > index ed3aacb04f17..e8413975bf1d 100644
> > --- a/kernel/module/Makefile
> > +++ b/kernel/module/Makefile
> > @@ -11,3 +11,6 @@ obj-y += main.o
> >   obj-$(CONFIG_MODULE_DECOMPRESS) += decompress.o
> >   obj-$(CONFIG_MODULE_SIG) += signing.o
> >   obj-$(CONFIG_LIVEPATCH) += livepatch.o
> > +ifdef CONFIG_MODULES
> 
> This ifdef is not needed anymore.
> 
> > +obj-$(CONFIG_MODULES_TREE_LOOKUP) += tree_lookup.o
> > +endif

Oops! Apologies this was an oversight.

-- 
Aaron Tomlin