lkml.org 
[lkml]   [2003]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    SubjectRe: Possible shared mapping bug in 2.4.23 (at least MIPS/Sparc)
    Linus Torvalds wrote:
    > > D-cache incoherence with unsuitably aligned multiple MAP_FIXED
    > > mappings is also observed on SH4, SH5, PA-RISC 1.1d. The kernel may
    > > have the same behaviour on those platforms: allowing a mapping that
    > > should not be allowed.
    >
    > Why?
    >
    > If the user asks for it, it's the users own damn fault. Nobody guarantees
    > cache coherency to users who require fixed addresses.

    I agree, the users asks for it and in rare cases it's even useful.

    If that's your view, then several architectures have code in
    arch_get_unmapped_base which tests MAP_FIXED - but that function isn't
    called in the MAP_FIXED case any more. The attached patch removes
    those branches. Trivial, but untested.

    (It would be nice to be able to query the kernel to learn what mapping
    alignment, if any, can be used to avoid aliasing problems though.
    Something like "SHMLBA: 16k" or "SHMLBA: none" in /proc/cpuinfo and
    whatever other interfaces present CPU info. On some architectures the
    value depends on the CPU model, determined at run time. On some
    architectures, there isn't a value).

    -- Jamie

    Patch: irq_idle-2.6.0-test4-01jl
    Summary: Remove unused MAP_FIXED branches from arch_get_unmapped_area

    Several architectures have code in arch_get_unmapped_base which tests
    MAP_FIXED - but that function isn't called in the MAP_FIXED case any more.

    For example, this code in arch/sparc/kernel/sys_sparc.c is quite typical:

    if (flags & MAP_FIXED) {
    /* We do not accept a shared mapping if it would violate
    * cache aliasing constraints.
    */
    if ((flags & MAP_SHARED) && (addr & (SHMLBA - 1)))
    return -EINVAL;
    return addr;
    }

    The function containing that test isn't called with MAP_FIXED these
    days, so it's redundant and misleading. This patch removes those tests
    from all the arch_get_unmapped_area functions which have them.


    diff -urN --exclude-from=dontdiff orig-2.6.0-test11/arch/mips/kernel/syscall.c mapalign-2.6.0-test11/arch/mips/kernel/syscall.c
    --- orig-2.6.0-test11/arch/mips/kernel/syscall.c 2003-09-02 20:49:13.000000000 +0100
    +++ mapalign-2.6.0-test11/arch/mips/kernel/syscall.c 2003-12-14 04:08:10.000000000 +0000
    @@ -63,16 +63,6 @@
    struct vm_area_struct * vmm;
    int do_color_align;

    - if (flags & MAP_FIXED) {
    - /*
    - * We do not accept a shared mapping if it would violate
    - * cache aliasing constraints.
    - */
    - if ((flags & MAP_SHARED) && (addr & shm_align_mask))
    - return -EINVAL;
    - return addr;
    - }
    -
    if (len > TASK_SIZE)
    return -ENOMEM;
    do_color_align = 0;
    diff -urN --exclude-from=dontdiff orig-2.6.0-test11/arch/sh/kernel/sys_sh.c mapalign-2.6.0-test11/arch/sh/kernel/sys_sh.c
    --- orig-2.6.0-test11/arch/sh/kernel/sys_sh.c 2003-07-08 21:55:03.000000000 +0100
    +++ mapalign-2.6.0-test11/arch/sh/kernel/sys_sh.c 2003-12-14 04:07:20.000000000 +0000
    @@ -54,15 +54,6 @@
    {
    struct vm_area_struct *vma;

    - if (flags & MAP_FIXED) {
    - /* We do not accept a shared mapping if it would violate
    - * cache aliasing constraints.
    - */
    - if ((flags & MAP_SHARED) && (addr & (SHMLBA - 1)))
    - return -EINVAL;
    - return addr;
    - }
    -
    if (len > TASK_SIZE)
    return -ENOMEM;
    if (!addr)
    diff -urN --exclude-from=dontdiff orig-2.6.0-test11/arch/sparc/kernel/sys_sparc.c mapalign-2.6.0-test11/arch/sparc/kernel/sys_sparc.c
    --- orig-2.6.0-test11/arch/sparc/kernel/sys_sparc.c 2003-08-27 22:55:57.000000000 +0100
    +++ mapalign-2.6.0-test11/arch/sparc/kernel/sys_sparc.c 2003-12-14 04:06:33.000000000 +0000
    @@ -40,15 +40,6 @@
    {
    struct vm_area_struct * vmm;

    - if (flags & MAP_FIXED) {
    - /* We do not accept a shared mapping if it would violate
    - * cache aliasing constraints.
    - */
    - if ((flags & MAP_SHARED) && (addr & (SHMLBA - 1)))
    - return -EINVAL;
    - return addr;
    - }
    -
    /* See asm-sparc/uaccess.h */
    if (len > TASK_SIZE - PAGE_SIZE)
    return -ENOMEM;
    diff -urN --exclude-from=dontdiff orig-2.6.0-test11/arch/sparc64/kernel/sys_sparc.c mapalign-2.6.0-test11/arch/sparc64/kernel/sys_sparc.c
    --- orig-2.6.0-test11/arch/sparc64/kernel/sys_sparc.c 2003-08-27 22:55:57.000000000 +0100
    +++ mapalign-2.6.0-test11/arch/sparc64/kernel/sys_sparc.c 2003-12-14 04:06:46.000000000 +0000
    @@ -52,15 +52,6 @@
    unsigned long start_addr;
    int do_color_align;

    - if (flags & MAP_FIXED) {
    - /* We do not accept a shared mapping if it would violate
    - * cache aliasing constraints.
    - */
    - if ((flags & MAP_SHARED) && (addr & (SHMLBA - 1)))
    - return -EINVAL;
    - return addr;
    - }
    -
    if (test_thread_flag(TIF_32BIT))
    task_size = 0xf0000000UL;
    if (len > task_size || len > -PAGE_OFFSET)
    -
    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/

    \
     
     \ /
      Last update: 2005-03-22 13:59    [W:0.054 / U:244.012 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site