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=7eb1e47696aa231b...
which has the following manual edits:
- 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.
- Break lines at 72-74 chars, yours were very short.
Somehow a limit of 60 had stuck in my mind.
Drop message-id
Drop cc stable tag
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