Re: [PATCH] ion: scatterlist offset not used for buffer map

2016-04-11 Thread John Einar Reitan
> I don't think this is right. A compound_page still has a page struct > for every page, you should be passing the page struct where your data > starts. Using an offset > PAGE_SIZE is going to break lots of places, > for example anywhere that uses kmap(sg_page(sg)). Was kind-of expecting that re

Re: [PATCH] ion: scatterlist offset not used for buffer map

2016-04-07 Thread John Einar Reitan
On Thu, Apr 07, 2016 at 12:37:50PM -0700, Laura Abbott wrote: > On 04/07/2016 04:29 AM, John Einar Reitan wrote: > > ion's default user/kernel page mapping code don't honor the offset > > option for scatterlists. It uses sg_page and expect the whole page to be > >

[PATCH] ion: scatterlist offset not used for buffer map

2016-04-07 Thread John Einar Reitan
ned-off-by: John Einar Reitan --- drivers/staging/android/ion/ion_heap.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/staging/android/ion/ion_heap.c b/drivers/staging/android/ion/ion_heap.c index ca15a87..e83002dc 100644 --- a/drivers/staging/android/ion/