search for: swiotlb_alloc_coherent

..., Jerome Glisse wrote: > > Could you bisect ? I would love to point finger upstream to the DMA > folk who made changes to that API without testing with GPU. Rummaging a bit, it might be... nouveau_bo_new() ... ttm_dma_pool_alloc_new_pages() dma_alloc_attrs() ops->alloc() == x86_swiotlb_alloc_coherent() x86_swiotlb_alloc_coherent() flags |= __GFP_NOWARN; swiotlb_alloc_coherent(..flags) swiotlb_alloc_coherent(..flags) attrs = (flags & __GFP_NOWARN) ? DMA_ATTR_NO_WARN : 0; swiotlb_alloc_buffer(..attr) swiotlb_alloc_buffer(..0) <== hm, pass zero inste...

On Thu, 2018-05-10 at 12:28 +0200, Mike Galbraith wrote: > On Thu, 2018-05-10 at 11:10 +0200, Mike Galbraith wrote: > > Greetings, > > > > When box is earning its keep, nouveau/swiotlb grumble.. a LOT. The > > below is from master.today. > > > > [12594.640959] nouveau 0000:01:00.0: swiotlb buffer is full (sz: 2097152 bytes) > > [12594.693000] nouveau

...d for device 0000:01:00.0 size=2097152 [ +0.000004] CPU: 6 PID: 1023 Comm: Xorg Not tainted 4.15.0-rc8 #1 [ +0.000003] Hardware name: Micro-Star International Co., Ltd. GX780/GT780/MS-1761, BIOS E1761IMS V3.01 05/02/2011 [ +0.000003] Call Trace: [ +0.000009] dump_stack+0x9f/0xe1 [ +0.000008] swiotlb_alloc_coherent+0xdf/0x150 [ +0.000010] ttm_dma_pool_get_pages+0x1ec/0x4b0 [ +0.000015] ttm_dma_populate+0x24c/0x340 [ +0.000011] ttm_tt_bind+0x23/0x50 [ +0.000006] ttm_bo_handle_move_mem+0x58c/0x5c0 [ +0.000015] ttm_bo_validate+0x152/0x190 [ +0.000004] ? ttm_bo_init_reserved+0x3d8/0x490 [ +0.000012]...

...t; [ +0.000004] CPU: 6 PID: 1023 Comm: Xorg Not tainted 4.15.0-rc8 #1 >> [ +0.000003] Hardware name: Micro-Star International Co., Ltd. GX780/GT780/MS-1761, BIOS E1761IMS V3.01 05/02/2011 >> [ +0.000003] Call Trace: >> [ +0.000009] dump_stack+0x9f/0xe1 >> [ +0.000008] swiotlb_alloc_coherent+0xdf/0x150 >> [ +0.000010] ttm_dma_pool_get_pages+0x1ec/0x4b0 >> [ +0.000015] ttm_dma_populate+0x24c/0x340 >> [ +0.000011] ttm_tt_bind+0x23/0x50 >> [ +0.000006] ttm_bo_handle_move_mem+0x58c/0x5c0 >> [ +0.000015] ttm_bo_validate+0x152/0x190 >> [ +0.0000...

...for device 0000:01:00.0 size=2097152 [ 1313.811038] CPU: 6 PID: 3026 Comm: Xorg Tainted: G E 4.15.0.g1291a0d5-master #355 [ 1313.811040] Hardware name: MEDION MS-7848/MS-7848, BIOS M7848W08.20C 09/23/2013 [ 1313.811041] Call Trace: [ 1313.811049] dump_stack+0x7c/0xb6 [ 1313.811053] swiotlb_alloc_coherent+0x13f/0x150 [ 1313.811060] ttm_dma_pool_alloc_new_pages+0x106/0x3c0 [ttm] [ 1313.811066] ttm_dma_pool_get_pages+0x10a/0x1e0 [ttm] [ 1313.811070] ttm_dma_populate+0x21f/0x2f0 [ttm] [ 1313.811075] ttm_tt_bind+0x2f/0x60 [ttm] [ 1313.811079] ttm_bo_handle_move_mem+0x51f/0x580 [ttm] [ 1313.811084]...

Greetings, When box is earning its keep, nouveau/swiotlb grumble.. a LOT. The below is from master.today. [12594.640959] nouveau 0000:01:00.0: swiotlb buffer is full (sz: 2097152 bytes) [12594.693000] nouveau 0000:01:00.0: swiotlb buffer is full (sz: 2097152 bytes) [12594.713787] nouveau 0000:01:00.0: swiotlb buffer is full (sz: 2097152 bytes) [12594.743413] nouveau 0000:01:00.0: swiotlb buffer

...size=2097152 > [ 1313.811038] CPU: 6 PID: 3026 Comm: Xorg Tainted: G E 4.15.0.g1291a0d5-master #355 > [ 1313.811040] Hardware name: MEDION MS-7848/MS-7848, BIOS M7848W08.20C 09/23/2013 > [ 1313.811041] Call Trace: > [ 1313.811049] dump_stack+0x7c/0xb6 > [ 1313.811053] swiotlb_alloc_coherent+0x13f/0x150 > [ 1313.811060] ttm_dma_pool_alloc_new_pages+0x106/0x3c0 [ttm] > [ 1313.811066] ttm_dma_pool_get_pages+0x10a/0x1e0 [ttm] > [ 1313.811070] ttm_dma_populate+0x21f/0x2f0 [ttm] > [ 1313.811075] ttm_tt_bind+0x2f/0x60 [ttm] > [ 1313.811079] ttm_bo_handle_move_mem+0x51f/0x...

...size=2097152 > [ +0.000004] CPU: 6 PID: 1023 Comm: Xorg Not tainted 4.15.0-rc8 #1 > [ +0.000003] Hardware name: Micro-Star International Co., Ltd. GX780/GT780/MS-1761, BIOS E1761IMS V3.01 05/02/2011 > [ +0.000003] Call Trace: > [ +0.000009] dump_stack+0x9f/0xe1 > [ +0.000008] swiotlb_alloc_coherent+0xdf/0x150 > [ +0.000010] ttm_dma_pool_get_pages+0x1ec/0x4b0 > [ +0.000015] ttm_dma_populate+0x24c/0x340 > [ +0.000011] ttm_tt_bind+0x23/0x50 > [ +0.000006] ttm_bo_handle_move_mem+0x58c/0x5c0 > [ +0.000015] ttm_bo_validate+0x152/0x190 > [ +0.000004] ? ttm_bo_init_reser...

...wiotlb: coherent allocation failed, size=2097152 [ 1035.437023] CPU: 0 PID: 1842 Comm: gnome-shell Not tainted 4.16.0-0.rc2.git0.1.fc28.x86_64 #1 [ 1035.437024] Hardware name: NVIDIA MCP7A/MCP7A, BIOS 6.00 PG 04/22/2009 [ 1035.437025] Call Trace: [ 1035.437036] dump_stack+0x5c/0x85 [ 1035.437040] swiotlb_alloc_coherent+0x1c3/0x1e0 [ 1035.437052] ttm_dma_pool_get_pages+0x21a/0x620 [ttm] [ 1035.437057] ttm_dma_populate+0xdd/0x390 [ttm] [ 1035.437062] ttm_tt_bind+0x2e/0x60 [ttm] [ 1035.437067] ttm_bo_handle_move_mem+0x4cf/0x550 [ttm] [ 1035.437073] ttm_bo_validate+0x119/0x130 [ttm] [ 1035.437104] ? drm_get_edi...

...> size=2097152 > [ 1035.437023] CPU: 0 PID: 1842 Comm: gnome-shell Not tainted > 4.16.0-0.rc2.git0.1.fc28.x86_64 #1 > [ 1035.437024] Hardware name: NVIDIA MCP7A/MCP7A, BIOS 6.00 PG 04/22/2009 > [ 1035.437025] Call Trace: > [ 1035.437036] dump_stack+0x5c/0x85 > [ 1035.437040] swiotlb_alloc_coherent+0x1c3/0x1e0 > [ 1035.437052] ttm_dma_pool_get_pages+0x21a/0x620 [ttm] > [ 1035.437057] ttm_dma_populate+0xdd/0x390 [ttm] > [ 1035.437062] ttm_tt_bind+0x2e/0x60 [ttm] > [ 1035.437067] ttm_bo_handle_move_mem+0x4cf/0x550 [ttm] > [ 1035.437073] ttm_bo_validate+0x119/0x130 [ttm] >...

Hi all, Here''s a work in progress series whcih does a partial revert of the previous swiotlb changes, and does a partial replacement with Becky Bruce''s series. The most important difference is Becky''s use of phys_addr_t rather than page+offset to represent arbitrary pages. This turns out to be simpler. I didn''t replicate the map_single_page changes, since

Hi Ingo, Here''s the chunk of patches to add Xen Dom0 support (it''s probably worth creating a new xen/dom0 topic branch for it). A dom0 Xen domain is basically the same as a normal domU domain, but it has extra privileges to directly access hardware. There are two issues to deal with: - translating to and from the domain''s pseudo-physical addresses and real machine

Hi, This is third approach (complete this time) for replacing struct dma_attrs with unsigned long. The main patch (2/45) doing the change is split into many subpatches for easier review (3-43). They should be squashed together when applying. *Important:* Patchset is *only* build tested on allyesconfigs: ARM, ARM64, i386, x86_64 and powerpc. Please provide reviewes and tests for other