Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Date:   Thu, 31 Mar 2022 19:34:11 -0400
From:   Steven Rostedt <rostedt@goodmis.org>
To:     Jakob Koschel <jakobkoschel@gmail.com>
Cc:     Ingo Molnar <mingo@redhat.com>, linux-kernel@vger.kernel.org,
        Mike Rapoport <rppt@kernel.org>,
        Brian Johannesmeyer <bjohannesmeyer@gmail.com>,
        Cristiano Giuffrida <c.giuffrida@vu.nl>,
        "Bos, H.J." <h.j.bos@vu.nl>
Subject: Re: [PATCH] ftrace: remove check of list iterator against head past
 the loop body
Message-ID: <20220331193411.2fdbeb88@gandalf.local.home>
In-Reply-To: <212AE797-912E-47D1-9DC4-03B5351AFC57@gmail.com>
References: <20220331223752.902726-1-jakobkoschel@gmail.com>
        <20220331191005.2954969f@gandalf.local.home>
        <212AE797-912E-47D1-9DC4-03B5351AFC57@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Fri, 1 Apr 2022 01:22:58 +0200
Jakob Koschel <jakobkoschel@gmail.com> wrote:

> > Can you move this to the first declaration to keep the nice "upside-down
> > x-mas tree" look.  
> 
> Thanks, I'll fix that up. It seems like this is not applied on the entire kernel
> making treewide changes a bit more difficult. Is it documented somewhere, which
> parts of the kernel enforce this? Just looking two lines down from here it
> seems to be 'broken' already so just from looking at existing code it's often
> hard to judge.

It's one of those things that some maintainers prefer (I'm one of them ;-)
because it makes it easier to read IMHO.

But as you noticed, it's broken even in the same file. That's because I
don't strictly enforce it. If there's a lot of code that looks good to go
in, I don't ask to fix it. But as this was a small trivial patch, I figured
I'd mention it.

Thus, it's something that you do when asked, but don't worry about doing it
across the board, you are not going to upset anybody by forgetting to do it.

-- Steve