linux-kernel - Re: [ANNOUNCE] [PATCH] [MTD] Flex-OneNAND MTD Driver available.

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]

Date:	Fri, 10 Oct 2008 07:57:51 +0900
From:	"Kyungmin Park" <kmpark@...radead.org>
To:	apgmoorthy <moorthy.apg@...sung.com>,
	"David Woodhouse" <dwmw2@...radead.org>
Cc:	linux-mtd@...ts.infradead.org, lkml <linux-kernel@...r.kernel.org>,
	dedekind@...radead.org
Subject: Re: [ANNOUNCE] [PATCH] [MTD] Flex-OneNAND MTD Driver available.

Hi,

I want to merge it if there are no objection.

David, please commit it at MTD tree.

Thank you,
Kyungmin Park.

On Mon, Sep 29, 2008 at 6:28 PM, apgmoorthy <moorthy.apg@...sung.com> wrote:
>
> Hi David,
>
>   Please do comment on this patch.
>
>   "Not to expose whole OOB" suggestion by Kyungmin should be fine , but
> looking for your suggestion.
>
> With Regards
> Gangheyamoorthy.A.P
>
> -----Original Message-----
> From: Artem Bityutskiy [mailto:dedekind@...radead.org]
> Sent: Friday, September 26, 2008 1:50 PM
> To: Amit Kumar Sharma
> Cc: Kyungmin Park; apgmoorthy; linux-mtd@...ts.infradead.org; Kyungmin Park;
> lkml; David Woodhouse
> Subject: Re: [ANNOUNCE] [PATCH] [MTD] Flex-OneNAND MTD Driver available.
>
>
> On Fri, 2008-09-26 at 13:31 +0530, Amit Kumar Sharma wrote:
>> Hi Artem
>>
>> we have following erro message  if we maintain 64 bytes
>> eccpos.
>>
>> /flash_eraseall -j /dev/mtd4
>> ./flash_eraseall: /dev/mtd4: unable to get NAND oobinfo
>>
>> ./nandwrite -j /dev/mtd2 /jffs2.img
>> MEMSETOOBSEL: Inappropriate ioctl for device
>>
>> but i think as OneNAND has hw ecc controller so Jffs2 will
>> not face any problem
>> still i think mtd should consider for 128 bytes eccpos for
>> 4K page size.
>
> Well, anyway, in Linux ABI is a holy cow, you cannot change it.
> That is a tough rule. Old user-space binaries have to always
> works.
>
> Thus you may do one of:
> 1. Invent a new ioctl for 4KiB page NANDs
> 2. Add sysfs support
> 3. Just do not expose whole OOB as Kyungmin suggested
>
> But I think assume dwmw2 would need to comment on this and
> tell which approach would agree on.
>
> --
> Best regards,
> Artem Bityutskiy (Битюцкий Артём)
>
>