From: Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Anthony Eden <aeden-EfZU8u+QLuPpgkiH4x7ZXw@public.gmane.org>
Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Subject: Re: [bug report] null ptr deref in nouveau_platform_probe (tegra186-p2771-0000)
Date: Thu, 21 Dec 2017 19:11:41 +0100 [thread overview]
Message-ID: <20171221181141.GA31262@ulmo> (raw)
In-Reply-To: <CAHMsQjXAXj-0gctfkpi+f8cuPrxF67BWNnG0ZsnTHL3u307vOA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
[-- Attachment #1.1: Type: text/plain, Size: 1243 bytes --]
On Thu, Dec 21, 2017 at 12:32:39PM -0500, Anthony Eden wrote:
> Hi Thierry,
>
> Thanks for the patch. I applied on top of linux-next-2017-12-14.
> Different output this time.
>
> [ 11.862495] WARNING: CPU: 1 PID: 254 at
> drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgf100.c:391
This looks like something that could potentially be fixed by this:
https://github.com/skeggsb/nouveau/commit/305d38cb1d20bf909e2daf6989bba4b27c3a3688
That said, it's probably worth looking at Ben's branch here:
https://github.com/skeggsb/nouveau/commits/master
The top three patches fix various issues that might be relevant.
Note that the above is Ben's nouveau repository (as opposed ot a Linux
kernel tree) and the patches won't apply as-is to the kernel tree. I've
had some success adding the repository as a remote to my kernel tree,
fetching it and cherry-picking patches. git is very good at finding the
right files to apply the patch to, even if they are in different
directories. One exception where it doesn't work is when files are added
(presumable because git doesn't have a hash yet to use for guessing the
right file).
Worst case you could try applying the changes manually, they're fairly
small.
Thierry
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 154 bytes --]
_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau
next prev parent reply other threads:[~2017-12-21 18:11 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-14 21:09 [bug report] null ptr deref in nouveau_platform_probe (tegra186-p2771-0000) Anthony Eden
[not found] ` <CAHMsQjWKhgm5BnngHtH6QeWumNrbSS=LaTRD=XCSy5+iovg7mA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-12-20 19:04 ` [Nouveau] " Thierry Reding
2017-12-21 17:32 ` Anthony Eden
[not found] ` <CAHMsQjXAXj-0gctfkpi+f8cuPrxF67BWNnG0ZsnTHL3u307vOA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-12-21 18:11 ` Thierry Reding [this message]
2017-12-21 19:37 ` Anthony Eden
[not found] ` <CAHMsQjW5HKpZWMgx9=77z4GC0u1pfgncv353G1LbMkNxMV=-6w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-12-22 8:13 ` Thierry Reding
-- strict thread matches above, loose matches on Subject: below --
2017-12-14 21:10 Anthony Eden
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=20171221181141.GA31262@ulmo \
--to=thierry.reding-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=aeden-EfZU8u+QLuPpgkiH4x7ZXw@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.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).