Ray Strode writes:
> Hey cool,
>
> On Friday, September 23, 2016, Eric Anholt wrote:
>>
>> The normal xinit is racy because it doesn't use -displayfd. This
>> implements the bare minimum for testing purposes, using -displayfd to
>> sequence starting the client, and avoids adding yet another dep
Adam Jackson writes:
> You want to make this non-static, so in the xinerama case you can do
> this translation once at the top level.
...
> You're not correcting stuff->coordMode in place, which means under
> xinerama you'll mutate geometry once per screen, which ain't right.
These seem in con
Hey cool,
On Friday, September 23, 2016, Eric Anholt wrote:
>
> The normal xinit is racy because it doesn't use -displayfd. This
> implements the bare minimum for testing purposes, using -displayfd to
> sequence starting the client, and avoids adding yet another dependency
> to the server.
Any
On 09/21/2016 05:08 PM, Maya Rashish wrote:
> pVideo->bus is uint8_t, always less than 256.
>
> Signed-off-by: Maya Rashish
> ---
> hw/xfree86/common/xf86pciBus.c | 6 +-
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/hw/xfree86/common/xf86pciBus.c b/hw/xfree86/common/xf8
On 09/22/2016 04:30 PM, Bob Terek wrote:
> On 09/21/2016 10:22 AM, Aaron Plattner wrote:
>> On 09/20/2016 02:07 AM, Eric Engestrom wrote:
>>> On Tue, Sep 20, 2016 at 01:34:40PM +0700, Antoine Martin wrote:
Signed-off-by: Antoine Martin
>>>
>>> Reviewed-by: Eric Engestrom
>>
>> Looks good to
On Fri, Sep 23, 2016 at 10:58:54 +0300, Eric Anholt wrote:
> The normal xinit is racy because it doesn't use -displayfd. This
> implements the bare minimum for testing purposes, using -displayfd to
> sequence starting the client, and avoids adding yet another dependency
> to the server.
>
> v2:
On Fri, Sep 23, 2016 at 04:45:37PM +0300, Hans de Goede wrote:
> Hi,
>
> On 09/23/2016 04:33 PM, Matthieu Herrb wrote:
> > Hi
> >
> > Adam, Keith, any chance to get those 2 patches merged for 1.19 ?
> > It would reduce the number of local patches I'm maintaining outside of
> > os/os-support.
> >
Hi,
On 09/23/2016 04:33 PM, Matthieu Herrb wrote:
Hi
Adam, Keith, any chance to get those 2 patches merged for 1.19 ?
It would reduce the number of local patches I'm maintaining outside of
os/os-support.
https://patchwork.freedesktop.org/patch/64866/
https://patchwork.freedesktop.org/patch/650
Hi
Adam, Keith, any chance to get those 2 patches merged for 1.19 ?
It would reduce the number of local patches I'm maintaining outside of
os/os-support.
https://patchwork.freedesktop.org/patch/64866/
https://patchwork.freedesktop.org/patch/65081/
If needed I can resend the patches there.
Than
Hi Adam, Keith,
Here is a pull-req consisting of a bunch of fixes written by
me reviewed by others as well as a bunch of fixes from patchwork /
the list which I've picked up and which are reviewed by me.
Note there is one unreviewed patch in here v2 of:
"glx: Always enable EXT_texture_from_pixma
Hi,
On 09/23/2016 03:33 PM, Eric Engestrom wrote:
On Fri, Sep 23, 2016 at 03:12:18PM +0300, Hans de Goede wrote:
Hi,
On 01/13/2016 07:47 AM, Kyle Guinn wrote:
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=93675
Signed-off-by: Kyle Guinn
Thanks for the patch, I've queued this up at:
Michel Dänzer writes:
> On 23/09/16 04:57 PM, Eric Anholt wrote:
>> The copy optimization in d37329cba42fa8e72fe4be8a7be18e512268b5bd
>> replicated a bug from last time we did a copy optimization, and didn't
>> get rendercheck run on it.
>
> Actually, I'm pretty sure I did run rendercheck, but di
On Fri, Sep 23, 2016 at 03:12:18PM +0300, Hans de Goede wrote:
> Hi,
>
> On 01/13/2016 07:47 AM, Kyle Guinn wrote:
> > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=93675
> >
> > Signed-off-by: Kyle Guinn
>
> Thanks for the patch, I've queued this up at:
>
> https://cgit.freedesktop.org/
Hi,
On 12/11/2015 01:05 PM, Daniel Martin wrote:
From: Daniel Martin
We get multiple udev events for actions like docking a laptop into its
station or plugging a monitor to the station. By consuming as much
events as we can, we reduce the number of output re-evalutions.
I.e. having a Lenovo X
Hi,
On 09/23/2016 03:23 PM, Emil Velikov wrote:
On 23 September 2016 at 12:50, Hans de Goede wrote:
Based on: https://patchwork.freedesktop.org/patch/85636/
Rewritten to also free the resources allocated by
panoramix_setup_ids().
The codeflow is a bit nasty to read, but the patch is perfect
On 23 September 2016 at 12:50, Hans de Goede wrote:
> Based on: https://patchwork.freedesktop.org/patch/85636/
>
> Rewritten to just not walk the pointer.
>
Since the free is just outside of context one could mention it and/or
elaborate a bit in general. Regardless the patch is spot on, so
Review
On Fri, 2016-09-23 at 01:15 +0300, Keith Packard wrote:
> +static void
> +FixCoordModePrevious(int npt, xPoint *ppt)
> +{
> +int x, y;
> +
> +x = ppt->x;
> +y = ppt->y;
> +npt--;
> +while (npt--) {
> +ppt++;
> +x = (ppt->x += x);
> +y = (ppt->y += y);
>
On 23 September 2016 at 12:50, Hans de Goede wrote:
> Based on: https://patchwork.freedesktop.org/patch/85636/
>
> Rewritten to also free the resources allocated by
> panoramix_setup_ids().
>
The codeflow is a bit nasty to read, but the patch is perfectly correct.
Reviewed-by: Emil Velikov
-Emi
Each xf86ScreenSetCursor() call calls:
xf86DriverLoadCursorARGB()
infoPtr->ShowCursor()
In succession, ending up in 2 drmModeSetCursor2() calls, with the second
effectively being a no-op. Keep track of having set the cursor already
in drmmode_load_cursor_argb_check() and unless hide() was called
Hi,
On 01/13/2016 07:47 AM, Kyle Guinn wrote:
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=93675
Signed-off-by: Kyle Guinn
Thanks for the patch, I've queued this up at:
https://cgit.freedesktop.org/~jwrdegoede/xserver
For a 1.19 bug-fix pull-req I'm preparing at.
Note I've simplifi
Hi David,
On 05/05/2016 10:39 AM, David CARLIER wrote:
Hi all,
This is a small patch to fix couple of small memory leaks.
Hope it s useful.
It is thanks for your patch, I've split 2 bits out into small patches
and I've dropped the os/rpcauth.c change I think that you are doing
the right thin
Based on: https://patchwork.freedesktop.org/patch/85636/
Rewritten to just not walk the pointer.
Signed-off-by: Hans de Goede
---
Xext/vidmode.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Xext/vidmode.c b/Xext/vidmode.c
index 499a2a8..ea3ad13 100644
--- a/Xext/vidmode.c
Based on: https://patchwork.freedesktop.org/patch/85636/
Rewritten to also free the resources allocated by
panoramix_setup_ids().
Signed-off-by: Hans de Goede
---
Xext/shm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Xext/shm.c b/Xext/shm.c
index 125000f..1b622e3 10
Hi,
On 09/23/2016 11:13 AM, Michel Dänzer wrote:
On 23/09/16 03:46 PM, Hans de Goede wrote:
On 09/20/2016 04:58 AM, Michel Dänzer wrote:
If there are other drivers which can't call xf86HandleColormaps for some
reason, a better solution would be to combine the per-screen gamma set
via pScrn->C
On 09/23/16 12:34, Emil Velikov wrote:
> On 23 September 2016 at 08:07, Alan Coopersmith
> wrote:
>> On 09/23/16 12:05 AM, Emil Velikov wrote:
>>>
>>> On 23 September 2016 at 07:31, Julien Cristau wrote:
On Thu, Sep 22, 2016 at 19:12:13 +0200, Heiko Becker wrote:
That seems li
On 23 September 2016 at 08:07, Alan Coopersmith
wrote:
> On 09/23/16 12:05 AM, Emil Velikov wrote:
>>
>> On 23 September 2016 at 07:31, Julien Cristau wrote:
>>>
>>> On Thu, Sep 22, 2016 at 19:12:13 +0200, Heiko Becker wrote:
>>>
Signed-off-by: Heiko Becker
---
Makefile.am | 2 +-
The following changes since commit d0c5d205a919fc1d2eb599356090b58b1bf0176d:
dix: Make InitCoreDevices() failures more verbose. (2016-09-21 21:11:40 +1000)
are available in the git repository at:
git://people.freedesktop.org/~ofourdan/xserver xwayland
for you to fetch changes up to 0a20d5f8
This code is based on local patches which had been sitting in
FreeBSD and OpenBSD ports.
Reviewed-by: Matthieu Herrb
Signed-off-by: François Tigeot
---
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index dcc3968..4fff769 100644
---
Hi,
Sure, here's the second version.
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel
Each xf86ScreenSetCursor() call calls:
xf86DriverLoadCursorARGB()
infoPtr->ShowCursor()
In succession, ending up in 2 drmModeSetCursor2() calls, with the second
effectively being a no-op. Keep track of having set the cursor already
in drmmode_load_cursor_argb_check() and unless hide() was called
On Fri, Sep 23, 2016 at 09:58:10AM +0200, François Tigeot wrote:
> This code is based on a local patch which had actually been sitting in
> FreeBSD ports since 2015.
>
> Signed-off-by: François Tigeot
> ---
> configure.ac | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/
Hi All,
On 09/16/2016 06:52 PM, Michael Thayer wrote:
When the X server asks us to load a hardware cursor, that request is always
followed up by a request to show it if we report success, or to hide it if
we report failure. Therefore it makes no sense to suppress the request if
the cursor is no
Hi All,
On 09/16/2016 06:51 PM, Michael Thayer wrote:
This change effectively reverts commit 074cf58. We were falling back from
drmModeSetCursor2() to drmModeSetCursor() whenever the first failed. This
fall-back only makes sense on pre-mid-2013 kernels which implemented the
cursor_set hook but
This code is based on a local patch which had actually been sitting in
FreeBSD ports since 2015.
Signed-off-by: François Tigeot
---
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index c09b854..b9e27d0 100644
--- a/configure.ac
+++ b/
On 23/09/16 03:46 PM, Hans de Goede wrote:
> On 09/20/2016 04:58 AM, Michel Dänzer wrote:
>
>> If there are other drivers which can't call xf86HandleColormaps for some
>> reason, a better solution would be to combine the per-screen gamma set
>> via pScrn->ChangeGamma with the per-CRTC gamma set vi
On 23/09/16 04:57 PM, Eric Anholt wrote:
> The copy optimization in d37329cba42fa8e72fe4be8a7be18e512268b5bd
> replicated a bug from last time we did a copy optimization, and didn't
> get rendercheck run on it.
Actually, I'm pretty sure I did run rendercheck, but didn't notice the
regression due t
I was confused by the behavior I'd written before. keithp and
mattst88 responded with shock that I would have made it so surprising,
as well.
v2: Point to index.html instead of problems.html, which won't exist if
we had no problems.
Signed-off-by: Eric Anholt
---
test/scripts/run-piglit.sh
v2: Check that SERVER_COMMAND is set.
Signed-off-by: Eric Anholt
---
test/.gitignore| 1 +
test/Makefile.am | 1 +
test/scripts/{xvfb-piglit.sh => run-piglit.sh} | 16 --
test/scripts/xvfb-piglit.sh| 73 ++---
The normal xinit is racy because it doesn't use -displayfd. This
implements the bare minimum for testing purposes, using -displayfd to
sequence starting the client, and avoids adding yet another dependency
to the server.
v2: Fix asprintf error checks.
Signed-off-by: Eric Anholt
---
test/.gitig
The copy optimization in d37329cba42fa8e72fe4be8a7be18e512268b5bd
replicated a bug from last time we did a copy optimization, and didn't
get rendercheck run on it. This is effectively a re-cherry-pick of
510c8605641803f1f5b5d2de6d3bb422b148e0e7.
Fixes rendercheck -t blend -o src -f x4r4g4b4,x3r4g
Hi,
On 09/22/2016 03:08 AM, Maya Rashish wrote:
pVideo->bus is uint8_t, always less than 256.
Signed-off-by: Maya Rashish
LGTM:
Reviewed-by: Hans de Goede
Regards,
Hans
---
hw/xfree86/common/xf86pciBus.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/hw/xfre
On 09/23/16 12:05 AM, Emil Velikov wrote:
On 23 September 2016 at 07:31, Julien Cristau wrote:
On Thu, Sep 22, 2016 at 19:12:13 +0200, Heiko Becker wrote:
Signed-off-by: Heiko Becker
---
Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Makefile.am b/Makefile.a
On 23 September 2016 at 07:31, Julien Cristau wrote:
> On Thu, Sep 22, 2016 at 19:12:13 +0200, Heiko Becker wrote:
>
>> Signed-off-by: Heiko Becker
>> ---
>> Makefile.am | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/Makefile.am b/Makefile.am
>> index 060112d..9a89bb
43 matches
Mail list logo