Received-SPF: pass (google.com: domain of linux-kernel+bounces-73804-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Message-ID: <bd0abacb-e0d5-40f5-b786-6021f6d8bd2e@linuxfoundation.org>
Date: Tue, 20 Feb 2024 16:07:28 -0700
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: Linux 6.8-rc5
Content-Language: en-US
To: Linus Torvalds <torvalds@linux-foundation.org>,
 Guenter Roeck <linux@roeck-us.net>, David Gow <davidgow@google.com>,
 Brendan Higgins <brendan.higgins@linux.dev>, Rae Moar <rmoar@google.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
 Matthew Auld <matthew.auld@intel.com>,
 Arunpravin Paneer Selvam <arunpravin.paneerselvam@amd.com>,
 =?UTF-8?Q?Christian_K=C3=B6nig?= <christian.koenig@amd.com>,
 kselftest list <linux-kselftest@vger.kernel.org>,
 KUnit Development <kunit-dev@googlegroups.com>,
 Shuah Khan <skhan@linuxfoundation.org>
References: <CAHk-=wi8vZD7EXZfob-yhfDERyfzWxzMOzG9FsOuaKU-v6+PHA@mail.gmail.com>
 <538327ff-8d34-41d5-a9ae-1a334744f5ae@roeck-us.net>
 <CAHk-=wj6xj_cGmsQK7g=hSfRZZNo-njC+u_1v3dE8fPZtjCBOg@mail.gmail.com>
 <CAHk-=wgJMOquDO5f8ShH1f4rzZwzApNVCw643m5-Yj+BfsFstA@mail.gmail.com>
From: Shuah Khan <skhan@linuxfoundation.org>
In-Reply-To: <CAHk-=wgJMOquDO5f8ShH1f4rzZwzApNVCw643m5-Yj+BfsFstA@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

On 2/20/24 13:16, Linus Torvalds wrote:
> On Tue, 20 Feb 2024 at 11:57, Linus Torvalds
> <torvalds@linux-foundation.org> wrote:
>>
>> It turns out that that commit is buggy for another reason, but it's
>> hidden by the fact that apparently KUNIT_ASSERT_FALSE_MSG() doesn't
>> check the format string.
> 
> The fix for that is this:
> 
>    --- a/include/kunit/test.h
>    +++ b/include/kunit/test.h
>    @@ -579,7 +579,7 @@ void __printf(2, 3) kunit_log_append(struct
> string_stream *log, const char *fmt,
> 
>     void __noreturn __kunit_abort(struct kunit *test);
> 
>    -void __kunit_do_failed_assertion(struct kunit *test,
>    +void __printf(6,7) __kunit_do_failed_assertion(struct kunit *test,
>                                   const struct kunit_loc *loc,
>                                   enum kunit_assert_type type,
>                                   const struct kunit_assert *assert,
> 
> but that causes a *lot* of noise (not just in drm_buddy_test.c), so
> I'm not going to apply that fix as-is. Clearly there's a lot of
> incorrect format parameters that have never been checked.
> 
> Instead adding Shuah and the KUnit people to the participants, and
> hoping that they will fix this up and we can get the format fixes for
> KUnit in the 6.9 timeframe.
> 

We will take care of this for 6.9

> Side note: when I apply the above patch, the suggestions gcc spews out
> look invalid. Gcc seems to suggest turning a a format string of '%d"
> to "%ld" for a size_t variable. That's wrong. It should be "%zu".
> 
> A 'size_t' can in fact be 'unsigned int' on some platforms (not just
> in theory), so %ld is really incorrect not just from a sign
> perspective.
> 
> Anyway, I guess I will commit the immediate drm_buddy_test.c fix to
> get rid of the build issue, but the KUnit message format string issue
> will have to be a "let's get this fixed up _later_" issue.
> 
>                Linus

Thank you.

-- Shuah