Re: [PATCH 0/2] mtd: ubi: Fix some initialisation failure cases

From: Zhihao Cheng <chengzhihao1@huawei.com>
To: Ben Hutchings <ben.hutchings@mind.be>, <linux-mtd@lists.infradead.org>
Subject: Re: [PATCH 0/2] mtd: ubi: Fix some initialisation failure cases
Date: Thu, 11 Apr 2024 11:30:41 +0800	[thread overview]
Message-ID: <f6191bef-9adb-9517-762d-ac95ebd3a4d1@huawei.com> (raw)
In-Reply-To: <cover.1712788087.git.ben.hutchings@mind.be>

在 2024/4/11 6:42, Ben Hutchings 写道:
> ubi currently fails to initialise if debugfs is enabled at build time
> but disabled at boot time (CONFIG_DEBUG_FS is enabled and debugfs=off
> is set on the kernel command line).  Errors from debugfs should always
> be ignored but this wasn't done everywhere.
> 
> While fixing this I also noticed that a recent change to ubi removed
> some cleanup from the ubi_init() failure path that I think is still
> needed.
> 
> This series should fix both problems.
> 
> Ben.

Hi, Ben. I have a same patch with your first one, I have merge our 
patchset into a new series [1]
[1] 
https://lore.kernel.org/linux-mtd/20240411031903.3050278-1-chengzhihao1@huawei.com/
> 
> Ben Hutchings (2):
>    mtd: ubi: Restore missing cleanup on ubi_init() failure path
>    mtd: ubi: Ignore all debugfs initialisation failures
> 
>   drivers/mtd/ubi/build.c | 15 ++++++---------
>   drivers/mtd/ubi/debug.c | 33 +++++++--------------------------
>   drivers/mtd/ubi/debug.h |  4 ++--
>   3 files changed, 15 insertions(+), 37 deletions(-)
> 


______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/