Re: [Cluster-devel] [gfs2:extents 14/14] fs/gfs2/extents.c:123:23-27: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) (fwd)

On 19/10/2023 06:27, Julia Lawall wrote:
> 
> 
> ---------- Forwarded message ----------
> Date: Thu, 19 Oct 2023 05:56:44 +0800
> From: kernel test robot <[email protected]>
> To: [email protected]
> Cc: [email protected], Julia Lawall <[email protected]>
> Subject: [gfs2:extents 14/14] fs/gfs2/extents.c:123:23-27: WARNING use
>      flexible-array member instead
>      (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-
>      and-one-element-arrays)
> 
> BCC: [email protected]
> CC: [email protected]
> CC: [email protected]

The lkp-tests contact info is out of date. I've opened a PR to fix it at 
https://github.com/intel/lkp-tests/pull/319 but it should probably use 
the latest MAINTAINERS file always. CCing the new gfs2 list.

Thanks,
Andy

> TO: Andreas Gruenbacher <[email protected]>
> 
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/gfs2/linux-gfs2.git extents
> head:   86474c69cac46872bd311318c02fb8e9e25abd10
> commit: 86474c69cac46872bd311318c02fb8e9e25abd10 [14/14] gfs2: Add some initial extents-based inode code
> :::::: branch date: 9 days ago
> :::::: commit date: 9 days ago
> config: x86_64-randconfig-101-20231018 (https://download.01.org/0day-ci/archive/20231019/[email protected]/config)
> compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
> reproduce: (https://download.01.org/0day-ci/archive/20231019/[email protected]/reproduce)
> 
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <[email protected]>
> | Reported-by: Julia Lawall <[email protected]>
> | Closes: https://lore.kernel.org/r/[email protected]/
> 
> cocci warnings: (new ones prefixed by >>)
>>> fs/gfs2/extents.c:123:23-27: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> 
> vim +123 fs/gfs2/extents.c
> 
> 86474c69cac468 Andreas Gruenbacher 2021-03-25  115
> 86474c69cac468 Andreas Gruenbacher 2021-03-25  116  /*
> 86474c69cac468 Andreas Gruenbacher 2021-03-25  117   * The path components in struct gfs2_extent_path are stored in "reverse"
> 86474c69cac468 Andreas Gruenbacher 2021-03-25  118   * order: the leaf is stored first, and when the tree grows at the root,
> 86474c69cac468 Andreas Gruenbacher 2021-03-25  119   * this happens at the end of the array.
> 86474c69cac468 Andreas Gruenbacher 2021-03-25  120   */
> 86474c69cac468 Andreas Gruenbacher 2021-03-25  121  struct gfs2_extent_path {
> 86474c69cac468 Andreas Gruenbacher 2021-03-25  122  	unsigned int p_height;
> 86474c69cac468 Andreas Gruenbacher 2021-03-25 @123  	struct gfs2_extent_pc p_pc[1];
> 86474c69cac468 Andreas Gruenbacher 2021-03-25  124  };
> 86474c69cac468 Andreas Gruenbacher 2021-03-25  125
>