RE: [PATCH 2/2] ubi: attach: do not return -EINVAL if the mtd->numeraseregions is 1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Brian, Artem,

Are you suggesting that this change be documented instead. Note that we are using NOR flash devices for our testing as you asked.

Can I request you to please clarify on acceptance of this patch ?

Thanks
Sanjeev

>-----Original Message-----
>From: Artem Bityutskiy [mailto:dedekind1@xxxxxxxxx]
>Sent: Thursday, August 04, 2016 12:01 PM
>To: Brian Norris; Richard Weinberger
>Cc: Rajeev Kumar; dwmw2@xxxxxxxxxxxxx; linux-mtd@xxxxxxxxxxxxxxxxxxx;
>stable@xxxxxxxxxxxxxxx
>Subject: Re: [PATCH 2/2] ubi: attach: do not return -EINVAL if the mtd-
>>numeraseregions is 1
>
>On Fri, 2016-07-29 at 11:24 -0700, Brian Norris wrote:
>> (Artem, any opinions, since you had the most opinion last time this
>> came
>> up?)
>
>Hi Brian,
>
>well, I do not have strong opinion. That UBI translates to "I do not know how to
>deal with multiple regions", nothing else.
>
>I think this was my understanding of numeraseregions
>
>1. numaraseregions=0: no regions, just uniform flash.
>
>2. numeraseregions=1: basically same as above, but friendlier to the region-
>aware software. Indeed, if I care about regions, I do not want to handle the
>special case of numerasereions=0, I want a common case with
>numerasereionts>0.
>
>Or to put it this way, from the drivers' writer POW:
>
>numaraseregions=0 - do not know anything about regions, do not care.
>Also pre-regions dirivers, old stuff
>
>numeraseregions=1 - I want my driver to be ideal. Non-aware SW won't look to
>the regions stuff, the aware SW will work nicely.
>
>So my take is: ban numeraseregions=1 if you feel like, but I recommend to just
>document the 0 (don't care/legacy) and 1 (same as 0, but care) and allow for
>both.
>
>Artem.
��.n��������+%������w��{.n�����������ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]