linux-kernel - Re: ubifs mount failure

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-next>] [day] [month] [year] [list]

Date:   Tue, 3 Dec 2019 09:41:34 +0100
From:   Sascha Hauer <s.hauer@...gutronix.de>
To:     Naga Sureshkumar Relli <nagasure@...inx.com>
Cc:     Richard Weinberger <richard@....at>,
        Miquel Raynal <miquel.raynal@...tlin.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-mtd@...ts.infradead.org" <linux-mtd@...ts.infradead.org>,
        Michal Simek <michals@...inx.com>,
        "siva.durga.paladugu@...ilnx.com" <siva.durga.paladugu@...ilnx.com>
Subject: Re: ubifs mount failure

Hi,

On Tue, Dec 03, 2019 at 04:52:32AM +0000, Naga Sureshkumar Relli wrote:
>    Hi,
> 
>     
> 
>    We have upgraded our Linux kernel to 5.4 from 4.19.
> 
>    And I tried mounting ubifs using this kernel on NAND partition with below
>    command and saw that
> 
>    There is an issue with memory allocation.
> 
>    #mount -t ubifs ubi0:data /mnt/
> 
>    mount: mounting ubi0:data on /mnt/ failed: Cannot allocate memory
> 
>     
> 
>    I saw that there is a commit on fs/ubifs/sb.c, where it is allocating all
>    the required memories at one shot.
> 
>    [1]https://lkml.org/lkml/2018/9/7/724
> 
>    By reverting the above patch, I am able to mount successfully the ubifs.
> 
>    By reverting this patch, we are allocating, writing and freeing in a
>    manner such that, we don’t see memory allocation issues.

Sorry, I can't see how this patch causes failing memory allocations. And
no, this is not expected. Could you sprinkle some printks and track down
where it fails? Is it the obvious place here:

	if (!sup || !mst || !idx || !ino || !cs) {
		err = -ENOMEM;
		goto out;
	}

If yes, which allocation fails and how much memory did we try to allocate?
If no, where does it fail? Also, where are you using UBIFS. Is it NAND flash
or NOR flash?

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |