Re: [PATCH 0/9] Add vsprintf extension %pU to print UUID/GUIDs and use i

xfs-masters

Re: [PATCH 0/9] Add vsprintf extension %pU to print UUID/GUIDs and use i

To:	Joe Perches <joe@xxxxxxxxxxx>
Subject:	Re: [PATCH 0/9] Add vsprintf extension %pU to print UUID/GUIDs and use it
From:	Huang Ying <ying.huang@xxxxxxxxx>
Date:	Tue, 29 Sep 2009 13:57:32 +0800
Cc:	"linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, Adrian Hunter <adrian.hunter@xxxxxxxxx>, Alex Elder <aelder@xxxxxxx>, Artem Bityutskiy <dedekind@xxxxxxxxxxxxx>, Christoph Hellwig <hch@xxxxxx>, Harvey Harrison <harvey.harrison@xxxxxxxxx>, Ingo Molnar <mingo@xxxxxxx>, Jeff Garzik <jgarzik@xxxxxxxxxx>, Laurent Pinchart <laurent.pinchart@xxxxxxxxx>, Matt Mackall <mpm@xxxxxxxxxxx>, Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx>, Neil Brown <neilb@xxxxxxx>, Steven Whitehouse <swhiteho@xxxxxxxxxx>, "xfs-masters@xxxxxxxxxxx" <xfs-masters@xxxxxxxxxxx>, Andi Kleen <andi@xxxxxxxxxxxxxx>, Greg KH <greg@xxxxxxxxx>
In-reply-to:	<cover.1254193019.git.joe@xxxxxxxxxxx>
References:	<cover.1254193019.git.joe@xxxxxxxxxxx>

Hi, Joe,

Thanks for the patch. I think that is a good idea.

For your patch. I think you need a changelog for each patch.

It seems that the binary representation of UUID can be little-endian
(used by most kernel components) or big-endian (defined by RFC4122, used
in network?). Maybe we should consider about that.

In fact, I find there are many different UUID/GUID definitions in
kernel, such as that in efi, many file systems, some drivers, etc. It
seems that every kernel components need UUID/GUID has its own
definition, so I think we should unify all the UUID/GUID definitions in
kernel too. The file attached is a draft unified UUID/GUID definition,
with byte-order issue in mind.

Any comment?

Best Regards,
Huang Ying

0010-uuid.patch
Description: Text Data

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH 4/9] drivers/firmware/dmi_scan.c: Use %pUX to print UUIDs, (continued) [PATCH 4/9] drivers/firmware/dmi_scan.c: Use %pUX to print UUIDs, Joe Perches [PATCH 5/9] drivers/md/md.c: Use %pU to print UUIDs, Joe Perches [PATCH 6/9] drivers/media/video/uvc: Use %pUr to print UUIDs, Joe Perches Re: [PATCH 6/9] drivers/media/video/uvc: Use %pUr to print UUIDs, Laurent Pinchart Re: [PATCH 6/9] drivers/media/video/uvc: Use %pUr to print UUIDs, Joe Perches [PATCH 7/9] fs/gfs2/sys.c: Use %pUX to print UUIDs, Joe Perches Re: [PATCH 7/9] fs/gfs2/sys.c: Use %pUX to print UUIDs, Steven Whitehouse [PATCH 8/9] fs/ubifs: Use %pUX to print UUIDs, Joe Perches Re: [PATCH 8/9] fs/ubifs: Use %pUX to print UUIDs, Artem Bityutskiy [PATCH 9/9] fs/xfs/xfs_log_recover.c: Use %pU to print UUIDs, Joe Perches Re: [PATCH 0/9] Add vsprintf extension %pU to print UUID/GUIDs and use it, Huang Ying <= Re: [PATCH 0/9] Add vsprintf extension %pU to print UUID/GUIDs and use it, Laurent Pinchart Re: [PATCH 0/9] Add vsprintf extension %pU to print UUID/GUIDs and use it, Joe Perches

Previous by Date:	[PATCH 9/9] fs/xfs/xfs_log_recover.c: Use %pU to print UUIDs, Joe Perches
Next by Date:	Re: [PATCH 7/9] fs/gfs2/sys.c: Use %pUX to print UUIDs, Steven Whitehouse
Previous by Thread:	[PATCH 9/9] fs/xfs/xfs_log_recover.c: Use %pU to print UUIDs, Joe Perches
Next by Thread:	Re: [PATCH 0/9] Add vsprintf extension %pU to print UUID/GUIDs and use it, Laurent Pinchart
Indexes:	[Date] [Thread] [Top] [All Lists]