Re: [Nouveau] [Mesa-dev] [RFC PATCH 5/8] nv50: prevent NULL pointer dereference with pipe_query functions

2015-06-23 Thread Samuel Pitoiset



On 06/23/2015 08:57 AM, Michel Dänzer wrote:

On 23.06.2015 06:02, Samuel Pitoiset wrote:


On 06/22/2015 10:52 PM, Ilia Mirkin wrote:

If query_create fails, why would any of these functions get called?

Because the HUD doesn't check if query_create() fails and it calls other
pipe_query functions with NULL pointer instead of a valid query object.

Could the HUD code be fixed instead?
It's definitely possible, and probably the best solution instead of 
preventing NULL pointer dereference in the underlying drivers. I'll make 
a patch.





___
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau


[Nouveau] [Bug 90932] gm107 font glitches with kernel 4.1rc

2015-06-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90932

--- Comment #3 from Marcus Moeller  ---
It still appears in Kernel 4.1 final.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau


[Nouveau] [Bug 90932] gm107 font glitches with kernel 4.1rc

2015-06-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90932

--- Comment #4 from Marcus Moeller  ---
Created attachment 116671
  --> https://bugs.freedesktop.org/attachment.cgi?id=116671&action=edit
Screencast showing the behavior

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau


[Nouveau] [Bug 90932] gm107 font glitches with kernel 4.1rc

2015-06-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90932

poma  changed:

   What|Removed |Added

 Attachment #116671|text/plain  |application/octet-stream
  mime type||

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau


[Nouveau] [Bug 90932] gm107 font glitches with kernel 4.1rc

2015-06-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90932

--- Comment #5 from Ilia Mirkin  ---
(In reply to Marcus Moeller from comment #3)
> It still appears in Kernel 4.1 final.

Updating Xorg to 1.17 and switching to the modesetting X ddx (instead of the
nouveau X ddx) should fix this. The reason you're seeing this in 4.1 and not
4.0 is that 4.1 finally has acceleration enabled by default for GM107.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau


[Nouveau] [Bug 90932] gm107 font glitches with kernel 4.1rc

2015-06-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90932

--- Comment #6 from poma  ---

/etc/X11/xorg.conf.d/00-modesetting.conf
Section "Device"
Identifier "video0"
Driver "modesetting"
EndSection

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau


[Nouveau] [Bug 90932] gm107 font glitches with kernel 4.1rc

2015-06-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90932

--- Comment #7 from Marcus Moeller  ---
Shouln't this be fixed in nouveau instead of moving to another driver? Or is
this just a suggestion for a workaround?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau


[Nouveau] [Bug 90932] gm107 font glitches with kernel 4.1rc

2015-06-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90932

--- Comment #8 from Ilia Mirkin  ---
(In reply to Marcus Moeller from comment #7)
> Shouln't this be fixed in nouveau instead of moving to another driver? Or is
> this just a suggestion for a workaround?

The fix in nouveau will be "remove glamor integration" and "fail to load for
GM107+". But we haven't fully decided on that *quite* yet. And I have an EXA
impl in the works for GM107, so perhaps we'll use that. TBD.

The problem that you're seeing is with glamor, and they changed the way font
rendering was done in X 1.17, which is why I'm suggesting you use that.

Separate from that, nouveau's integration with glamor is totally fubar'd and
should be removed, as per the above, but that has nothing to do with any
misrendering you might see. For example perhaps you might observe the lack of a
GLX_ARB_create_context_profile. This is a symptom of DRI2 support basically
missing, which is going to cause major problems.

Since nouveau's integration with glamor would be no different from
modesetting's integration with glamor, might as well just use that directly...
[There are one or two things nouveau can still do that modesetting can't, but
those differences are in the process of being eliminated.]

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau


[Nouveau] [Bug 90932] gm107 font glitches with kernel 4.1rc

2015-06-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90932

--- Comment #9 from Marcus Moeller  ---
Thanks for the explanation. For me the exa sounds like a great option. Besides
that we have tried the modesetting driver with the K620 card and noticed a much
higher CPU load on gnome-shell (even on inactivity).

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau