Re: [PATCH rebased] Use the new sg_page() helper

From: Linus Torvalds
Date: Tue Oct 23 2007 - 18:33:02 EST




On Tue, 23 Oct 2007, Emil Medve wrote:
>
> - sg->dma_address = page_to_phys(sg->page) + sg->offset;
> + sg->dma_address = page_to_phys(sg_page(sg)) + sg->offset;

This is "sg_phys(sg)"

> - consistent_sync(page_address(sg->page) + sg->offset,
> + consistent_sync(page_address(sg_page(sg)) + sg->offset,
> - consistent_sync(page_address(sg->page) + sg->offset,
> + consistent_sync(page_address(sg_page(sg)) + sg->offset,
> - consistent_sync(page_address(sg->page) + sg->offset,
> + consistent_sync(page_address(sg_page(sg)) + sg->offset,

Please write "page_address(sg_page(sg)) + sg->offset" as "sg_virt(sg)"
instead.

And send the patch to Jens, since he's the person who is in charge of
getting this all sorted out ;)

Ok?

Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/