[PATCH 2/2] drm/radeon: implement dynamic PTs allocation via SA

2012-09-27 Thread Dmitry Cherkassov
Hi Michel. 2012/9/27 Michel D?nzer : > On Die, 2012-09-25 at 18:44 +0400, Dmitry Cherkasov wrote: >> make dynamic allocation of page tables on demand in radeon_vm_update_pte >> >> Signed-off-by: Dmitry Cherkasov > > This change caused many piglit tests to fail for me on SI with > > radeon

[PATCH 2/2] drm/radeon: implement dynamic PTs allocation via SA

2012-09-27 Thread Michel Dänzer
On Don, 2012-09-27 at 16:18 +0400, Dmitry Cherkassov wrote: > Hi Michel. > > 2012/9/27 Michel D?nzer : > > On Die, 2012-09-25 at 18:44 +0400, Dmitry Cherkasov wrote: > >> make dynamic allocation of page tables on demand in radeon_vm_update_pte > >> > >> Signed-off-by: Dmitry Cherkasov > > > > Th

[PATCH 2/2] drm/radeon: implement dynamic PTs allocation via SA

2012-09-27 Thread Michel Dänzer
On Die, 2012-09-25 at 18:44 +0400, Dmitry Cherkasov wrote: > make dynamic allocation of page tables on demand in radeon_vm_update_pte > > Signed-off-by: Dmitry Cherkasov This change caused many piglit tests to fail for me on SI with radeon: The kernel rejected CS, see dmesg for more in

Re: [PATCH 2/2] drm/radeon: implement dynamic PTs allocation via SA

2012-09-27 Thread Dmitry Cherkassov
Hi Michel. 2012/9/27 Michel Dänzer : > On Die, 2012-09-25 at 18:44 +0400, Dmitry Cherkasov wrote: >> make dynamic allocation of page tables on demand in radeon_vm_update_pte >> >> Signed-off-by: Dmitry Cherkasov > > This change caused many piglit tests to fail for me on SI with > > radeon

Re: [PATCH 2/2] drm/radeon: implement dynamic PTs allocation via SA

2012-09-27 Thread Michel Dänzer
On Don, 2012-09-27 at 16:18 +0400, Dmitry Cherkassov wrote: > Hi Michel. > > 2012/9/27 Michel Dänzer : > > On Die, 2012-09-25 at 18:44 +0400, Dmitry Cherkasov wrote: > >> make dynamic allocation of page tables on demand in radeon_vm_update_pte > >> > >> Signed-off-by: Dmitry Cherkasov > > > > Th

Re: [PATCH 2/2] drm/radeon: implement dynamic PTs allocation via SA

2012-09-27 Thread Michel Dänzer
On Die, 2012-09-25 at 18:44 +0400, Dmitry Cherkasov wrote: > make dynamic allocation of page tables on demand in radeon_vm_update_pte > > Signed-off-by: Dmitry Cherkasov This change caused many piglit tests to fail for me on SI with radeon: The kernel rejected CS, see dmesg for more in

Re: [PATCH 2/2] drm/radeon: implement dynamic PTs allocation via SA

2012-09-26 Thread Dmitry Cherkassov
I've found introduced error in radeon_vm_free_pt, so will fix in next version of this patch. This one: --- a/drivers/gpu/drm/radeon/radeon_gart.c +++ b/drivers/gpu/drm/radeon/radeon_gart.c @@ -505,15 +505,6 @@ static void radeon_vm_free_pt(struct radeon_device *rdev, int driver_table_ent

[PATCH 2/2] drm/radeon: implement dynamic PTs allocation via SA

2012-09-26 Thread Dmitry Cherkasov
make dynamic allocation of page tables on demand in radeon_vm_update_pte Signed-off-by: Dmitry Cherkasov --- drivers/gpu/drm/radeon/radeon.h | 12 drivers/gpu/drm/radeon/radeon_gart.c | 106 ++ 2 files changed, 107 insertions(+), 11 deletions(-) diff

[PATCH 2/2] drm/radeon: implement dynamic PTs allocation via SA

2012-09-25 Thread Dmitry Cherkassov
I've found introduced error in radeon_vm_free_pt, so will fix in next version of this patch. This one: --- a/drivers/gpu/drm/radeon/radeon_gart.c +++ b/drivers/gpu/drm/radeon/radeon_gart.c @@ -505,15 +505,6 @@ static void radeon_vm_free_pt(struct radeon_device *rdev, int driver_table_ent

[PATCH 2/2] drm/radeon: implement dynamic PTs allocation via SA

2012-09-25 Thread Dmitry Cherkasov
make dynamic allocation of page tables on demand in radeon_vm_update_pte Signed-off-by: Dmitry Cherkasov --- drivers/gpu/drm/radeon/radeon.h | 12 drivers/gpu/drm/radeon/radeon_gart.c | 106 ++ 2 files changed, 107 insertions(+), 11 deletions(-) diff

[PATCH 2/2] drm/radeon: implement dynamic PTs allocation via SA

2012-09-25 Thread Christian König
The general idea looks good on first glance, but see below for further comments. On 25.09.2012 16:44, Dmitry Cherkasov wrote: > make dynamic allocation of page tables on demand in radeon_vm_update_pte > > Signed-off-by: Dmitry Cherkasov > --- > drivers/gpu/drm/radeon/radeon.h | 12

Re: [PATCH 2/2] drm/radeon: implement dynamic PTs allocation via SA

2012-09-25 Thread Christian König
The general idea looks good on first glance, but see below for further comments. On 25.09.2012 16:44, Dmitry Cherkasov wrote: make dynamic allocation of page tables on demand in radeon_vm_update_pte Signed-off-by: Dmitry Cherkasov --- drivers/gpu/drm/radeon/radeon.h | 12 drive