Booting up with DMA_API_DEBUG_SG=y generates a warning due to the driver
forgot to set dma_parms appropriately. Set it just after vmw_dma_masks()
in vmw_driver_load().
DMA-API: vmwgfx 0000:00:0f.0: mapping sg segment longer than device
claims to support [len=2097152] [max=65536]
WARNING: CPU: 2 PID: 261 at kernel/dma/debug.c:1232
debug_dma_map_sg+0x360/0x480
Hardware name: VMware, Inc. VMware Virtual Platform/440BX Desktop
Reference Platform, BIOS 6.00 04/13/2018
RIP: 0010:debug_dma_map_sg+0x360/0x480
Call Trace:
vmw_ttm_map_dma+0x3b1/0x5b0 [vmwgfx]
vmw_bo_map_dma+0x25/0x30 [vmwgfx]
vmw_otables_setup+0x2a8/0x750 [vmwgfx]
vmw_request_device_late+0x78/0xc0 [vmwgfx]
vmw_request_device+0xee/0x4e0 [vmwgfx]
vmw_driver_load.cold+0x757/0xd84 [vmwgfx]
drm_dev_register+0x1ff/0x340 [drm]
drm_get_pci_dev+0x110/0x290 [drm]
vmw_probe+0x15/0x20 [vmwgfx]
local_pci_probe+0x7a/0xc0
pci_device_probe+0x1b9/0x290
really_probe+0x1b5/0x630
driver_probe_device+0xa3/0x1a0
device_driver_attach+0x94/0xa0
__driver_attach+0xdd/0x1c0
bus_for_each_dev+0xfe/0x150
driver_attach+0x2d/0x40
bus_add_driver+0x290/0x350
driver_register+0xdc/0x1d0
__pci_register_driver+0xda/0xf0
vmwgfx_init+0x34/0x1000 [vmwgfx]
do_one_initcall+0xe5/0x40a
do_init_module+0x10f/0x3a0
load_module+0x16a5/0x1a40
__se_sys_finit_module+0x183/0x1c0
__x64_sys_finit_module+0x43/0x50
do_syscall_64+0xc8/0x606
entry_SYSCALL_64_after_hwframe+0x44/0xa9
Fixes: fb1d9738ca05 ("drm/vmwgfx: Add DRM driver for VMware Virtual GPU")
Suggested-by: Thomas Hellstrom <[email protected]>
Signed-off-by: Qian Cai <[email protected]>
---
drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c
index 4ff11a0077e1..5f690429eb89 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c
@@ -747,6 +747,8 @@ static int vmw_driver_load(struct drm_device *dev, unsigned long chipset)
if (unlikely(ret != 0))
goto out_err0;
+ dma_set_max_seg_size(dev->dev, U32_MAX);
+
if (dev_priv->capabilities & SVGA_CAP_GMR2) {
DRM_INFO("Max GMR ids is %u\n",
(unsigned)dev_priv->max_gmr_ids);
--
1.8.3.1
Reviewed-by: Thomas Hellstrom <[email protected]>
I'll just need to give this some more testing before queueing it on
vmwgfx-fixes.
Thanks,
Thomas
On Mon, 2019-06-03 at 16:44 -0400, Qian Cai wrote:
> Booting up with DMA_API_DEBUG_SG=y generates a warning due to the
> driver
> forgot to set dma_parms appropriately. Set it just after
> vmw_dma_masks()
> in vmw_driver_load().
>
> DMA-API: vmwgfx 0000:00:0f.0: mapping sg segment longer than device
> claims to support [len=2097152] [max=65536]
> WARNING: CPU: 2 PID: 261 at kernel/dma/debug.c:1232
> debug_dma_map_sg+0x360/0x480
> Hardware name: VMware, Inc. VMware Virtual Platform/440BX Desktop
> Reference Platform, BIOS 6.00 04/13/2018
> RIP: 0010:debug_dma_map_sg+0x360/0x480
> Call Trace:
> vmw_ttm_map_dma+0x3b1/0x5b0 [vmwgfx]
> vmw_bo_map_dma+0x25/0x30 [vmwgfx]
> vmw_otables_setup+0x2a8/0x750 [vmwgfx]
> vmw_request_device_late+0x78/0xc0 [vmwgfx]
> vmw_request_device+0xee/0x4e0 [vmwgfx]
> vmw_driver_load.cold+0x757/0xd84 [vmwgfx]
> drm_dev_register+0x1ff/0x340 [drm]
> drm_get_pci_dev+0x110/0x290 [drm]
> vmw_probe+0x15/0x20 [vmwgfx]
> local_pci_probe+0x7a/0xc0
> pci_device_probe+0x1b9/0x290
> really_probe+0x1b5/0x630
> driver_probe_device+0xa3/0x1a0
> device_driver_attach+0x94/0xa0
> __driver_attach+0xdd/0x1c0
> bus_for_each_dev+0xfe/0x150
> driver_attach+0x2d/0x40
> bus_add_driver+0x290/0x350
> driver_register+0xdc/0x1d0
> __pci_register_driver+0xda/0xf0
> vmwgfx_init+0x34/0x1000 [vmwgfx]
> do_one_initcall+0xe5/0x40a
> do_init_module+0x10f/0x3a0
> load_module+0x16a5/0x1a40
> __se_sys_finit_module+0x183/0x1c0
> __x64_sys_finit_module+0x43/0x50
> do_syscall_64+0xc8/0x606
> entry_SYSCALL_64_after_hwframe+0x44/0xa9
>
> Fixes: fb1d9738ca05 ("drm/vmwgfx: Add DRM driver for VMware Virtual
> GPU")
> Suggested-by: Thomas Hellstrom <[email protected]>
> Signed-off-by: Qian Cai <[email protected]>
> ---
> drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c
> b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c
> index 4ff11a0077e1..5f690429eb89 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c
> @@ -747,6 +747,8 @@ static int vmw_driver_load(struct drm_device
> *dev, unsigned long chipset)
> if (unlikely(ret != 0))
> goto out_err0;
>
> + dma_set_max_seg_size(dev->dev, U32_MAX);
> +
> if (dev_priv->capabilities & SVGA_CAP_GMR2) {
> DRM_INFO("Max GMR ids is %u\n",
> (unsigned)dev_priv->max_gmr_ids);