Re: [RESEND] [PATCH] powerpc: remove dead BIO_VMERGE_BOUNDARY definition

From: Jens Axboe
Date: Mon Dec 15 2008 - 02:29:07 EST


On Sun, Dec 14 2008, FUJITA Tomonori wrote:
> This is a resend of:
>
> http://marc.info/?l=linux-kernel&m=122482703616607&w=2
>
> =
> From: FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx>
> Subject: [PATCH] powerpc: remove dead BIO_VMERGE_BOUNDARY definition
>
> The block layer dropped the virtual merge feature
> (b8b3e16cfe6435d961f6aaebcfd52a1ff2a988c5). BIO_VMERGE_BOUNDARY
> definition is meaningless now (For POWER, BIO_VMERGE_BOUNDARY has been
> meaningless for a long time since POWER disables the virtual merge
> feature).
>
> Signed-off-by: FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx>

Acked-by: Jens Axboe <jens.axboe@xxxxxxxxxx>

> ---
> arch/powerpc/include/asm/io.h | 7 -------
> 1 files changed, 0 insertions(+), 7 deletions(-)
>
> diff --git a/arch/powerpc/include/asm/io.h b/arch/powerpc/include/asm/io.h
> index 08266d2..494cd8b 100644
> --- a/arch/powerpc/include/asm/io.h
> +++ b/arch/powerpc/include/asm/io.h
> @@ -713,13 +713,6 @@ static inline void * phys_to_virt(unsigned long address)
> */
> #define page_to_phys(page) ((phys_addr_t)page_to_pfn(page) << PAGE_SHIFT)
>
> -/* We do NOT want virtual merging, it would put too much pressure on
> - * our iommu allocator. Instead, we want drivers to be smart enough
> - * to coalesce sglists that happen to have been mapped in a contiguous
> - * way by the iommu
> - */
> -#define BIO_VMERGE_BOUNDARY 0
> -
> /*
> * 32 bits still uses virt_to_bus() for it's implementation of DMA
> * mappings se we have to keep it defined here. We also have some old
> --
> 1.5.5.GIT
>

--
Jens Axboe

--
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/