Re: [PATCH 3/4 v4] brd: Fix all partitions BUGs

[Boaz Harrosh]

On 01/17/2015 12:49 AM, Jens Axboe wrote:
> On 01/16/2015 03:32 PM, Tony Luck wrote:
>> On Wed, Jan 7, 2015 at 8:07 AM, Boaz Harrosh <boaz@xxxxxxxxxxxxx> wrote:
>>> From: Boaz Harrosh <boaz@xxxxxxxxxxxxx>
>>>
>>> This patch fixes up brd's partitions scheme, now enjoying all worlds.
>>
>> linux-next-20150116 includes this as commit
>> 937af5ecd0591e84ee54180fa97dcbe9bbe5fed6
>>
>> On ia64 I'm seeing:
>> ram0: unknown partition table
>> ram1: unknown partition table
>> ...
>> ram15: unknown partition table
>>
>> At first I blamed c8fa31730fc7 "brd: Request from fdisk 4k alignment" because it
>> has all sorts of comments about PAGE_SIZE and 4k ... and my page size isn't
>> 4k.  But, reverting that one didn't help.
>>
>> reverting 937af5ecd0 does make the message go away.
> 
> Boaz, why wasn't GENHD_FL_SUPPRESS_PARTITION_INFO retained?
> 

OK. This message is completely harmless. My vm's 3 FSfull devices
print this message day in and day out. Its when you have an FS on
a partition-less device.

Dmitry had a module param "show_partition" which controls the addition
of this flag. The reason to remove it is because for some reason
with it, mount by UUID would not work, same reason lsblk would not see
the partition on this device even with -a.

For some reason GENHD_FL_SUPPRESS_PARTITION_INFO does not only shut up
that harmless message above, it interferes with any partition operations.
(As well as any udev operation on the device)

Do you want that I add back Dmitry's "show_partition" module param? Or should
I add a new flag that would behave exactly as a regular device but only
suppress the above harmless and annoying message like:
	GENHD_FL_PARTITION_LESS_OK (Or something like that)

Your call, Tell me which way you want to go. For me the message above is a
daily routine with many other devices not only brd. 
(I would even just remove the message above, what is it for at all? What error
condition does it uncover? So OK I have plugged a new device and it is
uninitialized yet, how the "unknown partition table" helps me in anyway?
I'm not complaining I would really like to understand.)

Thanks
Boaz

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