Two X servers, one box

2012-07-03 Thread Cook, Rich
Hi, I am trying to run two instances of X.Org X Server 1.10.4 per node of our dual-GPU Red Hat cluster. I do this: /usr/bin/startx /usr/bin/xterm -display :0 -- -nolock -auth /g/g0/rcook/.Xauthority-0 :0 and this: /usr/bin/startx /usr/bin/xterm -display :1 -- -nolock -auth

Re: How to install/enable xf86-video-modesetting-0.4.0?

2012-07-03 Thread Alan Cox
On Tue, 3 Jul 2012 17:41:24 +0100 Moon Dust Junky moondustju...@gmail.com wrote: Hi, I have an Intel Atom N2600 netbook with GMA 3600 graphics. I read that I need the modesetting driver to get proper resolution of 1024x600. (see article https://gist.github.com/2925633) I am running

2 screens with independent xservers ?

2012-07-03 Thread Sérgio Basto
Hi, I got a laptop with one intel(0): Chipset: Arrandale is it possible ? when attach one screen monitor, put there a new xserver with is own status bar and full-screen etc. And mouse and keyboard change one from other, with x2vnc and x11vnc ? or solutions xdmx something. Thanks, -- Sérgio

Re: Two X servers, one box

2012-07-03 Thread stompdagg...@yahoo.com
any reason why X server per app?   An wise Scandinavian old man once said: in the end, everything is going to be alright From: Cook, Rich coo...@llnl.gov To: stompdagg...@yahoo.com stompdagg...@yahoo.com Cc: xorg@lists.x.org xorg@lists.x.org Sent: Tuesday,

[PATCH xorg-gtest 5/9] Split usage help text into a helper function

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/xorg-gtest_main.cpp | 25 ++--- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git a/src/xorg-gtest_main.cpp b/src/xorg-gtest_main.cpp index bbf232b..08927a0 100644 --- a/src/xorg-gtest_main.cpp +++

[PATCH xorg-gtest 4/9] Fix up a couple of doxygen links

2012-07-03 Thread Peter Hutterer
Introduced in e1c010f23272e61c28c73aa603b477ba6fbae875 Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- examples/xorg-gtest-example.cpp |2 +- include/xorg/gtest/evemu/xorg-gtest_device.h |2 +- include/xorg/gtest/xorg-gtest_environment.h |2 +-

[PATCH xorg-gtest 7/9] Add some comments to the whacky library building

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- examples/Makefile.am |3 +++ 1 file changed, 3 insertions(+) diff --git a/examples/Makefile.am b/examples/Makefile.am index b9b914b..4a3b633 100644 --- a/examples/Makefile.am +++ b/examples/Makefile.am @@ -29,10 +29,12 @@

[PATCH xorg-gtest 2/9] doc: shut up doxygen

2012-07-03 Thread Peter Hutterer
Errors/warnings on building documentation are enough, we don't need to see the whole spew Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- doc/Doxyfile |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/Doxyfile b/doc/Doxyfile index 488b15c..cf58184 100644 ---

[PATCH xorg-gtest 8/9] Rename XSERVER_LIBS to X11_LIBS

2012-07-03 Thread Peter Hutterer
XSERVER implies this has to do with the server, but these are the client libs. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- configure.ac |2 +- examples/Makefile.am |4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac

[PATCH xorg-gtest 9/9] Add note to Makefile.am that new sources must be added to xorg-gtest-all.cpp

2012-07-03 Thread Peter Hutterer
Hopefully saves the next poor sod weird linker errors. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/Makefile.am |1 + 1 file changed, 1 insertion(+) diff --git a/src/Makefile.am b/src/Makefile.am index 148f1f2..0b93df0 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@

Re: [PATCH 17/19 v2] Set padding bytes to 0 in WriteToClient

2012-07-03 Thread Keith Packard
Alan Coopersmith alan.coopersm...@oracle.com writes: Clear them out when needed instead of leaving whatever values were present in previously sent messages. Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com --- v2: Simplified as suggested by Keith to just use memset instead of a

[PATCH 0/5] unified sigio handling

2012-07-03 Thread Peter Hutterer
I left the first patch as-is because it's easy to cherry-pick to 1.12 this way. I also left the xf86BlockSIGIO() calls in though I don't think outside of the ddx actually uses them. We can probably drop them, either this cycle or the next. Cheers, Peter

[PATCH 1/5] xfree86: always enable SIGIO on OsVendorInit (#50957)

2012-07-03 Thread Peter Hutterer
Drivers call xf86InstallSIGIOHandler() for their fd on DEVICE_ON. That function does not actually enable the signal if it was blocked to begin with. As a result, if one vt-switches away from the server (SIGIO is blocked) and then triggers a server regeneration, the signal remains blocked and input

[PATCH 2/5] os: add OsBlockSIGIO and OsReleaseSIGIO

2012-07-03 Thread Peter Hutterer
Let the dix be in charge of changing the sigprocmask so we only have one entity that changes it. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/os.h |6 +++ os/utils.c | 54 +-- test/Makefile.am |3 +- test/os.c| 130

[PATCH 3/5] xfree86: use OsBlockSIGIO from the ddx

2012-07-03 Thread Peter Hutterer
We can ignore the wasset argument now since the DIX will keep proper refcounting. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- hw/xfree86/os-support/shared/sigio.c | 17 ++--- 1 file changed, 2 insertions(+), 15 deletions(-) diff --git

[PATCH 4/5] xfree86: drop ddx-specific SIGIO blocking

2012-07-03 Thread Peter Hutterer
The hooks are left for this cycle, we can drop it next cycle once the drivers that need it (e.g. wacom) have been updated. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- hw/xfree86/common/xf86Cursor.c | 12 +--- hw/xfree86/common/xf86Events.c | 14

Re: [PATCH xorg-gtest 1/9] Add a few linebreaks into the standard error notice

2012-07-03 Thread Keith Packard
Peter Hutterer peter.hutte...@who-t.net writes: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net These patches all seem innocuous enough; For the series, Reviewed-by: Keith Packard kei...@keithp.com -- keith.pack...@intel.com pgpPT88BkAcHC.pgp Description: PGP signature

Re: [PATCH 3/5] xfree86: use OsBlockSIGIO from the ddx

2012-07-03 Thread Keith Packard
Peter Hutterer peter.hutte...@who-t.net writes: We can ignore the wasset argument now since the DIX will keep proper refcounting. Ok, so I was thinking that we should change the API here, but really what we should do is plan to change users to use OsBlockSIGIO/OsReleaseSIGIO directly and plan

[PATCH] Fix alphamap interactions with wfb

2012-07-03 Thread Arcady Goldmints-Orlov
Set a destructor function on pixman images and call fbFinishAccess() from there, rather than directly from free_pixman_pict(). This ensures that fbFinishAccess() gets called even if pixman still has a reference to the image after free_pixman_pict(), as is the case for alphamaps. Reviewed-by:

Re: [PATCH 2/5] os: add OsBlockSIGIO and OsReleaseSIGIO

2012-07-03 Thread Keith Packard
Peter Hutterer peter.hutte...@who-t.net writes: void OsReleaseSignals(void) { #ifdef SIG_BLOCK if (--BlockedSignalCount == 0) { sigprocmask(SIG_SETMASK, PreviousSignalMask, 0); +OsReleaseSIGIO(); } #endif Should you use SIG_UNBLOCK instead of SIG_SETMASK?

Re: [PATCH 0/5] unified sigio handling

2012-07-03 Thread Keith Packard
Peter Hutterer peter.hutte...@who-t.net writes: I left the first patch as-is because it's easy to cherry-pick to 1.12 this way. I also left the xf86BlockSIGIO() calls in though I don't think outside of the ddx actually uses them. We can probably drop them, either this cycle or the next. I

[PATCH xorg-gtest 00/16] adding a bunch of functionality

2012-07-03 Thread Peter Hutterer
This branch is available on git://people.freedesktop.org/~whot/xorg-gtest :devel The main features here are adding a new XServer class that controls most of the server-related bits. The current API is maintained, the bits just move into different classes so existing tests should continue to

[PATCH xorg-gtest 01/16] Add a new class representing the X server

2012-07-03 Thread Peter Hutterer
This class is a Process, so it's a drop-in replacement for the current Environment, but it provides a few methods to talk to the server being started. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- configure.ac|2 +- include/Makefile.am

[PATCH xorg-gtest 03/16] xserver: store config, logfile, binary paths in the XServer object

2012-07-03 Thread Peter Hutterer
And initialise with the default settings Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h | 12 src/environment.cpp |4 +++- src/xserver.cpp | 26 +- 3 files

[PATCH xorg-gtest 02/16] xserver: add WaitForConnections()

2012-07-03 Thread Peter Hutterer
Moved from Environment to XServer class Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h |5 + src/environment.cpp | 34 +- src/xserver.cpp | 35

[PATCH xorg-gtest 04/16] xserver: move starting the process into the XServer object

2012-07-03 Thread Peter Hutterer
The API currently means we lose the ability to pass arbitrary options (aside from display/logfile/conf path), but this should be of minor impact only. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h |6 ++ src/environment.cpp

[PATCH xorg-gtest 05/16] xserver: move testing startup to the XServer object

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h |2 ++ src/environment.cpp | 35 --- src/xserver.cpp | 40 +++ 3 files changed, 42

[PATCH xorg-gtest 07/16] xserver: add Start() for default binary path

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h |4 src/xserver.cpp |4 2 files changed, 8 insertions(+) diff --git a/include/xorg/gtest/xorg-gtest-xserver.h b/include/xorg/gtest/xorg-gtest-xserver.h

[PATCH xorg-gtest 06/16] xserver: move Terminate and Kill handling here

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h | 13 src/environment.cpp | 31 +++- src/xserver.cpp | 34 +++ 3 files changed, 50

[PATCH xorg-gtest 08/16] xserver: return used display number from Start()

2012-07-03 Thread Peter Hutterer
Doesn't do much yet, but once we support -displayfd we can hook it up here. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h |7 +-- src/xserver.cpp |9 ++--- 2 files changed, 11 insertions(+), 5

[PATCH xorg-gtest 09/16] environment: remove default settings

2012-07-03 Thread Peter Hutterer
Keep those in the server only, not the environment. And only override the build-in ones when they've been set by main. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/environment.cpp | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git

[PATCH xorg-gtest 10/16] xserver: replace separate paths with an option/value map

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h | 18 src/xserver.cpp | 35 --- 2 files changed, 41 insertions(+), 12 deletions(-) diff --git

[PATCH xorg-gtest 11/16] environment: use SetOption, instead of separate calls

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/environment.cpp |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/environment.cpp b/src/environment.cpp index 01b2148..f28bec4 100644 --- a/src/environment.cpp +++ b/src/environment.cpp @@ -106,9 +106,9 @@

[PATCH xorg-gtest 12/16] process: add Start(program, vectorchar*)

2012-07-03 Thread Peter Hutterer
Same thing as the va_list but takes a std::vector of arguments instead. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-process.h | 16 src/process.cpp | 20 +--- 2 files changed, 29

[PATCH xorg-gtest 14/16] xserver: update documentation

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/include/xorg/gtest/xorg-gtest-xserver.h b/include/xorg/gtest/xorg-gtest-xserver.h index 707888e..6eb2241

[PATCH xorg-gtest 13/16] xserver: use new Process::Start(vector) call to populate argument list

2012-07-03 Thread Peter Hutterer
There's probably some better way to do this than strdup()ing everything, send me a patch if you know how. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/xserver.cpp | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/src/xserver.cpp

[PATCH xorg-gtest 15/16] test: add SetDisplayString()

2012-07-03 Thread Peter Hutterer
Don't rely on the environment alone, take the display string if it's been set. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-test.h |7 +++ src/test.cpp |8 +++- 2 files changed, 14 insertions(+), 1 deletion(-)

[PATCH xorg-gtest 16/16] Add Device::GetDeviceNode() to return device node path from an evemu device

2012-07-03 Thread Peter Hutterer
evemu doesn't export this information and even evemu-device just trawls through the file system to print this info. So do the same here, noting the time before evemu_create() and the ctime of the new device file. If the latter is later than the former and the device names match, we can assume this

[PATCH 00/10] integration testing for driver behaviour

2012-07-03 Thread Peter Hutterer
This series builds on top of chase' integration test branch, the whole lot is available for, ahaha, testing from git://people.freedesktop.org/~whot/xserver :driver-integration-testing It requires the xorg-gtest patches available from git://people.freedesktop.org/~whot/xorg-gtest :devel The

[PATCH 02/10] test: add integration-check target

2012-07-03 Thread Peter Hutterer
integration tests require a bit more effort than the other tests, so make it run on a separate target. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- Makefile.am |3 +++ test/Makefile.am |2 ++ test/integration/Makefile.am |9 ++--- 3

[PATCH 01/10] test: enable integration test directory only if building integration tests

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- test/Makefile.am | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/test/Makefile.am b/test/Makefile.am index f58faed..aacd0c7 100644 --- a/test/Makefile.am +++ b/test/Makefile.am @@ -1,10 +1,11 @@ +SUBDIRS =

[PATCH 04/10] test/integration: split input driver test from xi2 test

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- test/integration/Makefile.am | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/test/integration/Makefile.am b/test/integration/Makefile.am index 4b02463..eefcbe5 100644 --- a/test/integration/Makefile.am

[PATCH 03/10] test/integration: add basic checks for default core/xtest devices

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- test/integration/Makefile.am |2 +- test/integration/input_drivers.cpp | 128 2 files changed, 129 insertions(+), 1 deletion(-) create mode 100644 test/integration/input_drivers.cpp diff

[PATCH 05/10] test/integration: use new static calls for WaitFor*

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- test/integration/xi2.cpp | 172 ++ 1 file changed, 5 insertions(+), 167 deletions(-) diff --git a/test/integration/xi2.cpp b/test/integration/xi2.cpp index 1cbbe05..5dcf831 100644 ---

[PATCH 06/10] test/integration: decouple input_drivers test from standard mainloop

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- test/integration/input_drivers.cpp | 19 +++ 1 file changed, 19 insertions(+) diff --git a/test/integration/input_drivers.cpp b/test/integration/input_drivers.cpp index 5b3060b..bd0777f 100644 ---

[PATCH 07/10] test/integration: test for default evdev behaviour

2012-07-03 Thread Peter Hutterer
Default behaviour on linux is new devices are assigned the evdev driver and show up with the same name as the uinput device. Test for that. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- test/integration/input_drivers.cpp | 65 1 file changed,

[PATCH 08/10] test/integration: add behaviour tests for AutoAddDevices off

2012-07-03 Thread Peter Hutterer
Check if the default mouse/keyboard sections are added if AutoAddDevices is disabled. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- test/integration/Makefile.am |5 +- test/integration/xorg-conf-input-tests.cpp | 111 2 files changed,

[PATCH 09/10] test/integration: Add default mouse/keyboard driver behaviour

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- test/integration/xorg-conf-input-tests.cpp | 71 1 file changed, 71 insertions(+) diff --git a/test/integration/xorg-conf-input-tests.cpp b/test/integration/xorg-conf-input-tests.cpp index 1a9871e..76512ca

[PATCH 10/10] test/integration: add tests for evdev/synaptics default behaviour

2012-07-03 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- .../SynPS2_Synaptics_TouchPad/device.prop | 33 test/integration/xorg-conf-input-tests.cpp | 82 ++-- 2 files changed, 107 insertions(+), 8 deletions(-) create mode 100644

Re: [PATCH 15/36] xf86dga: handle DGAAvailable for gpu screens.

2012-07-03 Thread Dave Airlie
On Mon, Jul 2, 2012 at 11:52 PM, Keith Packard kei...@keithp.com wrote: Adam Jackson a...@nwnk.net writes: On 7/2/12 6:13 AM, Dave Airlie wrote: I think anywhere you see a screen number on the wire the safe thing to do is index protocolscreens not gpuscreens. DGAAvailable is also called

Re: [PATCH 04/36] dix: introduce gpu screens. (v3)

2012-07-03 Thread Dave Airlie
On Mon, Jul 2, 2012 at 9:07 PM, Adam Jackson a...@nwnk.net wrote: On 7/2/12 6:12 AM, Dave Airlie wrote: +void +RemoveGPUScreen(ScreenPtr pScreen) +{ +int idx, j; +if (!pScreen-isGPU) +return; + +idx = pScreen-myNum - GPU_SCREEN_OFFSET; +for (j = idx; j

Re: [PATCH 12/36] dix: add unattached list for attaching screens to initially.

2012-07-03 Thread Dave Airlie
On Mon, Jul 2, 2012 at 11:45 PM, Keith Packard kei...@keithp.com wrote: Dave Airlie airl...@gmail.com writes: On Mon, Jul 2, 2012 at 8:11 PM, Keith Packard kei...@keithp.com wrote: Well an unbound gpu you probably don't know its a slave, its just unbound. It's about to become a slave

Re: [PATCH 08/36] xfree86: add platform bus hotplug support (v2)

2012-07-03 Thread Dave Airlie
On Mon, Jul 2, 2012 at 10:31 PM, Keith Packard kei...@keithp.com wrote: Dave Airlie airl...@gmail.com writes: No they can all do it, just haven't the code to do the picking yet of a driver, and since USB is really the only thing I can hotplug so far. It seems like more driver ABI changes

[PATCH] dix: introduce gpu screens. (v4)

2012-07-03 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This patch introduces gpu screens into screenInfo. It adds interfaces for adding and removing gpu screens, along with adding private fixup, block handler support, and scratch pixmap init. GPU screens have a myNum that is offset by GPU_SCREEN_OFFSET (256),

Re: [PATCH 15/36] xf86dga: handle DGAAvailable for gpu screens.

2012-07-03 Thread Dave Airlie
On Tue, Jul 3, 2012 at 8:51 AM, Dave Airlie airl...@gmail.com wrote: On Mon, Jul 2, 2012 at 11:52 PM, Keith Packard kei...@keithp.com wrote: Adam Jackson a...@nwnk.net writes: On 7/2/12 6:13 AM, Dave Airlie wrote: I think anywhere you see a screen number on the wire the safe thing to do is

[PATCH] xf86dga: handle DGAAvailable for gpu screens. (v2)

2012-07-03 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com v2: Split out DGAAvailable into two interfaces, one for calls from protocol decoding and one for internal usage, after discussion with ajax and keithp. Signed-off-by: Dave Airlie airl...@redhat.com --- hw/xfree86/common/xf86DGA.c | 15

Re: [PATCH 21/36] dix: pixmap sharing infrastructure (v2)

2012-07-03 Thread Dave Airlie
On Mon, Jul 2, 2012 at 10:32 PM, Keith Packard kei...@keithp.com wrote: Dave Airlie airl...@gmail.com writes: On Mon, Jul 2, 2012 at 8:20 PM, Keith Packard kei...@keithp.com wrote: though a void * that just cases an fd is probably okay. That was my thinking. void * covers a lot of sins.

Re: [PATCH 23/36] randr: add initial scanout pixmap support (v2)

2012-07-03 Thread Dave Airlie
On Tue, Jul 3, 2012 at 12:51 AM, Keith Packard kei...@keithp.com wrote: Dave Airlie airl...@gmail.com writes: From: Dave Airlie airl...@redhat.com When randr notices a crtc configuration request for a slave device, it checks if the slave allocated pixmap exists and is suitable, if not it

Re: [PATCH 19/36] dix: add ability to link output slave gpus to the current gpu

2012-07-03 Thread Dave Airlie
On Tue, Jul 3, 2012 at 12:41 AM, Keith Packard kei...@keithp.com wrote: Dave Airlie airl...@gmail.com writes: +void +AttachOutputGPU(ScreenPtr pScreen, ScreenPtr new) +{ +assert(new-isGPU); +xorg_list_add(new-output_head, pScreen-output_slave_list); +new-current_master =

Re: [PATCH 25/36] randr: fixup constrain to work with slave screens.

2012-07-03 Thread Dave Airlie
On Tue, Jul 3, 2012 at 1:02 AM, Keith Packard kei...@keithp.com wrote: Adam Jackson a...@nwnk.net writes: I'm not sure this makes sense? The list of slave outputs is the list of outputs from other screens connected to _this_ screen's CRTC list, right? So _this_ screen's CRTC list is all you

Re: [PATCH 16/36] xf86: load modesetting driver on Linux hotplug

2012-07-03 Thread Dave Airlie
On Mon, Jul 2, 2012 at 11:55 PM, Keith Packard kei...@keithp.com wrote: Dave Airlie airl...@gmail.com writes: From: Dave Airlie airl...@redhat.com This driver is currently really the only choice for a hotplug at the moment, we can refine this in the future to try and pick the driver names.

Re: [PATCH 35/36] dix/randr: add a hook into screen to replace scanout pixmap

2012-07-03 Thread Dave Airlie
On Mon, Jul 2, 2012 at 10:22 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Mon, Jul 2, 2012 at 6:13 AM, Dave Airlie airl...@gmail.com wrote: From: Dave Airlie airl...@redhat.com For DRI2 in some offload cases we need to set a new pixmap on the crtc, this hook allows dri2 to call into randr

Re: [PATCH 32/36] xf86/cursor: fallback to sw cursor if we have slaves present.

2012-07-03 Thread Dave Airlie
On Mon, Jul 2, 2012 at 10:16 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Mon, Jul 2, 2012 at 6:13 AM, Dave Airlie airl...@gmail.com wrote: From: Dave Airlie airl...@redhat.com Current USB devices have no hw rendered cursors, so we need the master GPU to render the cursor, so whenever we

[PATCH v2] uload unused input modules

2012-07-03 Thread Michal Suchanek
Hello, since the patches got lost somewhere without even generating an error sending as plain email. Changes since v1: - split out module 'query' functions - rename some variables - fix some whitespace and braces Thanks Michal From a86d41ee113f76776a9410205ef6f90c78e41d22 Mon Sep 17

Re: [PATCH 11/36] xfree86: add framework for provider support in ddx.

2012-07-03 Thread Dave Airlie
On Mon, Jul 2, 2012 at 8:35 PM, Dave Airlie airl...@gmail.com wrote: On Mon, Jul 2, 2012 at 8:07 PM, Keith Packard kei...@keithp.com wrote: Dave Airlie airl...@gmail.com writes: +xf86ProviderPtr +xf86ProviderCreate(ScrnInfoPtr scrn, + const xf86ProviderFuncsRec *funcs,

Re: [PATCH] randr: add provider object (v7)

2012-07-03 Thread Dave Airlie
(forgot list first time I sent this). On Mon, Jul 2, 2012 at 10:03 PM, Keith Packard kei...@keithp.com wrote: Dave Airlie airl...@gmail.com writes: On Mon, Jul 2, 2012 at 8:54 PM, Keith Packard kei...@keithp.com wrote: Dave Airlie airl...@gmail.com writes: Yes thats going to be the

Re: [PATCH 18/36] privates: when pixmap privates increase, bump the totalPixmapSize

2012-07-03 Thread Dave Airlie
On Tue, Jul 3, 2012 at 12:41 AM, Keith Packard kei...@keithp.com wrote: Adam Jackson a...@nwnk.net writes: On 7/2/12 6:13 AM, Dave Airlie wrote: From: Dave Airlie airl...@redhat.com This bumps totalPixmapSize in all attached screens. Aieee. totalPixmapSize is now monotonically increasing

Re: [PATCH 17/36] privates: remove asserts that stop dynamic increasing of privates size.

2012-07-03 Thread Dave Airlie
On Mon, Jul 2, 2012 at 10:10 PM, Adam Jackson a...@nwnk.net wrote: On 7/2/12 6:13 AM, Dave Airlie wrote: From: Dave Airlie airl...@redhat.com These asserts stop us increasing the size of privates after create screen resources has been called, however for dynamic screens we need to be able

Re: [PATCH 35/36] dix/randr: add a hook into screen to replace scanout pixmap

2012-07-03 Thread Alex Deucher
On Tue, Jul 3, 2012 at 6:09 AM, Dave Airlie airl...@gmail.com wrote: On Mon, Jul 2, 2012 at 10:22 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Mon, Jul 2, 2012 at 6:13 AM, Dave Airlie airl...@gmail.com wrote: From: Dave Airlie airl...@redhat.com For DRI2 in some offload cases we need to

Re: [PATCH] randr: add provider object (v7)

2012-07-03 Thread Alex Deucher
On Tue, Jul 3, 2012 at 7:57 AM, Dave Airlie airl...@gmail.com wrote: (forgot list first time I sent this). On Mon, Jul 2, 2012 at 10:03 PM, Keith Packard kei...@keithp.com wrote: Dave Airlie airl...@gmail.com writes: On Mon, Jul 2, 2012 at 8:54 PM, Keith Packard kei...@keithp.com wrote:

[PATCH] xfree86: add autoAddGPU option (v2)

2012-07-03 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This option is to stop the X server adding non-primary devices as gpu screens. v2: fix per Keith's suggestion. Signed-off-by: Dave Airlie airl...@redhat.com --- hw/xfree86/common/xf86Config.c | 15 ++- hw/xfree86/common/xf86Globals.c

Re: [PATCH] randr: add provider object (v7)

2012-07-03 Thread Dave Airlie
On Tue, Jul 3, 2012 at 2:57 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Tue, Jul 3, 2012 at 7:57 AM, Dave Airlie airl...@gmail.com wrote: (forgot list first time I sent this). On Mon, Jul 2, 2012 at 10:03 PM, Keith Packard kei...@keithp.com wrote: Dave Airlie airl...@gmail.com writes:

[PATCH] dri2: add initial prime support. (v1.2)

2012-07-03 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This adds the initial prime support for dri2 offload. The main thing is when we get a connection from a prime client, we stored the information and mark all drawables from that client as prime. We then create all buffers for that drawable on the prime device

Re: [PATCH] randr: add provider object (v7)

2012-07-03 Thread Alex Deucher
On Tue, Jul 3, 2012 at 10:10 AM, Dave Airlie airl...@gmail.com wrote: On Tue, Jul 3, 2012 at 2:57 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Tue, Jul 3, 2012 at 7:57 AM, Dave Airlie airl...@gmail.com wrote: (forgot list first time I sent this). On Mon, Jul 2, 2012 at 10:03 PM, Keith

Re: [PATCH 25/36] randr: fixup constrain to work with slave screens.

2012-07-03 Thread Chris Bagwell
On Mon, Jul 2, 2012 at 5:13 AM, Dave Airlie airl...@gmail.com wrote: From: Dave Airlie airl...@redhat.com Current code constrains the cursor to the crtcs on the master device, for slave outputs to work we have to include their crtcs in the constrain calculations. Signed-off-by: Dave Airlie

Re: [PATCH xorg-gtest] Rename headers to use dashes only

2012-07-03 Thread Chase Douglas
On 07/02/2012 08:17 PM, Peter Hutterer wrote: Mixing dashes and spaces in xorg-gtest_foobar.h is a painful naming convention. Use dashes only instead, and provide compat headers plus a warning for those using the old header files. Those users should be using xorg-gtest.h anyway. Signed-off-by:

Re: [PATCH] randr: add provider object (v7)

2012-07-03 Thread Dave Airlie
On Tue, Jul 3, 2012 at 3:25 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Tue, Jul 3, 2012 at 10:10 AM, Dave Airlie airl...@gmail.com wrote: On Tue, Jul 3, 2012 at 2:57 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Tue, Jul 3, 2012 at 7:57 AM, Dave Airlie airl...@gmail.com wrote: (forgot

Re: [PATCH xorg-gtest 1/9] Add a few linebreaks into the standard error notice

2012-07-03 Thread Chase Douglas
Everything in the series looks good. Reviewed-by: Chase Douglas chase.doug...@canonical.com ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH] randr: add provider object (v7)

2012-07-03 Thread Alex Deucher
On Tue, Jul 3, 2012 at 10:49 AM, Dave Airlie airl...@gmail.com wrote: On Tue, Jul 3, 2012 at 3:25 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Tue, Jul 3, 2012 at 10:10 AM, Dave Airlie airl...@gmail.com wrote: On Tue, Jul 3, 2012 at 2:57 PM, Alex Deucher alexdeuc...@gmail.com wrote: On

Re: [PATCH xorg-gtest 01/16] Add a new class representing the X server

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: This class is a Process, so it's a drop-in replacement for the current Environment, but it provides a few methods to talk to the server being started. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- configure.ac

Re: [PATCH xorg-gtest 02/16] xserver: add WaitForConnections()

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: Moved from Environment to XServer class Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h |5 + src/environment.cpp | 34 +-

Re: [PATCH xorg-gtest 04/16] xserver: move starting the process into the XServer object

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: The API currently means we lose the ability to pass arbitrary options (aside from display/logfile/conf path), but this should be of minor impact only. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net ---

Re: [PATCH xorg-gtest 05/16] xserver: move testing startup to the XServer object

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net It makes sense to move the startup to the XServer object, but the environment should then contain an XServer object and start it. The point of the environment is that it starts up an xserver

Re: [PATCH xorg-gtest 06/16] xserver: move Terminate and Kill handling here

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h | 13 src/environment.cpp | 31 +++- src/xserver.cpp | 34

Re: [PATCH xorg-gtest 07/16] xserver: add Start() for default binary path

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h |4 src/xserver.cpp |4 2 files changed, 8 insertions(+) diff --git

Re: [PATCH xorg-gtest 08/16] xserver: return used display number from Start()

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: Doesn't do much yet, but once we support -displayfd we can hook it up here. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h |7 +-- src/xserver.cpp |9

Re: [PATCH xorg-gtest 09/16] environment: remove default settings

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: Keep those in the server only, not the environment. And only override the build-in ones when they've been set by main. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/environment.cpp | 22 +- 1 file changed,

Re: [PATCH] randr: add provider object (v7)

2012-07-03 Thread Keith Packard
Dave Airlie airl...@gmail.com writes: The current problem is I'm not sure any OS exposes muxless and mux in one OS. Mac OSX always uses muxed, Vista the same, and I think Windows 7 always exposes muxless if the bios reports optimus support or the AMD equivalent. So we pick the simplest mux

Re: [PATCH xorg-gtest 10/16] xserver: replace separate paths with an option/value map

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h | 18 src/xserver.cpp | 35 --- 2 files changed, 41 insertions(+), 12

Re: [PATCH xorg-gtest 11/16] environment: use SetOption, instead of separate calls

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/environment.cpp |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/environment.cpp b/src/environment.cpp index 01b2148..f28bec4 100644 ---

Re: [PATCH xorg-gtest 12/16] process: add Start(program, vectorchar*)

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: Same thing as the va_list but takes a std::vector of arguments instead. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-process.h | 16 src/process.cpp | 20

Re: [PATCH xorg-gtest 13/16] xserver: use new Process::Start(vector) call to populate argument list

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: There's probably some better way to do this than strdup()ing everything, send me a patch if you know how. Hopefully it will be easier if we use a vector of std::strings, as I mentioned in the previous patch review :). Signed-off-by: Peter

Re: [PATCH 08/36] xfree86: add platform bus hotplug support (v2)

2012-07-03 Thread Keith Packard
Dave Airlie airl...@gmail.com writes: no further changes needed, granted I've no idea what drivers other than modesetting will ever do proper hotplug, I suppose I could get a real PCIE dock at some point and hotplug radeon or nouveau, but there is a lot of work kernel side before we could

Re: [PATCH xorg-gtest 14/16] xserver: update documentation

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-xserver.h | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/include/xorg/gtest/xorg-gtest-xserver.h

Re: [PATCH 18/36] privates: when pixmap privates increase, bump the totalPixmapSize

2012-07-03 Thread Keith Packard
Dave Airlie airl...@gmail.com writes: So I'm not sure what you mean by per-screen private data, drivers have requirements for per-pixmap private data as well, currently by modesetting driver stashes the slave framebuffer id from the kernel into a pixmap private. By per-screen private data, I

Re: [PATCH 19/36] dix: add ability to link output slave gpus to the current gpu

2012-07-03 Thread Keith Packard
Dave Airlie airl...@gmail.com writes: I could add assert(!new-current_master); which should be true. Yeah, that's probably a good idea for now; I fear mis-use of this API will end with slaves being on multiple masters... -- keith.pack...@intel.com pgpyTbWgfZTTx.pgp Description: PGP

Re: [PATCH 21/36] dix: pixmap sharing infrastructure (v2)

2012-07-03 Thread Keith Packard
Dave Airlie airl...@gmail.com writes: On Mon, Jul 2, 2012 at 10:32 PM, Keith Packard kei...@keithp.com wrote: Dave Airlie airl...@gmail.com writes: On Mon, Jul 2, 2012 at 8:20 PM, Keith Packard kei...@keithp.com wrote: though a void * that just cases an fd is probably okay. That was my

Re: [PATCH 23/36] randr: add initial scanout pixmap support (v2)

2012-07-03 Thread Keith Packard
Dave Airlie airl...@gmail.com writes: So it made sense to stick it in the pixmap when I had two users, I suppose I could try and stash it somewhere else in each user but it might get a bit ugly. Meh. It's a bit weird to have it in the pixmap, but I guess it does seem simplest. --

Re: [PATCH xorg-gtest 15/16] test: add SetDisplayString()

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: Don't rely on the environment alone, take the display string if it's been set. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- include/xorg/gtest/xorg-gtest-test.h |7 +++ src/test.cpp |8 +++-

Re: [PATCH xorg-gtest 16/16] Add Device::GetDeviceNode() to return device node path from an evemu device

2012-07-03 Thread Chase Douglas
On 07/02/2012 11:44 PM, Peter Hutterer wrote: evemu doesn't export this information and even evemu-device just trawls through the file system to print this info. So do the same here, noting the time before evemu_create() and the ctime of the new device file. If the latter is later than the

  1   2   >