* [PATCH] cryptodev-module: fix with kernels 4.13 and newer
@ 2018-01-17 20:59 California Sullivan
2018-01-17 21:05 ` ✗ patchtest: failure for " Patchwork
2018-01-17 22:46 ` [PATCH] " Cal Sullivan
0 siblings, 2 replies; 3+ messages in thread
From: California Sullivan @ 2018-01-17 20:59 UTC (permalink / raw)
To: openembedded-core
Backport a patch from upstream to address designated-init errors such
as the following:
cryptodev-module/1.9-r0/cryptodev-linux-1.9/ioctl.c:1127:3: error:
positional initialization of field in 'struct' declared with
'designated_init' attribute [-Werror=designated-init]
| {0, },
| ^
Signed-off-by: California Sullivan <california.l.sullivan@intel.com>
---
.../cryptodev/cryptodev-module_1.9.bb | 1 +
.../0001-ioctl.c-Fix-build-with-linux-4.13.patch | 49 ++++++++++++++++++++++
2 files changed, 50 insertions(+)
create mode 100644 meta/recipes-kernel/cryptodev/files/0001-ioctl.c-Fix-build-with-linux-4.13.patch
diff --git a/meta/recipes-kernel/cryptodev/cryptodev-module_1.9.bb b/meta/recipes-kernel/cryptodev/cryptodev-module_1.9.bb
index 552eb6abaae..ed6d0ecae97 100644
--- a/meta/recipes-kernel/cryptodev/cryptodev-module_1.9.bb
+++ b/meta/recipes-kernel/cryptodev/cryptodev-module_1.9.bb
@@ -9,6 +9,7 @@ DEPENDS += "cryptodev-linux"
SRC_URI += " \
file://0001-Disable-installing-header-file-provided-by-another-p.patch \
+file://0001-ioctl.c-Fix-build-with-linux-4.13.patch \
"
EXTRA_OEMAKE='KERNEL_DIR="${STAGING_KERNEL_DIR}" PREFIX="${D}"'
diff --git a/meta/recipes-kernel/cryptodev/files/0001-ioctl.c-Fix-build-with-linux-4.13.patch b/meta/recipes-kernel/cryptodev/files/0001-ioctl.c-Fix-build-with-linux-4.13.patch
new file mode 100644
index 00000000000..eccaff4f3f1
--- /dev/null
+++ b/meta/recipes-kernel/cryptodev/files/0001-ioctl.c-Fix-build-with-linux-4.13.patch
@@ -0,0 +1,49 @@
+From f0d69774afb27ffc62bf353465fba145e70cb85a Mon Sep 17 00:00:00 2001
+From: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>
+Date: Mon, 4 Sep 2017 11:05:08 +0200
+Subject: [PATCH] ioctl.c: Fix build with linux 4.13
+
+Upstream-Status: backport
+
+git/ioctl.c:1127:3: error: positional initialization of field in 'struct' declared with 'designated_init' attribute [-Werror=designated-init]
+ {0, },
+ ^
+note: (near initialization for 'verbosity_ctl_dir[1]')
+git/ioctl.c:1136:3: error: positional initialization of field in 'struct' declared with 'designated_init' attribute [-Werror=designated-init]
+ {0, },
+ ^
+
+Linux kernel has added -Werror=designated-init around 4.11 (c834f0e8a8b)
+triggering build errors with gcc 5 and 6 (but not with gcc 4)
+
+Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>
+Signed-off-by: Cristian Stoica <cristian.stoica@nxp.com>
+---
+ ioctl.c | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/ioctl.c b/ioctl.c
+index 0385203..8d4a162 100644
+--- a/ioctl.c
++++ b/ioctl.c
+@@ -1124,7 +1124,7 @@ static struct ctl_table verbosity_ctl_dir[] = {
+ .mode = 0644,
+ .proc_handler = proc_dointvec,
+ },
+- {0, },
++ {},
+ };
+
+ static struct ctl_table verbosity_ctl_root[] = {
+@@ -1133,7 +1133,7 @@ static struct ctl_table verbosity_ctl_root[] = {
+ .mode = 0555,
+ .child = verbosity_ctl_dir,
+ },
+- {0, },
++ {},
+ };
+ static struct ctl_table_header *verbosity_sysctl_header;
+ static int __init init_cryptodev(void)
+--
+2.14.3
+
--
2.14.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* ✗ patchtest: failure for cryptodev-module: fix with kernels 4.13 and newer
2018-01-17 20:59 [PATCH] cryptodev-module: fix with kernels 4.13 and newer California Sullivan
@ 2018-01-17 21:05 ` Patchwork
2018-01-17 22:46 ` [PATCH] " Cal Sullivan
1 sibling, 0 replies; 3+ messages in thread
From: Patchwork @ 2018-01-17 21:05 UTC (permalink / raw)
To: Sullivan, California L; +Cc: openembedded-core
== Series Details ==
Series: cryptodev-module: fix with kernels 4.13 and newer
Revision: 1
URL : https://patchwork.openembedded.org/series/10589/
State : failure
== Summary ==
Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:
* Issue Upstream-Status is in incorrect format [test_upstream_status_presence_format]
Suggested fix Fix Upstream-Status format in 0001-ioctl.c-Fix-build-with-linux-4.13.patch
Current Upstream-Status: backport
Standard format Upstream-Status: <Valid status>
Valid status Pending, Accepted, Backport, Denied, Inappropriate [reason], Submitted [where]
If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).
---
Guidelines: https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] cryptodev-module: fix with kernels 4.13 and newer
2018-01-17 20:59 [PATCH] cryptodev-module: fix with kernels 4.13 and newer California Sullivan
2018-01-17 21:05 ` ✗ patchtest: failure for " Patchwork
@ 2018-01-17 22:46 ` Cal Sullivan
1 sibling, 0 replies; 3+ messages in thread
From: Cal Sullivan @ 2018-01-17 22:46 UTC (permalink / raw)
To: openembedded-core
Apologies, it looks like the same patch was submitted by someone else
already.
I will ack that one. Ignore this one.
Thanks,
Cal
On 01/17/2018 12:59 PM, California Sullivan wrote:
> Backport a patch from upstream to address designated-init errors such
> as the following:
>
> cryptodev-module/1.9-r0/cryptodev-linux-1.9/ioctl.c:1127:3: error:
> positional initialization of field in 'struct' declared with
> 'designated_init' attribute [-Werror=designated-init]
> | {0, },
> | ^
>
> Signed-off-by: California Sullivan <california.l.sullivan@intel.com>
> ---
> .../cryptodev/cryptodev-module_1.9.bb | 1 +
> .../0001-ioctl.c-Fix-build-with-linux-4.13.patch | 49 ++++++++++++++++++++++
> 2 files changed, 50 insertions(+)
> create mode 100644 meta/recipes-kernel/cryptodev/files/0001-ioctl.c-Fix-build-with-linux-4.13.patch
>
> diff --git a/meta/recipes-kernel/cryptodev/cryptodev-module_1.9.bb b/meta/recipes-kernel/cryptodev/cryptodev-module_1.9.bb
> index 552eb6abaae..ed6d0ecae97 100644
> --- a/meta/recipes-kernel/cryptodev/cryptodev-module_1.9.bb
> +++ b/meta/recipes-kernel/cryptodev/cryptodev-module_1.9.bb
> @@ -9,6 +9,7 @@ DEPENDS += "cryptodev-linux"
>
> SRC_URI += " \
> file://0001-Disable-installing-header-file-provided-by-another-p.patch \
> +file://0001-ioctl.c-Fix-build-with-linux-4.13.patch \
> "
>
> EXTRA_OEMAKE='KERNEL_DIR="${STAGING_KERNEL_DIR}" PREFIX="${D}"'
> diff --git a/meta/recipes-kernel/cryptodev/files/0001-ioctl.c-Fix-build-with-linux-4.13.patch b/meta/recipes-kernel/cryptodev/files/0001-ioctl.c-Fix-build-with-linux-4.13.patch
> new file mode 100644
> index 00000000000..eccaff4f3f1
> --- /dev/null
> +++ b/meta/recipes-kernel/cryptodev/files/0001-ioctl.c-Fix-build-with-linux-4.13.patch
> @@ -0,0 +1,49 @@
> +From f0d69774afb27ffc62bf353465fba145e70cb85a Mon Sep 17 00:00:00 2001
> +From: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>
> +Date: Mon, 4 Sep 2017 11:05:08 +0200
> +Subject: [PATCH] ioctl.c: Fix build with linux 4.13
> +
> +Upstream-Status: backport
> +
> +git/ioctl.c:1127:3: error: positional initialization of field in 'struct' declared with 'designated_init' attribute [-Werror=designated-init]
> + {0, },
> + ^
> +note: (near initialization for 'verbosity_ctl_dir[1]')
> +git/ioctl.c:1136:3: error: positional initialization of field in 'struct' declared with 'designated_init' attribute [-Werror=designated-init]
> + {0, },
> + ^
> +
> +Linux kernel has added -Werror=designated-init around 4.11 (c834f0e8a8b)
> +triggering build errors with gcc 5 and 6 (but not with gcc 4)
> +
> +Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>
> +Signed-off-by: Cristian Stoica <cristian.stoica@nxp.com>
> +---
> + ioctl.c | 4 ++--
> + 1 file changed, 2 insertions(+), 2 deletions(-)
> +
> +diff --git a/ioctl.c b/ioctl.c
> +index 0385203..8d4a162 100644
> +--- a/ioctl.c
> ++++ b/ioctl.c
> +@@ -1124,7 +1124,7 @@ static struct ctl_table verbosity_ctl_dir[] = {
> + .mode = 0644,
> + .proc_handler = proc_dointvec,
> + },
> +- {0, },
> ++ {},
> + };
> +
> + static struct ctl_table verbosity_ctl_root[] = {
> +@@ -1133,7 +1133,7 @@ static struct ctl_table verbosity_ctl_root[] = {
> + .mode = 0555,
> + .child = verbosity_ctl_dir,
> + },
> +- {0, },
> ++ {},
> + };
> + static struct ctl_table_header *verbosity_sysctl_header;
> + static int __init init_cryptodev(void)
> +--
> +2.14.3
> +
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-01-17 22:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-17 20:59 [PATCH] cryptodev-module: fix with kernels 4.13 and newer California Sullivan
2018-01-17 21:05 ` ✗ patchtest: failure for " Patchwork
2018-01-17 22:46 ` [PATCH] " Cal Sullivan
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.