[drm-next] drm/radeon: use IBs for VM page table updates

2013-02-01 Thread Christian König
Am 01.02.2013 15:04, schrieb Alex Deucher: > On Fri, Feb 1, 2013 at 5:25 AM, Christian K?nig > wrote: >> [SNAP] >> Wouldn't it make sense to use the IB functions (ib_get, ib_schedule, >> ib_free) here instead of filling the IB structure manually? > I tried that initially, but ran into several pro

[drm-next] drm/radeon: use IBs for VM page table updates

2013-02-01 Thread Christian König
Am 31.01.2013 22:37, schrieb alexdeucher at gmail.com: > From: Alex Deucher > > For very large page table updates, we can exceed the > size of the ring. To avoid this, use an IB to perform > the page table update. > > Signed-off-by: Alex Deucher > --- > > We may want to do something similar for

[drm-next] drm/radeon: use IBs for VM page table updates

2013-02-01 Thread Alex Deucher
On Fri, Feb 1, 2013 at 5:25 AM, Christian K?nig wrote: > Am 31.01.2013 22:37, schrieb alexdeucher at gmail.com: > >> From: Alex Deucher >> >> For very large page table updates, we can exceed the >> size of the ring. To avoid this, use an IB to perform >> the page table update. >> >> Signed-off-

Re: [drm-next] drm/radeon: use IBs for VM page table updates

2013-02-01 Thread Christian König
Am 01.02.2013 15:04, schrieb Alex Deucher: On Fri, Feb 1, 2013 at 5:25 AM, Christian König wrote: [SNAP] Wouldn't it make sense to use the IB functions (ib_get, ib_schedule, ib_free) here instead of filling the IB structure manually? I tried that initially, but ran into several problems and it

Re: [drm-next] drm/radeon: use IBs for VM page table updates

2013-02-01 Thread Alex Deucher
On Fri, Feb 1, 2013 at 5:25 AM, Christian König wrote: > Am 31.01.2013 22:37, schrieb alexdeuc...@gmail.com: > >> From: Alex Deucher >> >> For very large page table updates, we can exceed the >> size of the ring. To avoid this, use an IB to perform >> the page table update. >> >> Signed-off-by:

Re: [drm-next] drm/radeon: use IBs for VM page table updates

2013-02-01 Thread Christian König
Am 31.01.2013 22:37, schrieb alexdeuc...@gmail.com: From: Alex Deucher For very large page table updates, we can exceed the size of the ring. To avoid this, use an IB to perform the page table update. Signed-off-by: Alex Deucher --- We may want to do something similar for the BO move code a

[drm-next] drm/radeon: use IBs for VM page table updates

2013-01-31 Thread alexdeuc...@gmail.com
From: Alex Deucher For very large page table updates, we can exceed the size of the ring. To avoid this, use an IB to perform the page table update. Signed-off-by: Alex Deucher --- We may want to do something similar for the BO move code as we could potentially run out of ring space on a smal

[drm-next] drm/radeon: use IBs for VM page table updates

2013-01-31 Thread alexdeucher
From: Alex Deucher For very large page table updates, we can exceed the size of the ring. To avoid this, use an IB to perform the page table update. Signed-off-by: Alex Deucher --- We may want to do something similar for the BO move code as we could potentially run out of ring space on a smal