lkml.org 
[lkml]   [2014]   [Mar]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH v7 2/2] ARM hibernation / suspend-to-disk
    Date
    Sorry to those receiving repeated emails, I'm having an issue with some
    of the special characters names.

    Quoting Sebastian Capella (2014-03-05 02:50:05)
    > From: Russ Dill <Russ.Dill@ti.com>
    > Enable hibernation for ARM architectures and provide ARM
    > architecture specific calls used during hibernation.
    ...
    Quoting Stephen Boyd (2014-02-27 18:19:49)
    > On 02/27/14 17:47, Russ Dill wrote:
    > > On 02/27/2014 04:09 PM, Stephen Boyd wrote:
    > >> On 02/27/14 15:57, Sebastian Capella wrote:
    > >>> diff --git a/arch/arm/include/asm/memory.h
    > >>> b/arch/arm/include/asm/memory.h index 8756e4b..1079ea8
    > >>> 100644 ---
    > >>> a/arch/arm/include/asm/memory.h +++
    > >>> b/arch/arm/include/asm/memory.h @@ -291,6 +291,7 @@ static
    > >>> inline
    > >>> void *phys_to_virt(phys_addr_t x) */ #define __pa(x)
    > >>> __virt_to_phys((unsigned long)(x)) #define __va(x)
    > >>> ((void
    > >>> *)__phys_to_virt((phys_addr_t)(x)))
    > >>> +#define __pa_symbol(x)
    > >>> __pa(RELOC_HIDE((unsigned long)(x), 0))
    > >> Just curious, is there a reason for the RELOC_HIDE() here?
    > >> Or
    > >> __pa_symbol() for that matter? It looks like only x86 uses
    > >> this on
    > >> the __nosave_{begin,end} symbol. Maybe it's copy-pasta?
    > > From my understanding this needs to stick around so long as
    > > gcc 3.x is
    > > supported (did it get dropped yet?) on ARM Linux since it
    > > doesn't
    > > support -fno-strict-overflow.
    >
    > I don't think it's been dropped yet but I wonder if anyone has
    > tried
    > recent kernels with such a compiler?

    Hi Stephen,

    I was wondering if you had any objections or additional comments on
    the current patch.

    I've basically dropped the RELOC_HIDE. It hasn't proven to be an issue
    with the current compiler, and I was unable to revert back to 3.x
    compiler on the cortex-v7 platforms I'm using.

    Since this is being used on Hibernation, we would not be breaking any
    existing functionality in any event, and it sounds like this is unlikely
    to be an issue.

    I spoke informally to a friend who works on toolchains at Linaro and
    he suggested this may be related to the the strict-aliasing feature
    which appears to be disabled by default in the kernel
    (-fno-strict-aliasing) and generally not well liked.

    Unless someone objects, I'll plan to add this to Russell's patch system.

    Thanks,

    Sebastian



    \
     
     \ /
      Last update: 2014-03-07 06:01    [W:5.257 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site