Messages in this thread | | | Subject | Re: [PATCH v2] x86: xen: remove the use of VLAIS | From | Boris Ostrovsky <> | Date | Mon, 8 Jan 2018 10:54:24 -0500 |
| |
On 01/06/2018 04:39 PM, Nick Desaulniers wrote: > Variable Length Arrays In Structs (VLAIS) is not supported by Clang, and > frowned upon by others. > > https://lkml.org/lkml/2013/9/23/500 > > Here, the VLAIS was used because the size of the bitmap returned from > xen_mc_entry() depended on possibly (based on kernel configuration) > runtime sized data. Rather than declaring args as a VLAIS then calling > sizeof on *args, we calculate the appropriate sizeof args manually. > Further, we can get rid of the #ifdef's and rely on num_possible_cpus() > (thanks to a helpful checkpatch warning from an earlier version of this > patch). > > Suggested-by: Juergen Gross <jgross@suse.com> > Signed-off-by: Nick Desaulniers <nick.desaulniers@gmail.com>
Applied to for-linus-4.15.
-boris
| |