RE: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-14 Thread Deucher, Alexander
ave Airlie; Dmitry > Cherkasov > Subject: Re: [PATCH] Add 2-level GPUVM pagetables support to radeon > driver. > > On 13.09.2012 20:42, Jerome Glisse wrote: > > On Thu, Sep 13, 2012 at 2:37 PM, Alex Deucher > wrote: > >> On Thu, Sep 13, 2012 at 2:17 PM, Jerome Glis

Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-14 Thread Christian König
On 13.09.2012 20:42, Jerome Glisse wrote: On Thu, Sep 13, 2012 at 2:37 PM, Alex Deucher wrote: On Thu, Sep 13, 2012 at 2:17 PM, Jerome Glisse wrote: On Thu, Sep 13, 2012 at 10:13 AM, Dmitry Cherkasov wrote: PDE/PTE update code uses CP ring for memory writes. All page table entries are

Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-14 Thread Christian König
On 13.09.2012 20:42, Jerome Glisse wrote: On Thu, Sep 13, 2012 at 2:37 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Thu, Sep 13, 2012 at 2:17 PM, Jerome Glisse j.gli...@gmail.com wrote: On Thu, Sep 13, 2012 at 10:13 AM, Dmitry Cherkasov dcherkas...@gmail.com wrote: PDE/PTE update code

RE: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-14 Thread Deucher, Alexander
Cherkasov Subject: Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver. On 13.09.2012 20:42, Jerome Glisse wrote: On Thu, Sep 13, 2012 at 2:37 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Thu, Sep 13, 2012 at 2:17 PM, Jerome Glisse j.gli...@gmail.com wrote: On Thu, Sep 13

Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-13 Thread Jerome Glisse
On Thu, Sep 13, 2012 at 2:37 PM, Alex Deucher wrote: > On Thu, Sep 13, 2012 at 2:17 PM, Jerome Glisse wrote: >> On Thu, Sep 13, 2012 at 10:13 AM, Dmitry Cherkasov >> wrote: >>> PDE/PTE update code uses CP ring for memory writes. >>> All page table entries are preallocated for now in alloc_pt().

Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-13 Thread Alex Deucher
On Thu, Sep 13, 2012 at 2:17 PM, Jerome Glisse wrote: > On Thu, Sep 13, 2012 at 10:13 AM, Dmitry Cherkasov > wrote: >> PDE/PTE update code uses CP ring for memory writes. >> All page table entries are preallocated for now in alloc_pt(). >> >> It is made as whole because it's hard to divide it to

Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-13 Thread Jerome Glisse
On Thu, Sep 13, 2012 at 10:13 AM, Dmitry Cherkasov wrote: > PDE/PTE update code uses CP ring for memory writes. > All page table entries are preallocated for now in alloc_pt(). > > It is made as whole because it's hard to divide it to several patches > that compile and doesn't break anything

Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-13 Thread Dmitry Cherkassov
Christian, > as I already noted in our internal discussion, the first step to > hierarchical page table support should be to cleanup the set_page interface. > Please see the attached patch, it does exactly this. I suggest that you > rebase on it and try to don't touch the chipset specific code in

Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-13 Thread Christian König
Hi Dmitry, as I already noted in our internal discussion, the first step to hierarchical page table support should be to cleanup the set_page interface. Please see the attached patch, it does exactly this. I suggest that you rebase on it and try to don't touch the chipset specific code in

Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-13 Thread Christian König
Hi Dmitry, as I already noted in our internal discussion, the first step to hierarchical page table support should be to cleanup the set_page interface. Please see the attached patch, it does exactly this. I suggest that you rebase on it and try to don't touch the chipset specific code in

Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-13 Thread Dmitry Cherkassov
Christian, as I already noted in our internal discussion, the first step to hierarchical page table support should be to cleanup the set_page interface. Please see the attached patch, it does exactly this. I suggest that you rebase on it and try to don't touch the chipset specific code in

Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-13 Thread Jerome Glisse
On Thu, Sep 13, 2012 at 10:13 AM, Dmitry Cherkasov dcherkas...@gmail.com wrote: PDE/PTE update code uses CP ring for memory writes. All page table entries are preallocated for now in alloc_pt(). It is made as whole because it's hard to divide it to several patches that compile and doesn't

Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-13 Thread Alex Deucher
On Thu, Sep 13, 2012 at 2:17 PM, Jerome Glisse j.gli...@gmail.com wrote: On Thu, Sep 13, 2012 at 10:13 AM, Dmitry Cherkasov dcherkas...@gmail.com wrote: PDE/PTE update code uses CP ring for memory writes. All page table entries are preallocated for now in alloc_pt(). It is made as whole

Re: [PATCH] Add 2-level GPUVM pagetables support to radeon driver.

2012-09-13 Thread Jerome Glisse
On Thu, Sep 13, 2012 at 2:37 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Thu, Sep 13, 2012 at 2:17 PM, Jerome Glisse j.gli...@gmail.com wrote: On Thu, Sep 13, 2012 at 10:13 AM, Dmitry Cherkasov dcherkas...@gmail.com wrote: PDE/PTE update code uses CP ring for memory writes. All page