• Nicholas Piggin's avatar
    powerpc/pseries: Remove unused hcall tracing instruction · 61d7ebe0
    Nicholas Piggin authored
    When JUMP_LABEL=n, the tracepoint refcount test in the pre-call stores
    the refcount value to the stack, so the same value can be used for the
    post-call (presumably to avoid racing with the value concurrently
    changing).
    
    On little-endian (ELFv2) that might have just worked by luck, because
    32(r1) is STK_PARAM(R3) there and so the value save gets clobbered by
    the tracing code when it's non-zero, but fortunately r3 is the hcall
    number and 0 is an invalid hcall number so it should get clobbered by
    another non-zero value. In any case, commit cc1adb5f
    ("powerpc/pseries: Use jump labels for hcall tracepoints") removed the
    code that actually used the value stored, so now it's just dead code.
    
    It's fragile to be storing to the stack like this, and confusing. Better
    remove it.
    Signed-off-by: default avatarNicholas Piggin <npiggin@gmail.com>
    Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
    Link: https://msgid.link/20230509091600.70994-2-npiggin@gmail.com
    61d7ebe0
hvCall.S 6.86 KB