[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] UUID support for UFS
From: |
Vladimir 'phcoder' Serbinenko |
Subject: |
Re: [PATCH] UUID support for UFS |
Date: |
Tue, 21 Jul 2009 22:01:40 +0200 |
On Tue, Jul 21, 2009 at 7:14 PM, Pavel Roskin<address@hidden> wrote:
> On Tue, 2009-07-21 at 15:03 +0200, Vladimir 'phcoder' Serbinenko wrote:
>> + grub_sprintf (*uuid, "%08lx%08lx",
>> + (unsigned long) grub_le_to_cpu32 (data->sblock.uuidhi),
>> + (unsigned long) grub_le_to_cpu32 (data->sblock.uuidlow));
>
> unsigned long is 64-bit on x86_64. unsigned int would do just fine
> here.
Ok
> I would add a dash between the numbers to make it more readable unless
> there is a precedent where the dash is not used.
If you look into /dev/ufsid/ then you'll see that it has no dash. This
is also a syntax used in
set FreeBSD.vfs.root.mountfrom=ufs:ufsid/<id>
So making it with dash would need additional conversion to pass it to FreeBSD
>
>
> --
> Regards,
> Pavel Roskin
>
>
> _______________________________________________
> Grub-devel mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/grub-devel
>
--
Regards
Vladimir 'phcoder' Serbinenko
Personal git repository: http://repo.or.cz/w/grub2/phcoder.git
ufs.diff
Description: Text document
- [PATCH] UUID support for UFS, Vladimir 'phcoder' Serbinenko, 2009/07/21
- Re: [PATCH] UUID support for UFS, Pavel Roskin, 2009/07/21
- Re: [PATCH] UUID support for UFS,
Vladimir 'phcoder' Serbinenko <=
- Re: [PATCH] UUID support for UFS, Pavel Roskin, 2009/07/21
- Re: [PATCH] UUID support for UFS, Javier Martín, 2009/07/21
- Re: [PATCH] UUID support for UFS, Pavel Roskin, 2009/07/21
- Re: [PATCH] UUID support for UFS, Vladimir 'phcoder' Serbinenko, 2009/07/21
- Re: [PATCH] UUID support for UFS, Pavel Roskin, 2009/07/21
- Re: [PATCH] UUID support for UFS, Javier Martín, 2009/07/21
- Re: [PATCH] UUID support for UFS, Pavel Roskin, 2009/07/21
- Re: [PATCH] UUID support for UFS, Javier Martín, 2009/07/21
- Re: [PATCH] UUID support for UFS, Pavel Roskin, 2009/07/21
- Re: [PATCH] UUID support for UFS, Robert Millan, 2009/07/22