Hi,
On 08-09-16 07:59, Michel Dänzer wrote:
On 08/09/16 02:32 PM, Hans de Goede wrote:
On 08-09-16 03:10, Michel Dänzer wrote:
On 08/09/16 12:43 AM, Hans de Goede wrote:
xf86_crtc_rotate_coord_back() is described as "Given a cursor source
coordinate, rotate to a screen coordinate" which is e
On 08/09/16 04:18 PM, Hans de Goede wrote:
> On 08-09-16 07:59, Michel Dänzer wrote:
>> On 08/09/16 02:32 PM, Hans de Goede wrote:
>>> On 08-09-16 03:10, Michel Dänzer wrote:
On 08/09/16 12:43 AM, Hans de Goede wrote:
>
[0] BTW, do we really need two functions? The method used in
Hi,
On 08-09-16 09:24, Michel Dänzer wrote:
On 08/09/16 04:18 PM, Hans de Goede wrote:
On 08-09-16 07:59, Michel Dänzer wrote:
On 08/09/16 02:32 PM, Hans de Goede wrote:
On 08-09-16 03:10, Michel Dänzer wrote:
On 08/09/16 12:43 AM, Hans de Goede wrote:
[0] BTW, do we really need two funct
When a GPU gets hotplugged while X is already running, glamor_egl_init()
gets called and changes the current egl context, without updating
lastGLContext, potentially causing the next glamor call on another GPU to
run in the wrong context.
This causes glamor to e.g. crash in the next glamor_create_
xf86_crtc_rotate_coord should be the exact inverse operation of
xf86_crtc_rotate_coord_back, but when calculating x / y for 90 / 270
degrees rotation it was using height to calculate x / width to calculate y,
instead of the otherway around.
This was likely not noticed before since xf86_crtc_rotate
The CurrentCursor is always attached to the master GPU.
Signed-off-by: Hans de Goede
Reviewed-by: Michel Dänzer
---
hw/xfree86/ramdac/xf86Cursor.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/hw/xfree86/ramdac/xf86Cursor.c b/hw/xfree86/ramdac/xf86Cursor.c
index 0
This is a preparation patch for adding prime hw-cursor support.
Signed-off-by: Hans de Goede
Reviewed-by: Michel Dänzer
---
randr/randrstr.h | 6 ++
randr/rrcrtc.c | 19 +++
2 files changed, 25 insertions(+)
diff --git a/randr/randrstr.h b/randr/randrstr.h
index 1baa912.
From: Dave Airlie
This is a preparation patch for adding prime hw-cursor support.
Signed-off-by: Dave Airlie
Signed-off-by: Hans de Goede
Reviewed-by: Michel Dänzer
---
dix/privates.c | 9 +
include/privates.h | 3 +++
2 files changed, 12 insertions(+)
diff --git a/dix/privates.
From: Dave Airlie
This is a preparation patch for adding prime hw-cursor support.
Signed-off-by: Dave Airlie
Signed-off-by: Hans de Goede
---
Changes in v6 (Hans de Goede):
-Do not remove xorg_list_is_empty(&pScreen->pixmap_dirty_list)
check from xf86CursorSetCursor()
Changes in v5 (Hans de
When a slave-output is rotated the transformation is done on the blit
from master to slave GPU, so crtc->transform_in_use is not set, but we
still need to adjust the mouse position for things to work.
This commit modifies xf86_crtc_transform_cursor_position to not rely
on crtc->f_framebuffer_to_cr
Hi,
On 08-09-16 01:32, Alex Goins wrote:
Hi Hans,
Thanks for this, it will definitely be a big improvement.
I've been testing these patches against the NVIDIA driver. Some concerns -
What would be the best way to query from the master if the slave is using a HW
cursor? We typically composite
From: Dave Airlie
Currently with PRIME if we detect a secondary GPU,
we switch to using SW cursors, this isn't optimal,
esp for the intel/nvidia combinations, we have
no choice for the USB offload devices.
This patch checks on each slave screen if hw
cursors are enabled, and also calls set curso
Please consider reviewing my latest Xephyr patches (at least the latest one
- I'm afraid things may break in Xephyr running in mui-seat mode after
Keith's new event-handling API):
https://patchwork.freedesktop.org/patch/89109/
https://patchwork.freedesktop.org/patch/89110/
https://patchwork.freede
Will be used for getting unaccelerated motion events and later for
relative motions used by a pointer warp emulator.
Signed-off-by: Jonas Ådahl
---
configure.ac | 4 +++-
hw/xwayland/.gitignore | 2 ++
hw/xwayland/Makefile.am | 21 +
hw/xwayland/x
This function will be called when a pointer is grabbed non-root window
set as the 'confineTo'. This will enable the ddx to handle the
confinement their own way.
Signed-off-by: Jonas Ådahl
---
dix/events.c | 5 +
include/scrnintstr.h | 5 +
2 files changed, 10 insertions(+)
diff
Wait until wl_pointer.frame with dispatching the pointer motion event,
if wl_pointer.frame is supported by the compositor. This will later be
used to combine unaccelerated motion deltas with the absolute motion
delta.
Signed-off-by: Jonas Ådahl
---
hw/xwayland/xwayland-input.c | 75 +
Signed-off-by: Jonas Ådahl
---
hw/xwayland/xwayland.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/hw/xwayland/xwayland.c b/hw/xwayland/xwayland.c
index 847321e..5ceab0f 100644
--- a/hw/xwayland/xwayland.c
+++ b/hw/xwayland/xwayland.c
@@ -102,6 +102,12 @@ static De
Translate grabbing a pointer device with confineTo set to a window into
confining the Wayland pointer using the pointer constraints protocol.
This makes clients that depend on the pointer not going outside of the
window region, such as certain games and virtual machines viewers, to
function more pr
If there was an relative pointer motion within the same frame as an
absolute pointer motion, provide both the absolute coordinate and the
unaccelerated delta when setting the valuator mask.
If a frame contained only a relative motion, queue an absolute motion
with an unchanged position, but still
Add a valuator mask setter for setting absolute coordinate combined
with unaccelerated motion deltas. This will later be used by Xwayland
to combine a wl_pointer.motion() event with the unaccelerated delta of
a wp_relative_pointer.relative_motion() event.
Signed-off-by: Jonas Ådahl
---
dix/input
Emulate pointer warps by locking the pointer and sending relative
motion events instead of absolute. X will keep track of the "fake"
pointer cursor position given the relative motion events, and the
client warping the cursor will warp the faked cursor position.
Various requirements need to be met
This new vfunc will be called, if set, after a client has issued a
WarpPointer request. This is necessary for implementing pointer warp
emulation in Xwayland.
Signed-off-by: Jonas Ådahl
---
dix/events.c | 3 +++
include/scrnintstr.h | 9 +
2 files changed, 12 insertions(+)
diff
Hi,
This series adds support for raw pointer events, pointer confinement and
introduces pointer warp emulation to implement pointer-lock like features
clients use pointer warping for.
Raw pointer events is done by adding a new valuator mask setter that takes both
an absolute value and an unaccele
Put device class initialization in init_[device_class](xwl_seat) and
releasing in release_[device class](xwl_seat). The purpose is to make
it easier to add more type of initialization here later, without making
the function too large.
Signed-off-by: Jonas Ådahl
---
hw/xwayland/xwayland-input.c |
From: Krzysztof Sobiecki
Generating relative and absolute movement events from the same input
device is problematic, because an absolute pointer device doesn't
expect to see any relative motion events. To be able to generate
relative pointer motion events including unaccelerated deltas, create a
Will be used by the pointer warp emulator.
Signed-off-by: Jonas Ådahl
---
hw/xwayland/.gitignore | 2 ++
hw/xwayland/Makefile.am | 9 -
hw/xwayland/xwayland-input.c | 13 +
hw/xwayland/xwayland.h | 2 ++
4 files changed, 25 insertions(+), 1 deletion(-)
di
Hans de Goede writes:
> Signed-off-by: Hans de Goede
> ---
> Changes in v2:
> -Set lastGLContext to NULL instead of saving and restoring the current
> context
Reviewed-by: Keith Packard
--
-keith
signature.asc
Description: PGP signature
___
xorg-
This fix is for the following xorg.conf can work:
Section "ServerFlags"
Option "AutoAddGPU" "off"
EndSection
Section "Device"
Identifier "Amd"
Driver "ati"
BusID "PCI:1:0:0"
EndSection
Section "Device"
Identifier "Intel"
Driver "modesetting"
Two patches which ensure that input drivers aren't re-entered from the
xserver, the first is for deviceProc, the second for
setProperty. Reading input events is already taken care of by the
input thread.
--
-keith
___
xorg-devel@lists.x.org: X.Org develo
This keeps the input driver SetProperty function from being called
while input events are being processed.
Signed-off-by: Keith Packard
---
Xi/xiproperty.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Xi/xiproperty.c b/Xi/xiproperty.c
index 26d1206..b7a1f59 100644
--- a/Xi/xiproperty.c
This ensures that the deviceProc is never called while the input
thread is processing data from the device.
Signed-off-by: Keith Packard
---
dix/devices.c | 22 +++---
1 file changed, 19 insertions(+), 3 deletions(-)
diff --git a/dix/devices.c b/dix/devices.c
index 9de84a6..56aa
After updating to Xorg server 1.14.7 (from something much older) I'm
seeing letters disappear (become blank background) and reappear,
typically when the cursor is moving over them and flashing.
This is on an OpenIndiana.org system running under vmware, and using
the vmware_drv para-virtual driver.
Thanks Hans for the insight into detecting HW cursor usage.
+Daniel Vetter, could you comment on this possible issue with
drmModeMoveCursor() and drmModePageFlip() interfering with each other on i915?
> I'm encountering some very strange issues when running these with PRIME
> Synchronization enab
On 08/09/16 07:08 PM, Hans De Goede wrote:
> When a slave-output is rotated the transformation is done on the blit
> from master to slave GPU, so crtc->transform_in_use is not set, but we
> still need to adjust the mouse position for things to work.
>
> This commit modifies xf86_crtc_transform_cur
On 01/09/16 12:27 AM, Alex Deucher wrote:
> On Wed, Aug 31, 2016 at 4:56 AM, Michel Dänzer wrote:
>> From: Michel Dänzer
>>
>> Signed-off-by: Michel Dänzer
>
> Reviewed-by: Alex Deucher
Thanks, Alex.
If there are no objections, I'll push this change next week.
--
Earthling Michel Dänzer
On Thu, Sep 08, 2016 at 11:02:30AM -0600, Keith Packard wrote:
> This ensures that the deviceProc is never called while the input
> thread is processing data from the device.
>
> Signed-off-by: Keith Packard
> ---
> dix/devices.c | 22 +++---
> 1 file changed, 19 insertions(+), 3
On Thu, Sep 08, 2016 at 11:02:31AM -0600, Keith Packard wrote:
> This keeps the input driver SetProperty function from being called
> while input events are being processed.
>
> Signed-off-by: Keith Packard
> ---
> Xi/xiproperty.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Xi/xi
37 matches
Mail list logo