From: Willy Tarreau <w@1wt.eu>
To: "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>
Cc: "Manthey, Norbert" <nmanthey@amazon.de>,
"sashal@kernel.org" <sashal@kernel.org>,
"stable@vger.kernel.org" <stable@vger.kernel.org>
Subject: Re: Improving Linux Commit Backporting
Date: Mon, 7 Apr 2025 15:26:20 +0200 [thread overview]
Message-ID: <20250407132620.GA17471@1wt.eu> (raw)
In-Reply-To: <2025040728-shabby-laborer-4891@gregkh>
On Mon, Apr 07, 2025 at 09:31:08AM +0200, gregkh@linuxfoundation.org wrote:
> > We understand. We might make the tool available to help simplify the human effort of backporting. To
> > make this more successful, is there a way to identify the errors and learnings you mention from the
> > past? Avoiding them automatically early in the process helps keeping the errors away.
>
> Don't ignore fuzz, manually check, and verify, everything.
Also, diffing the file between the latest kernel and the one you're
backporting to helps discover changed assumptions. For example, a
group of functions might be called with similar assumptions in the
old kernel, with some tests replicated everywhere (say pointer foo
must not be NULL). In newer kernels this test is moved up in the
caller chain and is no longer performed in the lower functions. When
you want to backport a fix from this kernel to the old one, you may
need to reimplement yourself the nullity check that the old kernel
requires. And that's valid for locking and many other things in
general. There's no way to automatically discover these ones, aside
comparing the older and newer states of the file to see how it
evolved over time, and developing your own habits of remembering
that certain areas are different in your kernel, by doing lots of
backports there, as well as following LKML to try to spot some
changes that may affect your areas of interest. After a few years
on old kernel you can start to develop some reflexes but that's not
rocket science.
> good luck!
Seconded!
> greg k-h
Willy
next prev parent reply other threads:[~2025-04-07 13:26 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-03 13:15 Improving Linux Commit Backporting Manthey, Norbert
2025-04-03 13:45 ` gregkh
2025-04-03 13:57 ` gregkh
2025-04-03 14:51 ` gregkh
[not found] ` <a830cc37fd56d7e7d145342472ede2c924d86837.camel@amazon.de>
2025-04-05 7:21 ` gregkh
2025-04-07 7:26 ` Manthey, Norbert
2025-04-07 7:31 ` gregkh
2025-04-07 13:26 ` Willy Tarreau [this message]
2025-04-07 14:25 ` Sasha Levin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250407132620.GA17471@1wt.eu \
--to=w@1wt.eu \
--cc=gregkh@linuxfoundation.org \
--cc=nmanthey@amazon.de \
--cc=sashal@kernel.org \
--cc=stable@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox