Hi Jens,
thanks for patching this all up!
On 6/07/23 10:38, Jens Axboe wrote:
On 7/5/23 4:34?PM, Jens Axboe wrote:
On 7/5/23 4:25?PM, Michael Schmitz wrote:
Hi Jens,
On 6/07/23 10:13, Jens Axboe wrote:
should be enough for it to go into stable from 5.2 and onwards.
OK - I wasn't certain whether you wanted the Fixes or stable tag dropped.
(Greg didn't seem to object to the Fixes: as such, just to the
incorrect version prereq)
I think it's really confusing... A patch should only have a Fixes tag if
it's fixing a specific bug in that patch. Either it is, in which case
you would not need Cc stable at all since it's only in 6.5-rc, or it
It is fixing a bug in b6f3f28f60. I should have checked whether the
patch series had already gone to release, not just -rc, instead of
just adding the stable tag out of caution.
But this is the confusion - if it's fixing a bug in b6f3f28f60, then why
is it marked as needing to get backported much further back, predating
that commit?
I see - it doesn't need to be backported that far back _alone_. It
only needs to be applied after b6f3f28f60 once that one has been
backported.
OK I see - I think there's some serious misunderstandings here then :-)
That's probably understating it. I need to reread the patch submission
notes.
It sounds like it fixes a bug in b6f3f28f60 alone, and it has no
business going into stable. The commit should _just_ be marked with it
fixing that. If someone were to backport that previous series, then
their tooling or diligence should notice this dependency and this
current commit should be picked as well.
There should be no Cc: stable on this patch at all, I'll fix it up.
Here's what I have:
https://git.kernel.dk/cgit/linux/commit/?h=block-6.5&id=7eb1e47696aa231b1a567846bbe3a1e1befe1854
which has the following manual edits:
1) Change the title/subject line of the patch. "bugfix for Amiga
partition overflow check patch" means very little. The fact that this
patch is a bug fix for a previous commit is explicit with the Fixes
line.
2) Break lines at 72-74 chars, yours were very short.
Somehow a limit of 60 had stuck in my mind.
3) Drop message-id
4) Drop cc stable tag
5) Drop the revision history. This should be behind three '---' lines
and then it's dropped automatically.
Argh - looks like I've made that mistake more often than not in the past
few years. I'll remember that now.
Let's hope this is it for Amiga partition handling!
I should certainly hope so!
Cheers,
Michael
