All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] linux-libc-headers: backport uapi header list
@ 2016-02-02 18:05 Alexandru But
  2016-02-02 18:40 ` Saul Wold
  0 siblings, 1 reply; 5+ messages in thread
From: Alexandru But @ 2016-02-02 18:05 UTC (permalink / raw)
  To: openembedded-core

nvme uapi header was renamed from nvme.h to nvme_ioctl.h in linux 4.4, but the
header list was not updated. This causes efivar (and maybe other) package build
to fail. This should fix the build while the commit is backported in the
kernel.

Signed-off-by: Alexandru But <alexandru.but@ni.com>
---
 ...i-update-install-list-after-nvme.h-rename.patch | 34 ++++++++++++++++++++++
 .../linux-libc-headers/linux-libc-headers_4.4.bb   |  4 +++
 2 files changed, 38 insertions(+)
 create mode 100644 meta/recipes-kernel/linux-libc-headers/linux-libc-headers/0001-uapi-update-install-list-after-nvme.h-rename.patch

diff --git a/meta/recipes-kernel/linux-libc-headers/linux-libc-headers/0001-uapi-update-install-list-after-nvme.h-rename.patch b/meta/recipes-kernel/linux-libc-headers/linux-libc-headers/0001-uapi-update-install-list-after-nvme.h-rename.patch
new file mode 100644
index 0000000..4d9f5a1
--- /dev/null
+++ b/meta/recipes-kernel/linux-libc-headers/linux-libc-headers/0001-uapi-update-install-list-after-nvme.h-rename.patch
@@ -0,0 +1,34 @@
+From a9cf8284b45110a4d98aea180a89c857e53bf850 Mon Sep 17 00:00:00 2001
+From: Mike Frysinger <vapier@gentoo.org>
+Date: Sun, 10 Jan 2016 20:14:11 -0500
+Subject: [PATCH] uapi: update install list after nvme.h rename
+
+Commit 9d99a8dda154 ("nvme: move hardware structures out of the uapi
+version of nvme.h") renamed nvme.h to nvme_ioctl.h, but the uapi list
+still refers to nvme.h.  People trying to install the headers hit a
+failure as the header no longer exists.
+
+Cc: stable@vger.kernel.org
+Signed-off-by: Mike Frysinger <vapier@gentoo.org>
+Reviewed-by: Christoph Hellwig <hch@lst.de>
+Signed-off-by: Jens Axboe <axboe@fb.com>
+---
+ include/uapi/linux/Kbuild | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/include/uapi/linux/Kbuild b/include/uapi/linux/Kbuild
+index 628e6e6..88e1292 100644
+--- a/include/uapi/linux/Kbuild
++++ b/include/uapi/linux/Kbuild
+@@ -306,7 +306,7 @@ header-y += nfs_mount.h
+ header-y += nl80211.h
+ header-y += n_r3964.h
+ header-y += nubus.h
+-header-y += nvme.h
++header-y += nvme_ioctl.h
+ header-y += nvram.h
+ header-y += omap3isp.h
+ header-y += omapfb.h
+-- 
+2.6.1
+
diff --git a/meta/recipes-kernel/linux-libc-headers/linux-libc-headers_4.4.bb b/meta/recipes-kernel/linux-libc-headers/linux-libc-headers_4.4.bb
index 3763dc0..c8c7a10 100644
--- a/meta/recipes-kernel/linux-libc-headers/linux-libc-headers_4.4.bb
+++ b/meta/recipes-kernel/linux-libc-headers/linux-libc-headers_4.4.bb
@@ -5,5 +5,9 @@ SRC_URI_append_libc-musl = "\
     file://0002-libc-compat.h-prevent-redefinition-of-struct-ethhdr.patch \
     file://0003-remove-inclusion-of-sysinfo.h-in-kernel.h.patch \
    "
+SRC_URI_append ="\
+    file://0001-uapi-update-install-list-after-nvme.h-rename.patch \
+   "
+
 SRC_URI[md5sum] = "9a78fa2eb6c68ca5a40ed5af08142599"
 SRC_URI[sha256sum] = "401d7c8fef594999a460d10c72c5a94e9c2e1022f16795ec51746b0d165418b2"
-- 
2.6.1



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] linux-libc-headers: backport uapi header list
  2016-02-02 18:05 [PATCH] linux-libc-headers: backport uapi header list Alexandru But
@ 2016-02-02 18:40 ` Saul Wold
  2016-02-02 18:47   ` Bruce Ashfield
  0 siblings, 1 reply; 5+ messages in thread
From: Saul Wold @ 2016-02-02 18:40 UTC (permalink / raw)
  To: Alexandru But, openembedded-core

On Tue, 2016-02-02 at 20:05 +0200, Alexandru But wrote:
> nvme uapi header was renamed from nvme.h to nvme_ioctl.h in linux
> 4.4, but the
> header list was not updated. This causes efivar (and maybe other)
> package build
> to fail. This should fix the build while the commit is backported in
> the
> kernel.
> 
Would it not make more sense to fix efivar (and maybe others) instead
of changing the kernel headers, which were clearly changed for a reason
to start with.

This patch seems to be pending: 
https://gitweb.gentoo.org/repo/gentoo.git/tree/sys-libs/efivar/files/0.
21-nvme_ioctl.h.patch

And a more appropriate patch to the kernel headers would be backporting
this already merged patch:

https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/
?id=a9cf8284b45110a4d98aea180a89c857e53bf850

You can request that Bruce (the linux-yocto maintainer) backport this
patch via the linux-yocto@yoctoproject.org list.


Sau!
 
> Signed-off-by: Alexandru But <alexandru.but@ni.com>
> ---
>  ...i-update-install-list-after-nvme.h-rename.patch | 34
> ++++++++++++++++++++++
>  .../linux-libc-headers/linux-libc-headers_4.4.bb   |  4 +++
>  2 files changed, 38 insertions(+)
>  create mode 100644 meta/recipes-kernel/linux-libc-headers/linux-
> libc-headers/0001-uapi-update-install-list-after-nvme.h-rename.patch
> 
> diff --git a/meta/recipes-kernel/linux-libc-headers/linux-libc-
> headers/0001-uapi-update-install-list-after-nvme.h-rename.patch
> b/meta/recipes-kernel/linux-libc-headers/linux-libc-headers/0001-
> uapi-update-install-list-after-nvme.h-rename.patch
> new file mode 100644
> index 0000000..4d9f5a1
> --- /dev/null
> +++ b/meta/recipes-kernel/linux-libc-headers/linux-libc-headers/0001-
> uapi-update-install-list-after-nvme.h-rename.patch
> @@ -0,0 +1,34 @@
> +From a9cf8284b45110a4d98aea180a89c857e53bf850 Mon Sep 17 00:00:00
> 2001
> +From: Mike Frysinger <vapier@gentoo.org>
> +Date: Sun, 10 Jan 2016 20:14:11 -0500
> +Subject: [PATCH] uapi: update install list after nvme.h rename
> +
> +Commit 9d99a8dda154 ("nvme: move hardware structures out of the uapi
> +version of nvme.h") renamed nvme.h to nvme_ioctl.h, but the uapi
> list
> +still refers to nvme.h.  People trying to install the headers hit a
> +failure as the header no longer exists.
> +
> +Cc: stable@vger.kernel.org
> +Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> +Reviewed-by: Christoph Hellwig <hch@lst.de>
> +Signed-off-by: Jens Axboe <axboe@fb.com>
> +---
> + include/uapi/linux/Kbuild | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/include/uapi/linux/Kbuild b/include/uapi/linux/Kbuild
> +index 628e6e6..88e1292 100644
> +--- a/include/uapi/linux/Kbuild
> ++++ b/include/uapi/linux/Kbuild
> +@@ -306,7 +306,7 @@ header-y += nfs_mount.h
> + header-y += nl80211.h
> + header-y += n_r3964.h
> + header-y += nubus.h
> +-header-y += nvme.h
> ++header-y += nvme_ioctl.h
> + header-y += nvram.h
> + header-y += omap3isp.h
> + header-y += omapfb.h
> +-- 
> +2.6.1
> +
> diff --git a/meta/recipes-kernel/linux-libc-headers/linux-libc-
> headers_4.4.bb b/meta/recipes-kernel/linux-libc-headers/linux-libc-
> headers_4.4.bb
> index 3763dc0..c8c7a10 100644
> --- a/meta/recipes-kernel/linux-libc-headers/linux-libc-
> headers_4.4.bb
> +++ b/meta/recipes-kernel/linux-libc-headers/linux-libc-
> headers_4.4.bb
> @@ -5,5 +5,9 @@ SRC_URI_append_libc-musl = "\
>      file://0002-libc-compat.h-prevent-redefinition-of-struct-
> ethhdr.patch \
>      file://0003-remove-inclusion-of-sysinfo.h-in-kernel.h.patch \
>     "
> +SRC_URI_append ="\
> +    file://0001-uapi-update-install-list-after-nvme.h-rename.patch \
> +   "
> +
>  SRC_URI[md5sum] = "9a78fa2eb6c68ca5a40ed5af08142599"
>  SRC_URI[sha256sum] =
> "401d7c8fef594999a460d10c72c5a94e9c2e1022f16795ec51746b0d165418b2"
> -- 
> 2.6.1
> 


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] linux-libc-headers: backport uapi header list
  2016-02-02 18:40 ` Saul Wold
@ 2016-02-02 18:47   ` Bruce Ashfield
  2016-02-02 19:09     ` Saul Wold
  0 siblings, 1 reply; 5+ messages in thread
From: Bruce Ashfield @ 2016-02-02 18:47 UTC (permalink / raw)
  To: Saul Wold; +Cc: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 4895 bytes --]

On Tue, Feb 2, 2016 at 1:40 PM, Saul Wold <sgw@linux.intel.com> wrote:

> On Tue, 2016-02-02 at 20:05 +0200, Alexandru But wrote:
> > nvme uapi header was renamed from nvme.h to nvme_ioctl.h in linux
> > 4.4, but the
> > header list was not updated. This causes efivar (and maybe other)
> > package build
> > to fail. This should fix the build while the commit is backported in
> > the
> > kernel.
> >
> Would it not make more sense to fix efivar (and maybe others) instead
> of changing the kernel headers, which were clearly changed for a reason
> to start with.
>

Yep. But I recall some argument against that. I'm on a plane at the moment
and can't check my references.


>
> This patch seems to be pending:
> https://gitweb.gentoo.org/repo/gentoo.git/tree/sys-libs/efivar/files/0.
> 21-nvme_ioctl.h.patch
>
> And a more appropriate patch to the kernel headers would be backporting
> this already merged patch:
>
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/
> ?id=a9cf8284b45110a4d98aea180a89c857e53bf850
>
> You can request that Bruce (the linux-yocto maintainer) backport this
> patch via the linux-yocto@yoctoproject.org list.
>
>
Maybe I'm missing something, but wouldn't that just fix projects that build
against
linux-yocto ? If that's good enough, then yes, I can cherry pick the patch
(or
better yet, get it via -stable).

but we'd still need to update the linux-libc-headers, since unless these
apps
are building against the kernel_staging_dir, they are using the libc-headers
variant of the header.

Right ? .. or again, I may be missing what efivar is building against.

Bruce


>
> Sau!
>
> > Signed-off-by: Alexandru But <alexandru.but@ni.com>
> > ---
> >  ...i-update-install-list-after-nvme.h-rename.patch | 34
> > ++++++++++++++++++++++
> >  .../linux-libc-headers/linux-libc-headers_4.4.bb   |  4 +++
> >  2 files changed, 38 insertions(+)
> >  create mode 100644 meta/recipes-kernel/linux-libc-headers/linux-
> > libc-headers/0001-uapi-update-install-list-after-nvme.h-rename.patch
> >
> > diff --git a/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > headers/0001-uapi-update-install-list-after-nvme.h-rename.patch
> > b/meta/recipes-kernel/linux-libc-headers/linux-libc-headers/0001-
> > uapi-update-install-list-after-nvme.h-rename.patch
> > new file mode 100644
> > index 0000000..4d9f5a1
> > --- /dev/null
> > +++ b/meta/recipes-kernel/linux-libc-headers/linux-libc-headers/0001-
> > uapi-update-install-list-after-nvme.h-rename.patch
> > @@ -0,0 +1,34 @@
> > +From a9cf8284b45110a4d98aea180a89c857e53bf850 Mon Sep 17 00:00:00
> > 2001
> > +From: Mike Frysinger <vapier@gentoo.org>
> > +Date: Sun, 10 Jan 2016 20:14:11 -0500
> > +Subject: [PATCH] uapi: update install list after nvme.h rename
> > +
> > +Commit 9d99a8dda154 ("nvme: move hardware structures out of the uapi
> > +version of nvme.h") renamed nvme.h to nvme_ioctl.h, but the uapi
> > list
> > +still refers to nvme.h.  People trying to install the headers hit a
> > +failure as the header no longer exists.
> > +
> > +Cc: stable@vger.kernel.org
> > +Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> > +Reviewed-by: Christoph Hellwig <hch@lst.de>
> > +Signed-off-by: Jens Axboe <axboe@fb.com>
> > +---
> > + include/uapi/linux/Kbuild | 2 +-
> > + 1 file changed, 1 insertion(+), 1 deletion(-)
> > +
> > +diff --git a/include/uapi/linux/Kbuild b/include/uapi/linux/Kbuild
> > +index 628e6e6..88e1292 100644
> > +--- a/include/uapi/linux/Kbuild
> > ++++ b/include/uapi/linux/Kbuild
> > +@@ -306,7 +306,7 @@ header-y += nfs_mount.h
> > + header-y += nl80211.h
> > + header-y += n_r3964.h
> > + header-y += nubus.h
> > +-header-y += nvme.h
> > ++header-y += nvme_ioctl.h
> > + header-y += nvram.h
> > + header-y += omap3isp.h
> > + header-y += omapfb.h
> > +--
> > +2.6.1
> > +
> > diff --git a/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > headers_4.4.bb b/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > headers_4.4.bb
> > index 3763dc0..c8c7a10 100644
> > --- a/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > headers_4.4.bb
> > +++ b/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > headers_4.4.bb
> > @@ -5,5 +5,9 @@ SRC_URI_append_libc-musl = "\
> >      file://0002-libc-compat.h-prevent-redefinition-of-struct-
> > ethhdr.patch \
> >      file://0003-remove-inclusion-of-sysinfo.h-in-kernel.h.patch \
> >     "
> > +SRC_URI_append ="\
> > +    file://0001-uapi-update-install-list-after-nvme.h-rename.patch \
> > +   "
> > +
> >  SRC_URI[md5sum] = "9a78fa2eb6c68ca5a40ed5af08142599"
> >  SRC_URI[sha256sum] =
> > "401d7c8fef594999a460d10c72c5a94e9c2e1022f16795ec51746b0d165418b2"
> > --
> > 2.6.1
> >
>



-- 
"Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end"

[-- Attachment #2: Type: text/html, Size: 7500 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] linux-libc-headers: backport uapi header list
  2016-02-02 18:47   ` Bruce Ashfield
@ 2016-02-02 19:09     ` Saul Wold
  2016-02-03 10:39       ` Alexandru But
  0 siblings, 1 reply; 5+ messages in thread
From: Saul Wold @ 2016-02-02 19:09 UTC (permalink / raw)
  To: Bruce Ashfield; +Cc: Patches and discussions about the oe-core layer

On Tue, 2016-02-02 at 13:47 -0500, Bruce Ashfield wrote:
> 
> 
> On Tue, Feb 2, 2016 at 1:40 PM, Saul Wold <sgw@linux.intel.com>
> wrote:
> > On Tue, 2016-02-02 at 20:05 +0200, Alexandru But wrote:
> > > nvme uapi header was renamed from nvme.h to nvme_ioctl.h in linux
> > > 4.4, but the
> > > header list was not updated. This causes efivar (and maybe other)
> > > package build
> > > to fail. This should fix the build while the commit is backported
> > in
> > > the
> > > kernel.
> > >
> > Would it not make more sense to fix efivar (and maybe others)
> > instead
> > of changing the kernel headers, which were clearly changed for a
> > reason
> > to start with.
> Yep. But I recall some argument against that. I'm on a plane at the
> moment
> and can't check my references.
>  
> >  
> > This patch seems to be pending: 
> > https://gitweb.gentoo.org/repo/gentoo.git/tree/sys-libs/efivar/file
> > s/0.
> > 21-nvme_ioctl.h.patch
> > 
> > And a more appropriate patch to the kernel headers would be
> > backporting
> > this already merged patch:
> > 
> > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/com
> > mit/
> > ?id=a9cf8284b45110a4d98aea180a89c857e53bf850
> > 
> > You can request that Bruce (the linux-yocto maintainer) backport
> > this
> > patch via the linux-yocto@yoctoproject.org list.
> > 
> Maybe I'm missing something, but wouldn't that just fix projects that
> build against
> linux-yocto ? If that's good enough, then yes, I can cherry pick the
> patch (or
> better yet, get it via -stable).
> 
> but we'd still need to update the linux-libc-headers, since unless
> these apps
> are building against the kernel_staging_dir, they are using the libc-
> headers
> variant of the header. 
> 
> Right ? .. or again, I may be missing what efivar is building
> against.
> 
There seems to be a patch on the gentoo version of efivar to address
getting the correct header (the first patch I listed above) for efivar,
that in conjunction with adding the patch to linux-libc-headers to fix
the installation of the nvme-ioctl.h should solve the problem, at least
for efivar.

So the header file install of nvme-ioctl.h patch would be need to be
backported to both the linux-libc-headers and linux-yocto. My bad for
not checking the linux-libc-headers recipe more carefully.


Sau!



> Bruce
>  
> >  
> > Sau!
> >  
> > > Signed-off-by: Alexandru But <alexandru.but@ni.com>
> > > ---
> > >  ...i-update-install-list-after-nvme.h-rename.patch | 34
> > > ++++++++++++++++++++++
> > >  .../linux-libc-headers/linux-libc-headers_4.4.bb   |  4 +++
> > >  2 files changed, 38 insertions(+)
> > >  create mode 100644 meta/recipes-kernel/linux-libc-headers/linux-
> > > libc-headers/0001-uapi-update-install-list-after-nvme.h-
> > rename.patch
> > >
> > > diff --git a/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > > headers/0001-uapi-update-install-list-after-nvme.h-rename.patch
> > > b/meta/recipes-kernel/linux-libc-headers/linux-libc-headers/0001-
> > > uapi-update-install-list-after-nvme.h-rename.patch
> > > new file mode 100644
> > > index 0000000..4d9f5a1
> > > --- /dev/null
> > > +++ b/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > headers/0001-
> > > uapi-update-install-list-after-nvme.h-rename.patch
> > > @@ -0,0 +1,34 @@
> > > +From a9cf8284b45110a4d98aea180a89c857e53bf850 Mon Sep 17
> > 00:00:00
> > > 2001
> > > +From: Mike Frysinger <vapier@gentoo.org>
> > > +Date: Sun, 10 Jan 2016 20:14:11 -0500
> > > +Subject: [PATCH] uapi: update install list after nvme.h rename
> > > +
> > > +Commit 9d99a8dda154 ("nvme: move hardware structures out of the
> > uapi
> > > +version of nvme.h") renamed nvme.h to nvme_ioctl.h, but the uapi
> > > list
> > > +still refers to nvme.h.  People trying to install the headers
> > hit a
> > > +failure as the header no longer exists.
> > > +
> > > +Cc: stable@vger.kernel.org
> > > +Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> > > +Reviewed-by: Christoph Hellwig <hch@lst.de>
> > > +Signed-off-by: Jens Axboe <axboe@fb.com>
> > > +---
> > > + include/uapi/linux/Kbuild | 2 +-
> > > + 1 file changed, 1 insertion(+), 1 deletion(-)
> > > +
> > > +diff --git a/include/uapi/linux/Kbuild
> > b/include/uapi/linux/Kbuild
> > > +index 628e6e6..88e1292 100644
> > > +--- a/include/uapi/linux/Kbuild
> > > ++++ b/include/uapi/linux/Kbuild
> > > +@@ -306,7 +306,7 @@ header-y += nfs_mount.h
> > > + header-y += nl80211.h
> > > + header-y += n_r3964.h
> > > + header-y += nubus.h
> > > +-header-y += nvme.h
> > > ++header-y += nvme_ioctl.h
> > > + header-y += nvram.h
> > > + header-y += omap3isp.h
> > > + header-y += omapfb.h
> > > +--
> > > +2.6.1
> > > +
> > > diff --git a/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > > headers_4.4.bb b/meta/recipes-kernel/linux-libc-headers/linux-
> > libc-
> > > headers_4.4.bb
> > > index 3763dc0..c8c7a10 100644
> > > --- a/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > > headers_4.4.bb
> > > +++ b/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > > headers_4.4.bb
> > > @@ -5,5 +5,9 @@ SRC_URI_append_libc-musl = "\
> > >      file://0002-libc-compat.h-prevent-redefinition-of-struct-
> > > ethhdr.patch \
> > >      file://0003-remove-inclusion-of-sysinfo.h-in-kernel.h.patch
> > \
> > >     "
> > > +SRC_URI_append ="\
> > > +    file://0001-uapi-update-install-list-after-nvme.h-
> > rename.patch \
> > > +   "
> > > +
> > >  SRC_URI[md5sum] = "9a78fa2eb6c68ca5a40ed5af08142599"
> > >  SRC_URI[sha256sum] =
> > >
> > "401d7c8fef594999a460d10c72c5a94e9c2e1022f16795ec51746b0d165418b2"
> > > --
> > > 2.6.1
> > >
> > 
> 
> 


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] linux-libc-headers: backport uapi header list
  2016-02-02 19:09     ` Saul Wold
@ 2016-02-03 10:39       ` Alexandru But
  0 siblings, 0 replies; 5+ messages in thread
From: Alexandru But @ 2016-02-03 10:39 UTC (permalink / raw)
  To: Saul Wold; +Cc: Patches and discussions about the oe-core layer

On Tue, Feb 02, 2016 at 11:09:58AM -0800, Saul Wold wrote:
> On Tue, 2016-02-02 at 13:47 -0500, Bruce Ashfield wrote:
> > 
> > 
> > On Tue, Feb 2, 2016 at 1:40 PM, Saul Wold <sgw@linux.intel.com>
> > wrote:
> > > On Tue, 2016-02-02 at 20:05 +0200, Alexandru But wrote:
> > > > nvme uapi header was renamed from nvme.h to nvme_ioctl.h in linux
> > > > 4.4, but the
> > > > header list was not updated. This causes efivar (and maybe other)
> > > > package build
> > > > to fail. This should fix the build while the commit is backported
> > > in
> > > > the
> > > > kernel.
> > > >
> > > Would it not make more sense to fix efivar (and maybe others)
> > > instead
> > > of changing the kernel headers, which were clearly changed for a
> > > reason
> > > to start with.
> > Yep. But I recall some argument against that. I'm on a plane at the
> > moment
> > and can't check my references.
> > ??
> > > ??
> > > This patch seems to be pending:??
> > > https://gitweb.gentoo.org/repo/gentoo.git/tree/sys-libs/efivar/file
> > > s/0.
> > > 21-nvme_ioctl.h.patch
> > > 
> > > And a more appropriate patch to the kernel headers would be
> > > backporting
> > > this already merged patch:
> > > 
> > > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/com
> > > mit/
> > > ?id=a9cf8284b45110a4d98aea180a89c857e53bf850
> > > 
> > > You can request that Bruce (the linux-yocto maintainer) backport
> > > this
> > > patch via the linux-yocto@yoctoproject.org??list.
> > > 
> > Maybe I'm missing something, but wouldn't that just fix projects that
> > build against
> > linux-yocto ? If that's good enough, then yes, I can cherry pick the
> > patch (or
> > better yet, get it via -stable).
> > 
> > but we'd still need to update the linux-libc-headers, since unless
> > these apps
> > are building against the kernel_staging_dir, they are using the libc-
> > headers
> > variant of the header. 
> > 
> > Right ? .. or again, I may be missing what efivar is building
> > against.
> > 
> There seems to be a patch on the gentoo version of efivar to address
> getting the correct header (the first patch I listed above) for efivar,
> that in conjunction with adding the patch to linux-libc-headers to fix
> the installation of the nvme-ioctl.h should solve the problem, at least
> for efivar.
> 
> So the header file install of nvme-ioctl.h patch would be need to be
> backported to both the linux-libc-headers and linux-yocto. My bad for
> not checking the linux-libc-headers recipe more carefully.
> 
> 
> Sau!
> 

I should of added more context to this. Here is the analisys and the discussion
had after efivar start failing, along with some feedback from Bruce:
http://lists.openembedded.org/pipermail/openembedded-devel/2016-February/105790.html

The patch applyed to the linux-libc-headers is the one that got in the mainline
and Saul referenced above. The change should be backported to the stable 4.4
linux archive that this recipe uses, but until then we can have the build
working like this. Another option wich would spare the work to remove this
patch later on would be to backport this patch to linux 4.4.1 and update the
recipe version. The time window should be short enough to do it, but maybe
some would consider there is a high risk of breaking other things.

I sent the gentoo patch to meta-oe layer just after sending this one.
http://patches.openembedded.org/patch/114375/

Regards,
Alex But

> 
> 
> > Bruce
> > ??
> > > ??
> > > Sau!
> > > ??
> > > > Signed-off-by: Alexandru But <alexandru.but@ni.com>
> > > > ---
> > > > ??...i-update-install-list-after-nvme.h-rename.patch | 34
> > > > ++++++++++++++++++++++
> > > > ??.../linux-libc-headers/linux-libc-headers_4.4.bb??????|????4 +++
> > > > ??2 files changed, 38 insertions(+)
> > > > ??create mode 100644 meta/recipes-kernel/linux-libc-headers/linux-
> > > > libc-headers/0001-uapi-update-install-list-after-nvme.h-
> > > rename.patch
> > > >
> > > > diff --git a/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > > > headers/0001-uapi-update-install-list-after-nvme.h-rename.patch
> > > > b/meta/recipes-kernel/linux-libc-headers/linux-libc-headers/0001-
> > > > uapi-update-install-list-after-nvme.h-rename.patch
> > > > new file mode 100644
> > > > index 0000000..4d9f5a1
> > > > --- /dev/null
> > > > +++ b/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > > headers/0001-
> > > > uapi-update-install-list-after-nvme.h-rename.patch
> > > > @@ -0,0 +1,34 @@
> > > > +From a9cf8284b45110a4d98aea180a89c857e53bf850 Mon Sep 17
> > > 00:00:00
> > > > 2001
> > > > +From: Mike Frysinger <vapier@gentoo.org>
> > > > +Date: Sun, 10 Jan 2016 20:14:11 -0500
> > > > +Subject: [PATCH] uapi: update install list after nvme.h rename
> > > > +
> > > > +Commit 9d99a8dda154 ("nvme: move hardware structures out of the
> > > uapi
> > > > +version of nvme.h") renamed nvme.h to nvme_ioctl.h, but the uapi
> > > > list
> > > > +still refers to nvme.h.????People trying to install the headers
> > > hit a
> > > > +failure as the header no longer exists.
> > > > +
> > > > +Cc: stable@vger.kernel.org
> > > > +Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> > > > +Reviewed-by: Christoph Hellwig <hch@lst.de>
> > > > +Signed-off-by: Jens Axboe <axboe@fb.com>
> > > > +---
> > > > + include/uapi/linux/Kbuild | 2 +-
> > > > + 1 file changed, 1 insertion(+), 1 deletion(-)
> > > > +
> > > > +diff --git a/include/uapi/linux/Kbuild
> > > b/include/uapi/linux/Kbuild
> > > > +index 628e6e6..88e1292 100644
> > > > +--- a/include/uapi/linux/Kbuild
> > > > ++++ b/include/uapi/linux/Kbuild
> > > > +@@ -306,7 +306,7 @@ header-y += nfs_mount.h
> > > > + header-y += nl80211.h
> > > > + header-y += n_r3964.h
> > > > + header-y += nubus.h
> > > > +-header-y += nvme.h
> > > > ++header-y += nvme_ioctl.h
> > > > + header-y += nvram.h
> > > > + header-y += omap3isp.h
> > > > + header-y += omapfb.h
> > > > +--
> > > > +2.6.1
> > > > +
> > > > diff --git a/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > > > headers_4.4.bb b/meta/recipes-kernel/linux-libc-headers/linux-
> > > libc-
> > > > headers_4.4.bb
> > > > index 3763dc0..c8c7a10 100644
> > > > --- a/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > > > headers_4.4.bb
> > > > +++ b/meta/recipes-kernel/linux-libc-headers/linux-libc-
> > > > headers_4.4.bb
> > > > @@ -5,5 +5,9 @@ SRC_URI_append_libc-musl = "\
> > > > ??????????file://0002-libc-compat.h-prevent-redefinition-of-struct-
> > > > ethhdr.patch \
> > > > ??????????file://0003-remove-inclusion-of-sysinfo.h-in-kernel.h.patch
> > > \
> > > > ????????"
> > > > +SRC_URI_append ="\
> > > > +????????file://0001-uapi-update-install-list-after-nvme.h-
> > > rename.patch \
> > > > +??????"
> > > > +
> > > > ??SRC_URI[md5sum] = "9a78fa2eb6c68ca5a40ed5af08142599"
> > > > ??SRC_URI[sha256sum] =
> > > >
> > > "401d7c8fef594999a460d10c72c5a94e9c2e1022f16795ec51746b0d165418b2"
> > > > --
> > > > 2.6.1
> > > >
> > > 
> > 
> > 


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-02-03 10:39 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-02 18:05 [PATCH] linux-libc-headers: backport uapi header list Alexandru But
2016-02-02 18:40 ` Saul Wold
2016-02-02 18:47   ` Bruce Ashfield
2016-02-02 19:09     ` Saul Wold
2016-02-03 10:39       ` Alexandru But

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.