Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2;
Date:   Sun, 29 Oct 2023 14:50:33 +0100
From:   Miquel Raynal <miquel.raynal@bootlin.com>
To:     Richard Weinberger <richard@nod.at>
Cc:     ZhaoLong Wang <wangzhaolong1@huawei.com>,
        Vignesh Raghavendra <vigneshr@ti.com>,
        Artem Bityutskiy <Artem.Bityutskiy@nokia.com>,
        dpervushin <dpervushin@embeddedalley.com>,
        linux-mtd <linux-mtd@lists.infradead.org>,
        linux-kernel <linux-kernel@vger.kernel.org>,
        chengzhihao1 <chengzhihao1@huawei.com>,
        yi zhang <yi.zhang@huawei.com>,
        yangerkun <yangerkun@huawei.com>
Subject: Re: [PATCH v3] mtd: Fix gluebi NULL pointer dereference caused by
 ftl notifier
Message-ID: <20231029145033.0ad2682a@xps-13>
In-Reply-To: <174426702.13324.1698438610904.JavaMail.zimbra@nod.at>
References: <20231027012033.50280-1-wangzhaolong1@huawei.com>
        <20231027194026.1bc32dfe@xps-13>
        <174426702.13324.1698438610904.JavaMail.zimbra@nod.at>
Organization: Bootlin
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi Richard,

richard@nod.at wrote on Fri, 27 Oct 2023 22:30:10 +0200 (CEST):

> ----- Urspr=C3=BCngliche Mail -----
> > Von: "Miquel Raynal" <miquel.raynal@bootlin.com> =20
> >> Detailed reproduction information available at the link[1],
> >>=20
> >> The solution for the gluebi module is to run jffs2 on the UBI
> >> volume without considering working with ftl or mtdblock.[2]. =20
> >=20
> > I am sorry but ftl, gluebi, mtdblock, jffs2 and ubi in the same report
> > seem a little bit fuzzy. Are you sure about this sentence? =20
>=20
> ZhaoLong Wang found an interesting bug while stacking various MTD compone=
nts.
> I suggested to just deny mtdblock on top of gluebi to avoid the whole
> problem class instead of adding more duct tape.

Ack. I'll queue this once the log fixed and your official Ack-by
received for mtd/next after -rc1 is out.

Thanks,
Miqu=C3=A8l

>=20
> Thanks,
> //richard
>=20
> The issue reminds me of one of my favorite Monty Python sketches, https:/=
/www.youtube.com/watch?v=3DLFrdqQZ8FFc
> ;-)

:-)