Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:6 as permitted sender) client-ip=2620:137:e000::3:6;
Date:   Thu, 12 Oct 2023 21:34:53 +0200
From:   Thomas =?utf-8?Q?Wei=C3=9Fschuh?= <linux@weissschuh.net>
To:     "Paul E. McKenney" <paulmck@kernel.org>
Cc:     Willy Tarreau <w@1wt.eu>, Zhangjin Wu <falcon@tinylab.org>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] selftests/nolibc: add tests for multi-object linkage
Message-ID: <aa77a065-fcc9-4d3a-8531-fd994587c48f@t-8ch.de>
References: <20231012-nolibc-linkage-test-v1-1-315e682768b4@weissschuh.net>
 <ZSea+etQwlxbi+Ok@1wt.eu>
 <bfc17e76-fcbc-4ce6-97a8-c1ed72ed2a67@t-8ch.de>
 <33e9afcd-a1cd-4f67-829b-85c86500a93e@paulmck-laptop>
 <b278a643-3761-4699-bafc-df1b7245b8c2@t-8ch.de>
 <ca67eb2c-3918-4a1f-b3e6-2023fda5d6a3@paulmck-laptop>
 <6b66305f-8172-463e-a50d-324c0c33a6ea@t-8ch.de>
 <a5f1a910-dbac-44d8-b9f6-5725bea948b2@paulmck-laptop>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <a5f1a910-dbac-44d8-b9f6-5725bea948b2@paulmck-laptop>
Precedence: bulk

On 2023-10-12 12:06:33-0700, Paul E. McKenney wrote:
> On Thu, Oct 12, 2023 at 08:39:14PM +0200, Thomas Weißschuh wrote:
> > On 2023-10-12 11:25:02-0700, Paul E. McKenney wrote:
> > > [..]

> > > I have a signed tag urgent/nolibc.2023.10.12a in the -rcu tree, so
> > > please check the lead-in text for sanity.  (Everything after the digital
> > > signature is automatically generated.)
> > 
> > Looks good. But it's only a listing of the commit subjects, correct?
> 
> Pretty close, just a few added words on the last one.
> 
> So the question is whether there is some larger issue that Linus should
> be made aware of.  If these are just simple fixes for simple bugs,
> we should be good, but yes, I do need to ask.  ;-)

These are simple fixes for simple bugs.

Do you always have to ask specifically or can I just mention it in the
pull request in the future?

> [..]

> > > Ah, and have these been posted to a public mailing list?  If not, then I
> > > need to send them out.
> > 
> > All patches went through the lists as part of the normal developent
> > flow. They were not posted after rebasing.
> 
> I have been sending the group, so I might as well continue the tradition.

Sounds good. If you want me to do something different, please let me
know.

> There are a couple of substantive checkpatch complaints:
> 
> 4b4a30ea14d1 ("tools/nolibc: i386: Fix a stack misalign bug on _start")
> 	The Fixes SHA-1 should be limited to 12 hex digits.
> 	(I am ignoring this, but be prepared for Linus to gripe.
> 	If you decide to fix it, I would be happy to repull.)

Done.

> f2c7923763da ("selftests/nolibc: add tests for multi-object linkage")
> 	nolibc-test-linkage.c and nolibc-test-linkage.h need
> 	"//" comment for the SPDX comment header.  This one needs
> 	to be fixed, but this is not in the urgent stack, so there
> 	is some time.

nolibc limits itself intentionally to C89 language level which disallows
C++ style headers.

This should be covered by Documentation/process/license-rules.rst:

  If a specific tool cannot handle the standard comment style, then the
  appropriate comment mechanism which the tool accepts shall be used.


With that said:

Please pull the changes since the v6.6-rc1 tag from
https://git.kernel.org/pub/scm/linux/kernel/git/nolibc/linux-nolibc.git/

The branch 'fixes' up to and including
921992229b1f06df6b649860e4a5f3def1489866 for the v6.6 cycle.

The branch 'next' up to and including
b8c60e8fc6f755c2cdf7164931afdbfa670c6646 for linux-next.

No full test has been performed as only a commit message was changed.

Testing for full nolibc stack:
make run:             162 test(s): 162 passed,   0 skipped,   0 failed => status: success
make run-nolibc-test: 162 test(s): 160 passed,   2 skipped,   0 failed => status: warning


Thomas