Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Date:   Tue, 1 Mar 2022 08:00:00 -0800
From:   Luis Chamberlain <mcgrof@kernel.org>
To:     Christophe Leroy <christophe.leroy@csgroup.eu>
Cc:     Aaron Tomlin <atomlin@redhat.com>,
        "pmladek@suse.com" <pmladek@suse.com>,
        "cl@linux.com" <cl@linux.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>,
        "atomlin@atomlin.com" <atomlin@atomlin.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>,
        "jason.wessel@windriver.com" <jason.wessel@windriver.com>,
        "daniel.thompson@linaro.org" <daniel.thompson@linaro.org>
Subject: Re: [PATCH v9 00/14] module: core code clean up
Message-ID: <Yh5DAJxyaULX6p/L@bombadil.infradead.org>
References: <20220228234322.2073104-1-atomlin@redhat.com>
 <Yh1nGtM7MCMOI++l@bombadil.infradead.org>
 <6b6e02be-2d0a-31df-8375-baf79714d5b7@csgroup.eu>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <6b6e02be-2d0a-31df-8375-baf79714d5b7@csgroup.eu>
Sender: Luis Chamberlain <mcgrof@infradead.org>
Precedence: bulk

On Tue, Mar 01, 2022 at 07:07:30AM +0000, Christophe Leroy wrote:
> Hi Luis,
> 
> Le 01/03/2022 ? 01:21, Luis Chamberlain a ?crit?:
> > On Mon, Feb 28, 2022 at 11:43:08PM +0000, Aaron Tomlin wrote:
> >> Hi Luis,
> >>
> >> As per your suggestion [1], this is an attempt to refactor and split
> >> optional code out of core module support code into separate components.
> >> This version is based on Linus' commit 7993e65fdd0f ("Merge tag
> >> 'mtd/fixes-for-5.17-rc5' of
> >> git://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux").
> >>
> >> Petr,
> >>
> >> I decided to use preempt_disable() instead to remain consistent with the
> >> rest of the file. Unfortunately, I did not make time to boot test etc.
> > 
> > Aaron, thanks so much for doing this!
> > 
> > Since no boot tests are performed yet, I just pushed this to modules-testing
> > for now, after we get some boot test results I'll push to modules-next.
> > 
> > We should run kmod tests as well.
> > 
> 
> 
> My Kconfig patch and my two series still cleanly apply on v9. I don't 
> need to rebase/resend them.
> 
> 
> boot test is OK for me with Aaron's v9.

Great thanks yes, I intend on applying them as soon as I get some test
confirmations on modules-testing. It's all about testing now.

  Luis