From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932166AbdJUPc6 (ORCPT ); Sat, 21 Oct 2017 11:32:58 -0400 Received: from mail-qt0-f194.google.com ([209.85.216.194]:55366 "EHLO mail-qt0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932118AbdJUPc4 (ORCPT ); Sat, 21 Oct 2017 11:32:56 -0400 X-Google-Smtp-Source: ABhQp+S/PGFlb6PFkO7LD8nBMko8M4+w8MbtFrgjUxVKjyeuHbEqPOygcxPFtqi1vvuHQEX0xJLlHQ== Date: Sat, 21 Oct 2017 08:32:53 -0700 From: Tejun Heo To: Nick Desaulniers Cc: Li Zefan , Johannes Weiner , cgroups@vger.kernel.org, Linux Kernel Mailing List , Matthias Kaehlcke , Michael Davidson , Greg Hackmann , android-llvm@google.com Subject: Re: [PATCH] cgroup: reorder flexible array members of struct cgroup_root Message-ID: <20171021153253.GG1302522@devbig577.frc2.facebook.com> References: <20171017063322.11455-1-nick.desaulniers@gmail.com> <20171018133010.GD1302522@devbig577.frc2.facebook.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, Nick. On Fri, Oct 20, 2017 at 12:15:55AM -0700, Nick Desaulniers wrote: > > This is silly tho. We know the the root group embedded there won't > > have any ancestor_ids. > > Sure, but struct cgroup_root is still declared as having a struct > cgroup not declared as the final member. Why is that a problem tho? We know that it doesn't have any flexible array member so there's no storage allocated to it. > > Also, in general, nothing prevents us from > > doing something like the following. > > > > struct outer_struct { > > blah blah; > > struct inner_struct_with_flexible_array_member inner; > > unsigned long storage_for_flexible_array[NR_ENTRIES]; > > blah blah; > > }; > > At that point, then why have the struct with flexible array member in > the first place? Because there are different ways to use the struct? > >> or specific location of the member cgrp within struct cgroup_root AFAICT. > > I think we should just silence the bogus warning. > > Is the order of the members actually important? Otherwise it seems > that we're taking advantage of a GNU C extension for no real reason, > which is what I'm trying to avoid. Please reconsider. Here, not necessarily but I don't want to move it for a bogus reason. Why would we disallow embedding structs with flexible members in the middle when it can be done and is useful? If we want to discuss whether we want to avoid such usages in the kernel (but why?), sure, let's have that discussion but we can't decide that on "clang warns on it by default". Thanks. -- tejun