From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Date: Thu, 18 Oct 2018 15:25:28 +0200 From: Stefan Agner Subject: Re: WARNING: possible recursive locking detected in mnt_want_write/mnt_want_write_file_path In-Reply-To: References: Message-ID: To: Amir Goldstein , Miklos Szeredi Cc: Sasha Levin , overlayfs , Al Viro , linux-fsdevel , Greg KH List-ID: On 18.10.2018 13:50, Amir Goldstein wrote: > On Wed, Oct 17, 2018 at 9:37 PM Miklos Szeredi wrote: >> >> On Wed, Oct 17, 2018 at 5:24 PM, Amir Goldstein wrote: > >> > Miklos, >> > >> > You must have confused "the algorithm" by including a "fix" commit in >> > rc1 pull request >> > without mentioning that it "Fixes" a commit in the same pull request. >> >> I didn't know there were heuristics other than "Cc: stable@.." >> > > Yes. Given a Fixes: label, I bet Cc: stable is not needed to auto select > for stable?? Given a Fixes: probably would prevented it to be selected since the commit it references is not part of 4.18... Anyway, as expected, reverting the commit on stable 4.18.4 fixes the warning for me. Thanks for tracking down! -- Stefan > > Sasha, > > Please note that lack of Fixes: or Cc: stable from some subsystems > is a rather strong indication for NOT for stable. > > I suppose that in case of stable branch regression due to incorrect > patch selection there is a strong negative feedback to the algorithm? > > Thanks, > Amir.