public inbox for stable@vger.kernel.org
 help / color / mirror / Atom feed
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Mirsad Todorovac <mirsad.todorovac@alu.unizg.hr>
Cc: Luis Chamberlain <mcgrof@kernel.org>,
	stable@vger.kernel.org, Dan Carpenter <error27@gmail.com>
Subject: Re: [RESEND PATCH v5 1/3] test_firmware: prevent race conditions by a correct implementation of locking
Date: Mon, 31 Jul 2023 13:33:42 +0200	[thread overview]
Message-ID: <2023073147-sleeve-regular-46cd@gregkh> (raw)
In-Reply-To: <a09b4fa3-d6dc-b7a9-f815-d6f43211910b@alu.unizg.hr>

On Mon, Jul 31, 2023 at 01:29:19PM +0200, Mirsad Todorovac wrote:
> On 31.7.2023. 1:50, Luis Chamberlain wrote:
> > On Sat, Jul 29, 2023 at 11:17:45AM +0200, Mirsad Todorovac wrote:
> > > ---
> > > v5.1
> > >   resending to v5.4 stable branch verbatim according to Luis Chamberlain instruction
> > 
> > If this is a backport of an upstream patch you must mention the commit
> > ID at the top. After
> > 
> > For instance, here is a random commit from v5.15.y branch for stable:
> > 
> > bpf: Add selftests to cover packet access corner cases
> > commit b560b21f71eb4ef9dfc7c8ec1d0e4d7f9aa54b51 upstream.
> > 
> > <the upstream commit log>
> 
> Hello,
> 
> I have reviewed the module again and I found no new weaknesses, so it is only
> a backport from the same commit in torvalds, master, 6.4, 6.1, 5.15 and
> 5.10 trees/branches.
> 
> This is a bit confusing and I am doing this for the first time. In fact, there
> was probably a glitch in the patchwork because the comment to the
> Cc: stable@vger.kernel.org said "# 5.4" ...
> 
> However, I do not know which commit ID to refer to:
> 
> torvalds 4acfe3dfde685a5a9eaec5555351918e2d7266a1
> master   4acfe3dfde685a5a9eaec5555351918e2d7266a1
> 6.4      4acfe3dfde685a5a9eaec5555351918e2d7266a1
> 6.1      6111f0add6ffc93612d4abe9fec002319102b1c0
> 5.15     bfb0b366e8ec23d9a9851898d81c829166b8c17b
> 5.10     af36f35074b10dda0516cfc63d209accd4ef4d17
> 
> Each of the branches 6.4, 6.1, 5.15 and 5.10 appear to have a different commit
> ID.
> 
> Probably the right commit ID should be:
> 
> test_firmware: prevent race conditions by a correct implementation of locking
> 
> commit 4acfe3dfde685a5a9eaec5555351918e2d7266a1 master
> 
> Will the patchwork figure this out or should I RESEND with a clean slate?
> 
> But first I would appreciate a confirmation that I did it right this time ...

I don't understand at all what you are trying to do here.

Is this a patch for Linus's tree?  If so, great, let's apply it there.

Is this a patch for the stable kernel(s)?  If so, great, what is the git
id in Linus's tree and what stable kernel(s) should it be applied to?

That's all we need to know and right now, I have no idea...

confused,

greg k-h

  reply	other threads:[~2023-07-31 11:33 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-29  9:17 [RESEND PATCH v5 1/3] test_firmware: prevent race conditions by a correct implementation of locking Mirsad Todorovac
2023-07-30 23:50 ` Luis Chamberlain
2023-07-31 11:29   ` Mirsad Todorovac
2023-07-31 11:33     ` Greg Kroah-Hartman [this message]
2023-07-31 12:32       ` Mirsad Todorovac
  -- strict thread matches above, loose matches on Subject: below --
2023-05-09  8:47 Mirsad Goran Todorovac

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=2023073147-sleeve-regular-46cd@gregkh \
    --to=gregkh@linuxfoundation.org \
    --cc=error27@gmail.com \
    --cc=mcgrof@kernel.org \
    --cc=mirsad.todorovac@alu.unizg.hr \
    --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