From: Simon Horman <horms@verge.net.au>
To: linux-sh@vger.kernel.org
Subject: Re: sh7724 regression: commit 8222dbe21e79338de92d5e1956cd1e3994cc9f93
Date: Wed, 02 Mar 2016 00:20:55 +0000 [thread overview]
Message-ID: <20160302002055.GF17986@verge.net.au> (raw)
In-Reply-To: <CAMuHMdWR0RZhGrX4qW+fHwOYxAXffNinUdhrL=SZsMn5BUKoeA@mail.gmail.com>
[CC Super H maintainers: Sato-san and Rich Felker]
On Mon, Feb 29, 2016 at 12:03:38PM +0100, David Hildenbrand wrote:
> > On 02/29/2016 09:24 AM, David Hildenbrand wrote:
> > >> On 02/27/2016 03:28 PM, John Paul Adrian Glaubitz wrote:
> > >>> Hi Hans!
> > >>>
> > >>> On 02/27/2016 03:20 PM, Geert Uytterhoeven wrote:
> > >>>>> I noticed that 4.1 was ok and v4.2 wasn't, so I did a git bisect and ended up with
> > >>>>> commit 8222dbe21e79338de92d5e1956cd1e3994cc9f93 (sched/preempt, mm/fault: Decouple
> > >>>>> preemption from the page fault logic) as the culprit.
> > >>>
> > >>> If you like, please report this the kernel bug tracker and set
> > >>> "Hardware" to "SuperH". This way we are able to keep track of
> > >>> the issue.
> > >>>
> > >>> I think Yoshinori Sato and Rich Felker will have a look at this.
> > >>>
> > >>> Adrian
> > >>>
> > >>
> > >> Done: https://bugzilla.kernel.org/show_bug.cgi?id\x113321
> > >>
> > >> Regards,
> > >>
> > >> Hans
> > >>
> > >
> > > Looks like the code then requires to not be allowed to sleep/preempt in these
> > > sections (just like kmap_atomic, or kmap_coherent on mips).
> > >
> > > So the fix should be simple as (untested):
> >
> > That fixes it for me:
> >
> > Tested-by: Hans Verkuil <hans.verkuil@cisco.com>
> >
> > BTW, your patch is garbled and I had to manually make the change (not a big
> > deal for a simple change like this).
> >
>
> Thanks for testing. I just copy-pasted the patch, so this was somehow
> expected :)
>
> @maintainers, how to proceed with this? Shall I send that patch again directly
> to the linux-sh list?
>
> David
next prev parent reply other threads:[~2016-03-02 0:20 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-27 14:20 sh7724 regression: commit 8222dbe21e79338de92d5e1956cd1e3994cc9f93 Geert Uytterhoeven
2016-02-27 14:28 ` John Paul Adrian Glaubitz
2016-02-27 14:33 ` Hans Verkuil
2016-02-29 8:24 ` David Hildenbrand
2016-02-29 9:01 ` Hans Verkuil
2016-02-29 11:03 ` David Hildenbrand
2016-03-02 0:20 ` Simon Horman [this message]
2016-03-11 11:06 ` David Hildenbrand
2016-03-13 19:08 ` Rich Felker
2016-03-17 0:35 ` Rich Felker
2016-03-17 17:38 ` Rich Felker
2016-03-29 6:33 ` David Hildenbrand
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=20160302002055.GF17986@verge.net.au \
--to=horms@verge.net.au \
--cc=linux-sh@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.