lkml.org 
[lkml]   [2017]   [Oct]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] cgroup: reorder flexible array members of struct cgroup_root
Date
When compiling arch/x86/boot/compressed/eboot.c with HOSTCC=clang, the
following warning is observed:

./include/linux/cgroup-defs.h:391:16: warning: field 'cgrp' with
variable sized type 'struct cgroup' not at the end of a struct or class
is a GNU extension [-Wgnu-variable-sized-type-not-at-end]
struct cgroup cgrp;
^
Flexible array members are a C99 feature, but must be the last member of
a struct. Structs with flexible members composed in other structs must
also be the final members, unless using GNU C extensions.

struct cgroup_root's member cgrp is a struct cgroup, struct cgroup's
member ancestor_ids is a flexible member.

Signed-off-by: Nick Desaulniers <nick.desaulniers@gmail.com>
---
Alternatively, we could:
* Not use flexible array members. The flexible array members and allocation
strategy, added in commit b11cfb5807e30 mentions the hot path, so this is
likely not an option?
* Disable this warning for HOSTCC==clang. I'd rather not, since there's
nothing really requiring the use of this particular GNU C extension here,
or specific location of the member cgrp within struct cgroup_root AFAICT.

include/linux/cgroup-defs.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/linux/cgroup-defs.h b/include/linux/cgroup-defs.h
index ade4a78a54c2..2ef256932bf3 100644
--- a/include/linux/cgroup-defs.h
+++ b/include/linux/cgroup-defs.h
@@ -387,9 +387,6 @@ struct cgroup_root {
/* Unique id for this hierarchy. */
int hierarchy_id;

- /* The root cgroup. Root is destroyed on its release. */
- struct cgroup cgrp;
-
/* for cgrp->ancestor_ids[0] */
int cgrp_ancestor_id_storage;

@@ -410,6 +407,9 @@ struct cgroup_root {

/* The name for this hierarchy - may be empty */
char name[MAX_CGROUP_ROOT_NAMELEN];
+
+ /* The root cgroup. Root is destroyed on its release. */
+ struct cgroup cgrp;
};

/*
--
2.11.0
\
 
 \ /
  Last update: 2017-10-17 08:35    [W:0.088 / U:1.228 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site