Re: [RESEND][PATCH v2] block: partition: Add partition specific uevent callbacks for partition info

From: Kees Cook
Date: Mon Jan 25 2016 - 17:50:26 EST


On Fri, Dec 18, 2015 at 8:07 PM, John Stultz <john.stultz@xxxxxxxxxx> wrote:
> On Thu, Dec 10, 2015 at 12:00 PM, John Stultz <john.stultz@xxxxxxxxxx> wrote:
>> From: San Mehat <san@xxxxxxxxxx>
>>
>> This patch has been carried in the Android tree for quite some
>> time and is one of the few patches required to get a mainline
>> kernel up and running with an exsiting Android userspace. So I
>> wanted to submit it for review and consideration if it should
>> be merged.
>>
>> For partitions, add new uevent parameters 'PARTN' which
>> specifies the partitions index in the table, and 'PARTNAME',
>> which specifies PARTNAME specifices the partition name of a
>> partition device.
>>
>> Android's userspace uses this for creating device node links from the
>> partition name and number: ie:
>> /dev/block/platform/soc/by-name/system
>> or
>> /dev/block/platform/soc/by-num/p1
>>
>> One can see its usage here:
>> https://android.googlesource.com/platform/system/core/+/master/init/devices.cpp#355
>> and
>> https://android.googlesource.com/platform/system/core/+/master/init/devices.cpp#494
>>
>> Cc: Jens Axboe <axboe@xxxxxxxxx>
>> Cc: Rom Lemarchand <romlem@xxxxxxxxxx>
>> Cc: Android Kernel Team <kernel-team@xxxxxxxxxxx>
>> Cc: Jeff Moyer <jmoyer@xxxxxxxxxx>
>> Cc: harald@xxxxxxxxxx
>> Cc: kay@xxxxxxxxxx
>> Signed-off-by: Dima Zavin <dima@xxxxxxxxxxx>
>> [Dropped NPARTS and reworded commit message for context]
>> Signed-off-by: John Stultz <john.stultz@xxxxxxxxxx>
>
>
> Ping? Any thoughts on this? Am I missing someone on the CC list I
> should be including?
>
> thanks
> -john
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/

Hi Jens,

Are you able to take this into your tree? Should akpm carry it instead?

Thanks!

-Kees

--
Kees Cook
Chrome OS & Brillo Security