linux-kernel - RE: [PATCH 2/2] arch: arm64: configs: Enable UBI and UBIFS

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <PH0PR11MB567334A09B2D4A1D3DDC09F2CB20A@PH0PR11MB5673.namprd11.prod.outlook.com>
Date:   Sat, 24 Jun 2023 03:42:21 +0000
From:   "Lee, Kah Jing" <kah.jing.lee@...el.com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        Dinh Nguyen <dinguyen@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Catalin Marinas <catalin.marinas@....com>,
        Will Deacon <will@...nel.org>
CC:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH 2/2] arch: arm64: configs: Enable UBI and UBIFS

> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> Sent: Friday, 23 June, 2023 6:18 PM
> To: Lee, Kah Jing <kah.jing.lee@...el.com>; Dinh Nguyen
> <dinguyen@...nel.org>; Rob Herring <robh+dt@...nel.org>; Krzysztof
> Kozlowski <krzysztof.kozlowski+dt@...aro.org>; Conor Dooley
> <conor+dt@...nel.org>; Catalin Marinas <catalin.marinas@....com>; Will
> Deacon <will@...nel.org>
> Cc: linux-kernel@...r.kernel.org
> Subject: Re: [PATCH 2/2] arch: arm64: configs: Enable UBI and UBIFS
> 
> On 23/06/2023 12:03, Lee, Kah Jing wrote:
> >>>>>
> >>>>> Signed-off-by: Alif Zakuan Yuslaimi
> >>>>> <alif.zakuan.yuslaimi@...el.com>
> >>>>> Signed-off-by: Kah Jing Lee <kah.jing.lee@...el.com>
> >>>>> ---
> >>>>>  arch/arm64/configs/defconfig | 2 ++
> >>>>>  1 file changed, 2 insertions(+)
> >>>>>
> >>>>> diff --git a/arch/arm64/configs/defconfig
> >>>>> b/arch/arm64/configs/defconfig index a24609e14d50..23a6ebcb1a9e
> >>>> 100644
> >>>>> --- a/arch/arm64/configs/defconfig
> >>>>> +++ b/arch/arm64/configs/defconfig
> >>>>> @@ -470,6 +470,8 @@ CONFIG_IPMI_DEVICE_INTERFACE=m
> >>>> CONFIG_IPMI_SI=m
> >>>>> CONFIG_HW_RANDOM=y  CONFIG_HW_RANDOM_VIRTIO=y
> >>>>> +CONFIG_MTD_UBI=y
> >>>>> +CONFIG_UBIFS_FS=y
> >>>>
> >>>> Not modules?
> >>> Yes, we need to boot with rootfs with UBIFS.
> >>
> >> So you miss init ramdisk.
> > Currently we are using the bootargs to mount the rootfs from QSPI NOR
> flash:
> > [    0.000000] Kernel command line: earlycon panic=-1 ubi.mtd=1
> root=ubi0:rootfs rootfstype=ubifs rw rootwait
> > Is it possible to mount the ubifs rootfs with the ubifs=m config during boot?
> 
> I think yes. rootfs devices are for example modules, so filesystem can be as
> well.
Was going through mtd ubifs page - http://www.linux-mtd.infradead.org/faq/ubifs.html
Quoted: 'In order to mount UBIFS as the root file system, you have to compile 
UBIFS into the kernel (instead of compiling it as a kernel module) and specify 
proper kernel boot arguments and make the kernel mount UBIFS on boot.'

It seems like we need UBIFS config to be compiled as kernel built-in.
The kernel module would work if we mounted UBIFS as filesystem device 
after rootfs mounted, but not as rootfs.
Let me know if that understanding is correct.
Thanks.
> 
> 
> Best regards,
> Krzysztof

Regards, 
Lee, Kah Jing