Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1406439

Re: [PATCH v1 1/1] sysctl: introduce uuid_le and uuid_be

From Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Newsgroups linux.kernel
Subject Re: [PATCH v1 1/1] sysctl: introduce uuid_le and uuid_be
Date 2016-05-24 22:30 +0200
Message-ID <rCr7H-84g-5@gated-at.bofh.it> (permalink)
References <rCojv-6cC-7@gated-at.bofh.it> <rCqY2-7Zt-23@gated-at.bofh.it>
Organization Intel Finland Oy

Show all headers | View raw


On Tue, 2016-05-24 at 13:15 -0700, Andrew Morton wrote:
> On Tue, 24 May 2016 20:27:27 +0300 Andy Shevchenko <andriy.shevchenko@
> linux.intel.com> wrote:
> 
> > By default the sysctl interface returns random UUID in big endian
> > format.
> > Sometimes it's not suitable, e.g. using generated UUID for EFI
> > variable name.
> > Provide uuid_le and uuid_be to comprehence that interface.
> > 
> > ...
> > 
> >  drivers/char/random.c       | 44
> > +++++++++++++++++++++++++++++++++++++++++
> >  include/uapi/linux/sysctl.h |  4 +++-
> >  kernel/sysctl_binary.c      | 48
> > ++++++++++++++++++++++++++++++++++++++++++++
> 
> Why does the kernel need to do this?  If userspace wants a random uuid
> then it can grab a random number and cook up the UUID itself.

Good question. I dunno why we have uuid in the first place there. At
some point I noticed people who recommend to use that file to get a
random UUID(s), but apparently they don't aware that endianess matters. 

P.S. I thought to send this as RFC, but decided to drop the notation in
last minute. So, it's okay to drop it.

-- 
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v1 1/1] sysctl: introduce uuid_le and uuid_be Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2016-05-24 19:30 +0200
  Re: [PATCH v1 1/1] sysctl: introduce uuid_le and uuid_be Andrew Morton <akpm@linux-foundation.org> - 2016-05-24 22:20 +0200
    Re: [PATCH v1 1/1] sysctl: introduce uuid_le and uuid_be Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2016-05-24 22:30 +0200
      Re: [PATCH v1 1/1] sysctl: introduce uuid_le and uuid_be Theodore Ts'o <tytso@mit.edu> - 2016-05-26 02:50 +0200

csiph-web