From: Adam Williamson <awilliam@redhat.com>
To: Roman Gushchin <guro@fb.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH] writeback, cgroup: remove wb from offline list before releasing refcnt
Date: Fri, 23 Jul 2021 12:15:08 -0700 [thread overview]
Message-ID: <b5d25fba13479cb3aa03f1a054873c122edf7310.camel@redhat.com> (raw)
In-Reply-To: <YPsU9/8ZijTjHTWh@carbon.dhcp.thefacebook.com>
On Fri, 2021-07-23 at 12:13 -0700, Roman Gushchin wrote:
> On Fri, Jul 23, 2021 at 11:33:43AM -0700, Adam Williamson wrote:
> > Hi folks! I'm not subscribed to the list and am trying to send this
> > using git-send-email, apologies for any weirdness related to that.
> >
> > Just wanted to confirm the issue that this patch attempts to address,
> > and ask if it can be reviewed/moved along. I look after Fedora's
> > openQA automated test instance, and in tests of recent Fedora Rawhide
> > composes, we're seeing at least one failure per compose that's caused
> > by this crash (it usually prevents the system shutting down or
> > rebooting correctly in a test which requires that to happen).
> >
> > I can't actually confirm the fix as I can't really easily change our
> > tests to run on a non-official kernel build, and our kernel maintainer
> > (reasonably) says he won't backport the patch until it's at least got
> > some review. But I'm definitely seeing the problem!
> >
> > Thanks folks.
> > --
> > Adam Williamson
> > Fedora QA
> > IRC: adamw | Twitter: adamw_ha
> > https://www.happyassassin.net
> >
>
> Hello, Adam!
>
> The patch was picked up by Andrew Morton and will be soon merged into the Linus's tree.
> Currently you can find it in the "next" tree (can't provide a hash because it's not stable
> there).
>
> Please, note that there it another fix called "writeback, cgroup: do not reparent dax inodes",
> which you likely want to backport too.
>
> Both fixes got some testing and reviews.
Hi Roman! Aha, great, thanks for the info. I didn't see any follow-up
on LKML so I figured it was in limbo. I'll let our kernel maintainer
know. Thanks again!
--
Adam Williamson
Fedora QA
IRC: adamw | Twitter: adamw_ha
https://www.happyassassin.net
prev parent reply other threads:[~2021-07-23 19:15 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-16 20:10 [PATCH] writeback, cgroup: remove wb from offline list before releasing refcnt Roman Gushchin
2021-07-23 18:33 ` Adam Williamson
2021-07-23 19:13 ` Roman Gushchin
2021-07-23 19:15 ` Adam Williamson [this message]
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=b5d25fba13479cb3aa03f1a054873c122edf7310.camel@redhat.com \
--to=awilliam@redhat.com \
--cc=guro@fb.com \
--cc=linux-kernel@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;
as well as URLs for NNTP newsgroup(s).