[RFC][PATCH] drm/prime: fixed to allocate sg table considering contiguous pages
김승우
sw0312.kim at samsung.com
Sun Apr 28 19:16:42 PDT 2013
Hello Rahul,
Thanks for notifying.
As your comment, it is same patch with yours, so just ignore my patch.
Besg Regards,
- Seung-Woo Kim
On 2013년 04월 26일 18:14, Rahul Sharma wrote:
> Hi Seung Woo,
>
> I had posted the same solution at
> http://lists.freedesktop.org/archives/dri-devel/2013-January/034119.html.
> This has been pulled in drm-intel-next.
>
> regards,
> Rahul Sharma.
>
>
>
> On Fri, Apr 26, 2013 at 2:18 PM, Seung-Woo Kim <sw0312.kim at samsung.com>wrote:
>
>> Allocating scatter table with sg_alloc_table() does not consider
>> contiguous pages. Because sg_alloc_table_from_pages() merges
>> contigous pages into a signle scatter entry, this patch fixes to
>> allocate scatter table with it from drm_prime_pages_to_sg().
>>
>> Signed-off-by: Seung-Woo Kim <sw0312.kim at samsung.com>
>> ---
>> drivers/gpu/drm/drm_prime.c | 8 ++------
>> 1 files changed, 2 insertions(+), 6 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/drm_prime.c b/drivers/gpu/drm/drm_prime.c
>> index 366910d..8c098a3 100644
>> --- a/drivers/gpu/drm/drm_prime.c
>> +++ b/drivers/gpu/drm/drm_prime.c
>> @@ -401,21 +401,17 @@ int drm_prime_fd_to_handle_ioctl(struct drm_device
>> *dev, void *data,
>> struct sg_table *drm_prime_pages_to_sg(struct page **pages, int nr_pages)
>> {
>> struct sg_table *sg = NULL;
>> - struct scatterlist *iter;
>> - int i;
>> int ret;
>>
>> sg = kmalloc(sizeof(struct sg_table), GFP_KERNEL);
>> if (!sg)
>> goto out;
>>
>> - ret = sg_alloc_table(sg, nr_pages, GFP_KERNEL);
>> + ret = sg_alloc_table_from_pages(sg, pages, nr_pages,
>> + 0, PAGE_SIZE * nr_pages, GFP_KERNEL);
>> if (ret)
>> goto out;
>>
>> - for_each_sg(sg->sgl, iter, nr_pages, i)
>> - sg_set_page(iter, pages[i], PAGE_SIZE, 0);
>> -
>> return sg;
>> out:
>> kfree(sg);
>> --
>> 1.7.4.1
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>>
>
>
>
>
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
--
Seung-Woo Kim
Samsung Software R&D Center
--
More information about the dri-devel
mailing list