Re: [PATCH] FILESYSTEMS: Make "advanced partitions" immediately unselectable.

From: Grant Coady
Date: Mon Jul 28 2008 - 17:43:46 EST


On Mon, 28 Jul 2008 14:42:47 -0400 (EDT), "Robert P. J. Day" <rpjday@xxxxxxxxxxxxxx> wrote:

>On Mon, 28 Jul 2008, Randy Dunlap wrote:
>
>> > yes, it's nitpicking but, as it's laid out right now, it's
>> > unnecessarily confusing and inconsistent. but i don't know if
>> > there's a trivial fix for it.
>>
>> Yes, I'd call MSDOS partitions basic, not Advanced, and then put all
>> others under an Advanced menu that can be toggled on/off easily with
>> menuconfig.
>>
>> Not quite trivial, but not messy either.
>>
>> Patch below. Comments?
>
>... patch snipped ...
>
> i suspect that's as good as it's going to get, but it's unfortunate
>that it adds yet another level of submenu under "Partition types."
>personally, i was pondering a top level submenu entry of "Advanced
>partition types" in which the (non-advanced) MSDOS partition support
>was yanked out of there *entirely*.
>
> it strikes me that, at least on x86, MSDOS partition support is so
>fundamental that it should take work for someone to turn it *off*.
>perhaps simply have it on by default, and force someone to go under
>"Configure standard kernel features (for small systems)"
>(CONFIG_EMBEDDED) to make it go away?
>
> seriously, how often is someone going to build a kernel for x86 and
>not need MSDOS partition support? just curious.

I've been compiling msdos support as modules for ages, and I have one
x86 machine here that hasn't referenced an msdos filesystem for over
four years ;)

Grant.
--
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/