Re: [PATCH v2 22/25] drm/vmwgfx: Compute dumb-buffer sizes with drm_mode_size_dumb()

2025-01-12 Thread Zack Rusin
On Thu, Jan 9, 2025 at 10:03 AM Thomas Zimmermann wrote: > > Call drm_mode_size_dumb() to compute dumb-buffer scanline pitch > and buffer size. No alignment required. > > Signed-off-by: Thomas Zimmermann > Cc: Zack Rusin > Cc: Broadcom internal kernel review list >

Re: [PATCH v2 1/2] drm/ttm: Change ttm_device_init to use a struct instead of multiple bools

2024-10-03 Thread Zack Rusin
n of adding yet another bool flag parameter to the > function. > > Cc: Christian König > Cc: amd-...@lists.freedesktop.org > Cc: intel-...@lists.freedesktop.org > Cc: nouv...@lists.freedesktop.org > Cc: spice-devel@lists.freedesktop.org > Cc: dri-de...@lists.freedesktop.org >

Re: [PATCH v2] drm/qxl: Pin buffer objects for internal mappings

2024-07-10 Thread Zack Rusin
1-65d02bb02...@suse.de/ > Tested-by: David Kaplan > Reviewed-by: Daniel Vetter > Cc: Thomas Zimmermann > Cc: Dmitry Osipenko > Cc: Christian König > Cc: Zack Rusin > Cc: Dave Airlie > Cc: Gerd Hoffmann > Cc: virtualizat...@lists.linux.dev > Cc: spice-devel@

Re: [PATCH 00/13] drm: Fix reservation locking for pin/unpin and console

2024-02-28 Thread Zack Rusin
1f4a62d64 > prerequisite-patch-id: f5d4bf16dc45334254527c2e31ee21ba4582761c > prerequisite-patch-id: 734c87e610747779aa41be12eb9e4c984bdfa743 > prerequisite-patch-id: 0aa359f6144c4015c140c8a6750be19099c676fb > prerequisite-patch-id: c67e5d886a47b7d0266d81100837557fda34cb24 > prerequisite-patch-id: cbc453ee02fae02af22fbfdce56ab732c7a88c36 > -- > 2.43.2 > That's a really nice cleanup! I already gave a r-b for 9/13. For the rest: Acked-by: Zack Rusin z

Re: [PATCH 08/13] drm/qxl: Acquire reservation lock in GEM pin/unpin callbacks

2024-02-28 Thread Zack Rusin
On Tue, Feb 27, 2024 at 6:39 AM Thomas Zimmermann wrote: > > Acquire the reservation lock directly in GEM pin callback. Same for > unpin. Prepares for further changes. > > Dma-buf locking semantics require callers to hold the buffer's > reservation lock when invoking the pin and unpin callbacks. P

Re: [PATCH 09/13] drm/gem: Acquire reservation lock in drm_gem_{pin/unpin}()

2024-02-28 Thread Zack Rusin
m_gem_object *obj) > { > - vmw_gem_pin_private(obj, false); > + struct vmw_bo *vbo = to_vmw_bo(obj); > + > + vmw_bo_pin_reserved(vbo, false); > } > > static struct sg_table *vmw_gem_object_get_sg_table(struct drm_gem_object > *obj) > diff --git a/include/drm/drm_gem_shmem_helper.h > b/include/drm/drm_gem_shmem_helper.h > index eb12aa9a8c556..efbc9f27312b5 100644 > --- a/include/drm/drm_gem_shmem_helper.h > +++ b/include/drm/drm_gem_shmem_helper.h > @@ -175,15 +175,8 @@ static inline void > drm_gem_shmem_object_print_info(struct drm_printer *p, unsign > static inline int drm_gem_shmem_object_pin(struct drm_gem_object *obj) > { > struct drm_gem_shmem_object *shmem = to_drm_gem_shmem_obj(obj); > - int ret; > > - ret = dma_resv_lock_interruptible(shmem->base.resv, NULL); > - if (ret) > - return ret; > - ret = drm_gem_shmem_pin_locked(shmem); > - dma_resv_unlock(shmem->base.resv); > - > - return ret; > + return drm_gem_shmem_pin_locked(shmem); > } > > /** > @@ -197,9 +190,7 @@ static inline void drm_gem_shmem_object_unpin(struct > drm_gem_object *obj) > { > struct drm_gem_shmem_object *shmem = to_drm_gem_shmem_obj(obj); > > - dma_resv_lock(shmem->base.resv, NULL); > drm_gem_shmem_unpin_locked(shmem); > - dma_resv_unlock(shmem->base.resv); > } > > Ah, I see. Looks great. Reviewed-by: Zack Rusin

[Spice-devel] [PATCH v5 4/9] drm/qxl: Use the hotspot properties from cursor planes

2023-07-18 Thread Zack Rusin
From: Zack Rusin Atomic modesetting got support for mouse hotspots via the hotspot properties. Port the legacy kms hotspot handling to the new properties on cursor planes. Signed-off-by: Zack Rusin Reviewed-by: Gerd Hoffmann Cc: Dave Airlie Cc: Daniel Vetter Cc: virtualizat...@lists.linux

[Spice-devel] [PATCH v5 1/9] drm: Disable the cursor plane on atomic contexts with virtualized drivers

2023-07-18 Thread Zack Rusin
From: Zack Rusin Cursor planes on virtualized drivers have special meaning and require that the clients handle them in specific ways, e.g. the cursor plane should react to the mouse movement the way a mouse cursor would be expected to and the client is required to set hotspot properties on it in

[Spice-devel] [PATCH v4 1/8] drm: Disable the cursor plane on atomic contexts with virtualized drivers

2023-06-27 Thread Zack Rusin
From: Zack Rusin Cursor planes on virtualized drivers have special meaning and require that the clients handle them in specific ways, e.g. the cursor plane should react to the mouse movement the way a mouse cursor would be expected to and the client is required to set hotspot properties on it in

[Spice-devel] [PATCH v4 4/8] drm/qxl: Use the hotspot properties from cursor planes

2023-06-27 Thread Zack Rusin
From: Zack Rusin Atomic modesetting got support for mouse hotspots via the hotspot properties. Port the legacy kms hotspot handling to the new properties on cursor planes. Signed-off-by: Zack Rusin Reviewed-by: Gerd Hoffmann Cc: Dave Airlie Cc: Daniel Vetter Cc: virtualizat...@lists.linux

[Spice-devel] [PATCH v3 4/8] drm/qxl: Use the hotspot properties from cursor planes

2023-06-26 Thread Zack Rusin
From: Zack Rusin Atomic modesetting got support for mouse hotspots via the hotspot properties. Port the legacy kms hotspot handling to the new properties on cursor planes. Signed-off-by: Zack Rusin Reviewed-by: Gerd Hoffmann Cc: Dave Airlie Cc: Daniel Vetter Cc: virtualizat...@lists.linux

[Spice-devel] [PATCH v3 1/8] drm: Disable the cursor plane on atomic contexts with virtualized drivers

2023-06-26 Thread Zack Rusin
From: Zack Rusin Cursor planes on virtualized drivers have special meaning and require that the clients handle them in specific ways, e.g. the cursor plane should react to the mouse movement the way a mouse cursor would be expected to and the client is required to set hotspot properties on it in

Re: [Spice-devel] [PATCH v2 1/8] drm: Disable the cursor plane on atomic contexts with virtualized drivers

2023-05-04 Thread Zack Rusin
On Wed, 2023-05-03 at 10:54 +0300, Pekka Paalanen wrote: > On Wed, 3 May 2023 03:35:29 + > Zack Rusin wrote: > > > On Tue, 2023-05-02 at 11:32 +0200, Javier Martinez Canillas wrote: > > > !! External Email > > > > > > Daniel Vetter writes: >

Re: [Spice-devel] [PATCH v2 1/8] drm: Disable the cursor plane on atomic contexts with virtualized drivers

2023-05-04 Thread Zack Rusin
On Wed, 2023-05-03 at 09:48 +0200, Javier Martinez Canillas wrote: > Zack Rusin writes: > > > On Tue, 2023-05-02 at 11:32 +0200, Javier Martinez Canillas wrote: > > > !! External Email > > > > > > Daniel Vetter writes: > > > > > > >

Re: [Spice-devel] [PATCH v2 1/8] drm: Disable the cursor plane on atomic contexts with virtualized drivers

2023-05-03 Thread Zack Rusin
On Tue, 2023-05-02 at 11:32 +0200, Javier Martinez Canillas wrote: > !! External Email > > Daniel Vetter writes: > > > On Mon, Jul 11, 2022 at 11:32:39PM -0400, Zack Rusin wrote: > > > From: Zack Rusin > > > > > > Cursor planes on virtual

[Spice-devel] [PATCH v2 1/8] drm: Disable the cursor plane on atomic contexts with virtualized drivers

2022-07-12 Thread Zack Rusin
From: Zack Rusin Cursor planes on virtualized drivers have special meaning and require that the clients handle them in specific ways, e.g. the cursor plane should react to the mouse movement the way a mouse cursor would be expected to and the client is required to set hotspot properties on it in

[Spice-devel] [PATCH v2 4/8] drm/qxl: Use the hotspot properties from cursor planes

2022-07-12 Thread Zack Rusin
From: Zack Rusin Atomic modesetting got support for mouse hotspots via the hotspot properties. Port the legacy kms hotspot handling to the new properties on cursor planes. Signed-off-by: Zack Rusin Cc: Dave Airlie Cc: Gerd Hoffmann Cc: Daniel Vetter Cc: virtualizat...@lists.linux

[Spice-devel] [PATCH 3/6] drm/qxl: Create mouse hotspot properties on cursor planes

2022-06-02 Thread Zack Rusin
From: Zack Rusin Atomic modesetting got support for mouse hotspots via the hotspot properties. Drivers need to create those properties on cursor planes which require the mouse hotspot coordinates. Add the code creating hotspot properties and port away from old legacy hotspot API. The legacy

[Spice-devel] [PATCH v2 4/5] drm/qxl: Use TTM builtin resource manager debugfs code

2022-04-12 Thread Zack Rusin
From: Zack Rusin Switch to using the TTM resource manager debugfs helpers. The functionality is largely the same. The TTM resource managers need to stay valid for as long as the drm debugfs_root is valid. Signed-off-by: Zack Rusin Cc: Dave Airlie Cc: Gerd Hoffmann Cc: Daniel Vetter Cc

[Spice-devel] [PATCH 4/5] drm/qxl: Use TTM builtin resource manager debugfs code

2022-04-06 Thread Zack Rusin
From: Zack Rusin Switch to using the TTM resource manager debugfs helpers. The functionality is largely the same, except that the entries live under /sys/kernel/debug/ttm/ and their lifetimes are managed by TTM. Signed-off-by: Zack Rusin Cc: Dave Airlie Cc: Gerd Hoffmann Cc: Daniel Vetter

Re: [Spice-devel] [patch 2/7] drm/vmgfx: Replace kmap_atomic()

2021-03-05 Thread Zack Rusin
usage with the given pgprot. > > Remove the NULL pointer check for the map. These functions return a valid > address for valid pages and the return was bogus anyway as it would have > left preemption and pagefaults disabled. > > Signed-off-by: Thomas Gleixner > Cc: VMware Grap

Re: [Spice-devel] [PATCH 14/15] drm/vmwgfx: Remove references to struct drm_device.pdev

2020-12-02 Thread Zack Rusin
> On Dec 2, 2020, at 09:27, Thomas Zimmermann wrote: > > Hi > > Am 02.12.20 um 09:01 schrieb Thomas Zimmermann: >> Hi >> Am 30.11.20 um 21:59 schrieb Zack Rusin: >>> >>> >>>> On Nov 24, 2020, at 06:38, Thomas Zimmermann wrote: >&

Re: [Spice-devel] [PATCH 14/15] drm/vmwgfx: Remove references to struct drm_device.pdev

2020-12-01 Thread Zack Rusin
vmwgfx/vmwgfx_cmdbuf.c | 8 > drivers/gpu/drm/vmwgfx/vmwgfx_drv.c| 27 +- > drivers/gpu/drm/vmwgfx/vmwgfx_fb.c | 2 +- Reviewed-by: Zack Rusin z ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http