From: Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: Ilia Mirkin <imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org>
Cc: "nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org"
<nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>,
Maarten Lankhorst
<maarten.lankhorst-GeWIH/nMZzLQT0dZR+AlfA@public.gmane.org>
Subject: Re: "enable dri3 support without glamor" causes gnome-shell regression on nv4x
Date: Mon, 3 Aug 2015 19:31:21 +0200 [thread overview]
Message-ID: <55BFA569.4080302@redhat.com> (raw)
In-Reply-To: <CAKb7UviW5e_1nkxXJ_Rud2SGF8=QUbjyr1aKdSG5h=o=_K7AAw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
Hi,
On 03-08-15 17:36, Ilia Mirkin wrote:
> On Mon, Aug 3, 2015 at 9:02 AM, Hans de Goede <hdegoede@redhat.com> wrote:
>> Hi,
>>
>> On 30-07-15 16:09, Ilia Mirkin wrote:
>>>
>>> FWIW this is a fail on nv50+ as well. See for example
>>> https://bugs.freedesktop.org/show_bug.cgi?id=91445
>>>
>>> My suspicion is that this is due to the lack of PUSH_KICK in the *Done
>>> exa handlers -- works fine with DRI2, but DRI3 has no synchronization
>>> and so the commands never get flushed out. Easily verified by sticking
>>> PUSH_KICK's everywhere.
>>
>>
>> I do not believe that that is the problem, in my case it clearly
>> seems to be a pitch / swizzle problem rather then a synchronizarion
>> problem, here is what my desktop with gnome shell looks like when
>> using DRI2:
>>
>> https://fedorapeople.org/~jwrdegoede/nv46-gnome-shell-good.jpg
>>
>> And this is what it looks like when using DRI3:
>>
>> https://fedorapeople.org/~jwrdegoede/nv46-gnome-shell-bad.jpg
>>
>> The DRI2 screenshot is made with Mario's 2 patches on top of
>> current master:
>>
>> http://lists.freedesktop.org/archives/nouveau/2015-July/021740.html
>> http://lists.freedesktop.org/archives/nouveau/2015-July/021741.html
>>
>> And then adding Option "DRI" "2" to xorg.conf.
>
> His patches should have defaulted it to DRI 2 I think, so this is
> unnecessary. In fact you should have had to say "DRI" "3" to get DRI3
> with his patches.
> --
>>
>> I've also tried disabling EXA using Option "AccelMethod" "none",
>> but that seems to also automatically disable all DRI, leading to
>> software rendering.
>>
>> I discussed this with Ben this morning and he suggested that this
>> is likely a Mesa issue since with DRI3 mesa rather then the ddx
>> allocs the surfaces. I've tried disabling swizzling in the
>> mesa code by forcing nv30_miptree_create() to always take
>> the code path for linear textures, but that leads to the exact
>> same result as before that change.
>
> Ah yes. Very different problem indeed. I actually suspect it has to do
> with swizzling. Look at the white pattern of the moon -- it's all in a
> line. That means that it expected some locality and instead it got
> drawn all on a line. If it were merely a stride problem, I'd expect to
> see strips of the moon below and offset from one another.
>
> So... take a look at nv30_miptree_from_handle -- I wonder if it can
> now receive swizzled textures where it couldn't before.
Ok, that does go in the direction I am expecting the problem to be,
but I'm afraid I'm going to need a bit more guidance, what exactly
am I looking for in that function / which "knobs" should I try to
vary / play with to maybe fix this ?
Regards,
Hans
_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau
next prev parent reply other threads:[~2015-08-03 17:31 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-30 13:16 "enable dri3 support without glamor" causes gnome-shell regression on nv4x Hans de Goede
[not found] ` <55BA23AF.1040308-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-07-30 14:09 ` Ilia Mirkin
[not found] ` <CAKb7Uvh97tFMq5c=sW4GD7pciFNiriM8TwgZ3KKu5vYiyhfzpQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-08-03 13:02 ` Hans de Goede
[not found] ` <55BF665A.4020600-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-08-03 15:24 ` poma
2015-08-03 15:36 ` Ilia Mirkin
[not found] ` <CAKb7UviW5e_1nkxXJ_Rud2SGF8=QUbjyr1aKdSG5h=o=_K7AAw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-08-03 17:31 ` Hans de Goede [this message]
[not found] ` <55BFA569.4080302-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-08-03 18:09 ` Ilia Mirkin
[not found] ` <CAKb7Uvg5SQvg_T++4mpL2_3EFKO8i4CCfz5MeLxe6P5WKTtPKg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-08-10 12:47 ` Hans de Goede
[not found] ` <55C89D5A.9060803-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-08-11 12:21 ` Ilia Mirkin
[not found] ` <CAKb7UvgkfVYvjUeKDFm_skkZn+NhgRPkvTnQcRLOqH40FE9rJg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-08-11 13:13 ` Hans de Goede
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=55BFA569.4080302@redhat.com \
--to=hdegoede-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
--cc=imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org \
--cc=maarten.lankhorst-GeWIH/nMZzLQT0dZR+AlfA@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 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.