[PATCH] parisc: remove dead BIO_VMERGE_BOUNDARY and BIO_VMERGE_MAX_SIZE definitions - Kernel

This is a discussion on [PATCH] parisc: remove dead BIO_VMERGE_BOUNDARY and BIO_VMERGE_MAX_SIZE definitions - Kernel ; The block layer dropped the virtual merge feature (b8b3e16cfe6435d961f6aaebcfd52a1ff2a988c5). BIO_VMERGE_BOUNDARY and BIO_VMERGE_MAX_SIZE definitions are meaningless now. Signed-off-by: FUJITA Tomonori --- arch/parisc/include/asm/io.h | 6 ------ arch/parisc/kernel/setup.c | 5 ----- drivers/parisc/ccio-dma.c | 2 -- drivers/parisc/sba_iommu.c | 2 -- 4 files changed, 0 ...

+ Reply to Thread
Results 1 to 2 of 2

Thread: [PATCH] parisc: remove dead BIO_VMERGE_BOUNDARY and BIO_VMERGE_MAX_SIZE definitions

  1. [PATCH] parisc: remove dead BIO_VMERGE_BOUNDARY and BIO_VMERGE_MAX_SIZE definitions

    The block layer dropped the virtual merge feature
    (b8b3e16cfe6435d961f6aaebcfd52a1ff2a988c5). BIO_VMERGE_BOUNDARY and
    BIO_VMERGE_MAX_SIZE definitions are meaningless now.

    Signed-off-by: FUJITA Tomonori
    ---
    arch/parisc/include/asm/io.h | 6 ------
    arch/parisc/kernel/setup.c | 5 -----
    drivers/parisc/ccio-dma.c | 2 --
    drivers/parisc/sba_iommu.c | 2 --
    4 files changed, 0 insertions(+), 15 deletions(-)

    diff --git a/arch/parisc/include/asm/io.h b/arch/parisc/include/asm/io.h
    index 55ddb18..b92efb3 100644
    --- a/arch/parisc/include/asm/io.h
    +++ b/arch/parisc/include/asm/io.h
    @@ -4,12 +4,6 @@
    #include
    #include

    -extern unsigned long parisc_vmerge_boundary;
    -extern unsigned long parisc_vmerge_max_size;
    -
    -#define BIO_VMERGE_BOUNDARY parisc_vmerge_boundary
    -#define BIO_VMERGE_MAX_SIZE parisc_vmerge_max_size
    -
    #define virt_to_phys(a) ((unsigned long)__pa(a))
    #define phys_to_virt(a) __va(a)
    #define virt_to_bus virt_to_phys
    diff --git a/arch/parisc/kernel/setup.c b/arch/parisc/kernel/setup.c
    index 7d27853..8d8b024 100644
    --- a/arch/parisc/kernel/setup.c
    +++ b/arch/parisc/kernel/setup.c
    @@ -58,11 +58,6 @@ int parisc_bus_is_phys __read_mostly = 1; /* Assume no IOMMU is present */
    EXPORT_SYMBOL(parisc_bus_is_phys);
    #endif

    -/* This sets the vmerge boundary and size, it's here because it has to
    - * be available on all platforms (zero means no-virtual merging) */
    -unsigned long parisc_vmerge_boundary = 0;
    -unsigned long parisc_vmerge_max_size = 0;
    -
    void __init setup_cmdline(char **cmdline_p)
    {
    extern unsigned int boot_args[];
    diff --git a/drivers/parisc/ccio-dma.c b/drivers/parisc/ccio-dma.c
    index dcc1e99..62cc21d 100644
    --- a/drivers/parisc/ccio-dma.c
    +++ b/drivers/parisc/ccio-dma.c
    @@ -1578,8 +1578,6 @@ static int __init ccio_probe(struct parisc_device *dev)

    ioc_count++;

    - parisc_vmerge_boundary = IOVP_SIZE;
    - parisc_vmerge_max_size = BITS_PER_LONG * IOVP_SIZE;
    parisc_has_iommu();
    return 0;
    }
    diff --git a/drivers/parisc/sba_iommu.c b/drivers/parisc/sba_iommu.c
    index bc73b96..bdc9a0a 100644
    --- a/drivers/parisc/sba_iommu.c
    +++ b/drivers/parisc/sba_iommu.c
    @@ -1979,8 +1979,6 @@ sba_driver_callback(struct parisc_device *dev)
    proc_create("sba_iommu-bitmap", 0, root, &sba_proc_bitmap_fops);
    #endif

    - parisc_vmerge_boundary = IOVP_SIZE;
    - parisc_vmerge_max_size = IOVP_SIZE * BITS_PER_LONG;
    parisc_has_iommu();
    return 0;
    }
    --
    1.5.4.2

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

  2. Re: [PATCH] parisc: remove dead BIO_VMERGE_BOUNDARY and BIO_VMERGE_MAX_SIZE definitions

    On Fri, Oct 24, 2008 at 02:41:01PM +0900, FUJITA Tomonori wrote:
    > The block layer dropped the virtual merge feature
    > (b8b3e16cfe6435d961f6aaebcfd52a1ff2a988c5). BIO_VMERGE_BOUNDARY and
    > BIO_VMERGE_MAX_SIZE definitions are meaningless now.


    Sweet!
    I've never liked the fact that the block layer did this but recognize
    it was useful to maximize the number of SG entries used for
    each IO sent to the storage controller.

    > Signed-off-by: FUJITA Tomonori


    Acked-by: Grant Grundler

    thanks,
    grant

    > ---
    > arch/parisc/include/asm/io.h | 6 ------
    > arch/parisc/kernel/setup.c | 5 -----
    > drivers/parisc/ccio-dma.c | 2 --
    > drivers/parisc/sba_iommu.c | 2 --
    > 4 files changed, 0 insertions(+), 15 deletions(-)
    >
    > diff --git a/arch/parisc/include/asm/io.h b/arch/parisc/include/asm/io.h
    > index 55ddb18..b92efb3 100644
    > --- a/arch/parisc/include/asm/io.h
    > +++ b/arch/parisc/include/asm/io.h
    > @@ -4,12 +4,6 @@
    > #include
    > #include
    >
    > -extern unsigned long parisc_vmerge_boundary;
    > -extern unsigned long parisc_vmerge_max_size;
    > -
    > -#define BIO_VMERGE_BOUNDARY parisc_vmerge_boundary
    > -#define BIO_VMERGE_MAX_SIZE parisc_vmerge_max_size
    > -
    > #define virt_to_phys(a) ((unsigned long)__pa(a))
    > #define phys_to_virt(a) __va(a)
    > #define virt_to_bus virt_to_phys
    > diff --git a/arch/parisc/kernel/setup.c b/arch/parisc/kernel/setup.c
    > index 7d27853..8d8b024 100644
    > --- a/arch/parisc/kernel/setup.c
    > +++ b/arch/parisc/kernel/setup.c
    > @@ -58,11 +58,6 @@ int parisc_bus_is_phys __read_mostly = 1; /* Assume no IOMMU is present */
    > EXPORT_SYMBOL(parisc_bus_is_phys);
    > #endif
    >
    > -/* This sets the vmerge boundary and size, it's here because it has to
    > - * be available on all platforms (zero means no-virtual merging) */
    > -unsigned long parisc_vmerge_boundary = 0;
    > -unsigned long parisc_vmerge_max_size = 0;
    > -
    > void __init setup_cmdline(char **cmdline_p)
    > {
    > extern unsigned int boot_args[];
    > diff --git a/drivers/parisc/ccio-dma.c b/drivers/parisc/ccio-dma.c
    > index dcc1e99..62cc21d 100644
    > --- a/drivers/parisc/ccio-dma.c
    > +++ b/drivers/parisc/ccio-dma.c
    > @@ -1578,8 +1578,6 @@ static int __init ccio_probe(struct parisc_device *dev)
    >
    > ioc_count++;
    >
    > - parisc_vmerge_boundary = IOVP_SIZE;
    > - parisc_vmerge_max_size = BITS_PER_LONG * IOVP_SIZE;
    > parisc_has_iommu();
    > return 0;
    > }
    > diff --git a/drivers/parisc/sba_iommu.c b/drivers/parisc/sba_iommu.c
    > index bc73b96..bdc9a0a 100644
    > --- a/drivers/parisc/sba_iommu.c
    > +++ b/drivers/parisc/sba_iommu.c
    > @@ -1979,8 +1979,6 @@ sba_driver_callback(struct parisc_device *dev)
    > proc_create("sba_iommu-bitmap", 0, root, &sba_proc_bitmap_fops);
    > #endif
    >
    > - parisc_vmerge_boundary = IOVP_SIZE;
    > - parisc_vmerge_max_size = IOVP_SIZE * BITS_PER_LONG;
    > parisc_has_iommu();
    > return 0;
    > }
    > --
    > 1.5.4.2

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

+ Reply to Thread