Nouveau - Oct 2012 - [Bug 56347] New: NVPlatformProbe routine does not mark the entity as Shared for ZaphodHeads.

https://bugs.freedesktop.org/show_bug.cgi?id=56347

          Priority: medium
            Bug ID: 56347
          Assignee: nouveau at lists.freedesktop.org
           Summary: NVPlatformProbe routine does not mark the entity as
                    Shared for ZaphodHeads.
        QA Contact: xorg-team at lists.x.org
          Severity: normal
    Classification: Unclassified
                OS: All
          Reporter: exc at rbcmail.ru
          Hardware: All
            Status: NEW
           Version: unspecified
         Component: Driver/nouveau
           Product: xorg

NVPlatformProbe routine does not mark the entity as Shared for ZaphodHeads.
It is used by xorg since version 1.13 so there is no possibility to use
ZaphodHeads.

This config:
 75 Section "Device"
 76     Identifier  "CardA1"
 77     Screen    0 
 78         Option     "ZaphodHeads"  "DVI-I-1"
 79     Option  "Monitor-DVI-I-1" "Monitor0"
 80         Option     "NoAccel"    "off"           #
[<bool>]
 81     Driver      "nouveau"
 82     BusID       "PCI:1:0:0"
 83 EndSection
 84 
 85 Section "Device"
 86     Identifier  "CardA2"
 87         Option     "ZaphodHeads"  "VGA-1"
 88     Screen     1
 89     Option  "Monitor-VGA-1" "Monitor1"
 90         Option     "NoAccel"    "off"           #
[<bool>]
 91     Driver      "nouveau"
 92     BusID       "PCI:1:0:0"
 93 EndSection
causes xorg to print "Requested Entity already in use!" and use
PCI:1:0:0 as
single-head adapter.

Thanks.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20121024/d89ef6d5/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=56347

--- Comment #1 from Damian Nowak <nowaker at geozone.pl> ---
Confirming the bug.

02:59 < Nowaker> after distro upgrade I am no longer able to use my 2
gpus, 3
monitors. looks like zaphodheads doesn't work as expected. Screen0 on GPU0
is
OK, Screen1 on GPU0 fails, Screen2 on GPU1 is OK. fail message: Screen 1
deleted because of no matching config section. xorg.conf:
http://wklej.org/id/555276/txt/ This is the bug that was fixed a year ago for
me. https://bugs.freedesktop.org/show_bug.cgi?id=39099
03:00 < Nowaker> instead of getting two screens of GPU0 with zaphodheads,
I get
one (output DVI-I-1) and a clone (output DVI-I-2)
03:01 < Nowaker> something must have been broken along the way, I will
check
the version of nouveau I was using in my previous distro
03:27 < Nowaker> definitely ZaphodHeads broken in current nouveau
03:28 < Nowaker> [  5849.183] Requested Entity already in use!
03:28 < Nowaker> this is not supposed to happen when ZaphodHeads is
defined
03:31 < Nowaker> confirmed,
http://www.spinics.net/lists/xorg/msg54660.html

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20121101/9022708a/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=56347

Damian Nowak <nowaker at geozone.pl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|unspecified                 |git

--- Comment #2 from Damian Nowak <nowaker at geozone.pl> ---
I tried calling xf86SetEntityShared(0) in nv_driver.c before xf86AllocateScreen
and xf86AddEntityToScreen are called. Unfortunately, it isn't enough.

It would be great if any hacker could take a look at it. ZaphodHeads is the
only way of dealing with multiple GPUs and monitors on Nouveau.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20121101/e5dde421/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=56347

--- Comment #3 from Emil Velikov <emil.l.velikov at gmail.com> ---
Guys can you try the patch series as posted in here

http://lists.freedesktop.org/archives/nouveau/2012-November/011347.html

Only the last patch is needed in this case, although I have added a few more
checks to be on the safe side

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20121101/2fc7e57b/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=56347

Damian Nowak <nowaker at geozone.pl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED

--- Comment #4 from Damian Nowak <nowaker at geozone.pl> ---
Nope, doesn't work. :(

http://upload.nowaker.net/nwkr/1351803759_Xorg.0.log
[  1809.910] (II) [drm] nouveau interface version: 1.0.0
[  1809.910] Requested Entity already in use!
[  1809.910] (II) [drm] nouveau interface version: 1.0.0
[  1809.910] (II) [drm] nouveau interface version: 1.0.0
[  1809.910] (EE) Screen 1 deleted because of no matching config section.

I applied all the four patches.
(full patch: http://upload.nowaker.net/nwkr/1351804213_xexaxo-diff)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20121101/fae01387/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=56347

--- Comment #5 from Emil Velikov <emil.l.velikov at gmail.com> ---
Created attachment 69408
  --> https://bugs.freedesktop.org/attachment.cgi?id=69408&action=edit
Four stage patch adding the relevant functions to NVPlatformProbe

Guy can you try the patch series

I have reworked it a bit, to be more consistent with the coding style and
eliminate some minor memory leak I have introduced

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20121102/872d2ecc/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=56347

--- Comment #6 from rodrigo <rodrigorivascosta at gmail.com> ---
I have this very same bug and just tested your patch over the master/HEAD of
today. It works like a charm!

You made my day. Thank you.
--
Rodrigo

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20121103/7374ed35/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=56347

--- Comment #7 from Damian Nowak <nowaker at geozone.pl> ---
Yeah, it works. Just like before, you can add me to Tested-By in the commit.
Can't wait to see that in the upstream. ;-)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20121104/04cf9dd9/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=56347

Emil Velikov <emil.l.velikov at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20121105/919a6b3f/attachment.html>