Re: [PATCH 2.6.11] aoe [7/12]: support configuration ofAOE_PARTITIONS from Kconfig

[Ed L Cashin]

Christoph Hellwig <hch@xxxxxxxxxxxxx> writes:

> On Tue, Mar 29, 2005 at 11:06:16AM -0500, Ed L Cashin wrote:
>> >
>> > NACK.  this changes devices nodes based on a compile-time option.  
>> 
>> I'm not sure I follow.  This configuration option sets the number of
>> partitions per device in the driver.  It doesn't create device nodes.
>> 
>> If the user has udev, then the device nodes are created correctly (on
>> Fedora Core 3), so that if the driver is configured with 1 partition
>> per device, the minor numbers for the disks are low.  
>> 
>> The folks I've talked to who aren't using udev but want one partition
>> per device already know that they have to re-create their device
>> files.
>
> It changes a kernel ABI, so people that have different config options
> set can't use the same userland.  It's a really big no-go.

I don't know if it matters now that we have udev.  When udev manages
the device nodes it all just works, so there's practically not much of
an issue.  The UUIDs in Software RAID and LVM make this even less of
an issue.  Besides, if they aren't using udev and are using a custom
kernel *and* using this configure option, then they're changing the
ABI on purpose for a practical benefit.

If you're saying that it's bad in principal, then that's another
story.  If that's what you mean, then it's a Linux policy issue, and
to follow convention I'd think that we'd need another major number.
That would be like the partitionable md devices, etc.

To me, the latter seems like the uglier way to go, and it would be
more permanent, while the configure option seems ugly but convenient
for this transitional stage, and could go away when high minor numbers
are well supported on the systems currently in use.

-- 
  Ed L Cashin <ecashin@xxxxxxxxxx>

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