MIME-Version: 1.0
In-Reply-To: <7e39ef18-3e60-8cc9-ec4f-1cd02ade171f@zytor.com>
References: <cover.1505846562.git.jpoimboe@redhat.com> <31e96e6bcfcb47725e15a093b9c31660dfaad430.1505846562.git.jpoimboe@redhat.com>
 <eddc6128-9ae8-a5ea-0174-98dfadf7d07b@zytor.com> <CACT4Y+YrgBgKSR1ViWCuw2rxe9jrk3YNnGXXo1MNL+3rsFPaMw@mail.gmail.com>
 <7e39ef18-3e60-8cc9-ec4f-1cd02ade171f@zytor.com>
From: Dmitry Vyukov <dvyukov@google.com>
Date: Wed, 20 Sep 2017 20:01:02 +0200
Message-ID: <CACT4Y+b1EagSNhwsVkgt4sH8PHrLPO7Mp+Z+JNBWo+qxO+LMSg@mail.gmail.com>
Subject: Re: [PATCH 2/2] x86/asm: Fix inline asm call constraints for clang
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: Josh Poimboeuf <jpoimboe@redhat.com>,
        "x86@kernel.org" <x86@kernel.org>, LKML <linux-kernel@vger.kernel.org>,
        Ingo Molnar <mingo@kernel.org>, Thomas Gleixner <tglx@linutronix.de>,
        Andy Lutomirski <luto@kernel.org>,
        Linus Torvalds <torvalds@linux-foundation.org>,
        Alexander Potapenko <glider@google.com>,
        Matthias Kaehlcke <mka@chromium.org>, Arnd Bergmann <arnd@arndb.de>,
        Peter Zijlstra <peterz@infradead.org>,
        Andrey Ryabinin <aryabinin@virtuozzo.com>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 989
Lines: 20

On Wed, Sep 20, 2017 at 7:46 PM, H. Peter Anvin <hpa@zytor.com> wrote:
> On 09/20/17 10:38, Dmitry Vyukov wrote:
>>
>> I think we need just the frame itself and RSP pointing below this
>> frame. If we don't have a frame, CALL instruction will smash whatever
>> RSP happens to point to. Compiler doesn't have to setup RSP to point
>> below used part of stack in leaf functions.
>>
>
> In the kernel it does.  Redzoning is not allowed in the kernel, because
> interrupts or exceptions would also smash the redzone.

I see... But it's the same for user-space signals, the first thing a
signal should do is to skip the redzone. I guess interrupt handlers
should switch to interrupt stack which avoids smashing redzone
altogether. Do you mean nested interrupts/exceptions in interrupts?
In my experience frames in leaf functions can have pretty large
performance penalty. Wonder if we have we considered changing
interrupt/exception handlers to avoid smashing redzones and disable
leaf frames?