DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 9F8BD9D0F6
Date: Fri, 30 Jun 2017 10:55:00 -0500
From: Josh Poimboeuf <jpoimboe@redhat.com>
To: Andy Lutomirski <luto@kernel.org>
Cc: X86 ML <x86@kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        live-patching@vger.kernel.org,
        Linus Torvalds <torvalds@linux-foundation.org>,
        Jiri Slaby <jslaby@suse.cz>, Ingo Molnar <mingo@kernel.org>,
        "H. Peter Anvin" <hpa@zytor.com>,
        Peter Zijlstra <peterz@infradead.org>, Mike Galbraith <efault@gmx.de>
Subject: Re: [PATCH v2 6/8] x86/entry: add unwind hint annotations
Message-ID: <20170630155500.vnn5szd6smpn5nrz@treble>
References: <CALCETrWJeNXiEPPVecc2OGRENj220zNeFCYgm+Q2P7ZiMvoVFg@mail.gmail.com>
 <20170629190559.ttw52ahwtsjynayx@treble>
 <CALCETrUNhPgrQV9Sj+ZP3A_jSHbww4XPpBLJQ15OmH3nsHBsdg@mail.gmail.com>
 <20170629214134.c36krjhvzegwkfjk@treble>
 <F1E11F3A-039D-48B2-A57D-7881E93028DD@amacapital.net>
 <20170630021249.cqkszxaqtwakmzpg@treble>
 <CALCETrWBDOgfuX+uSukZ+_PHT4_j87LXY-DXPvCZ_KEXcnkWwg@mail.gmail.com>
 <CALCETrWEaR=hwQzfaVw6F8gtaTY_r1RCssK6YFnP7Z0ZP9ZcJg@mail.gmail.com>
 <20170630131148.yszdjakekrwqlf6k@treble>
 <CALCETrWgHQ93G8B8yyX2Ok_rBVJxfsm_2jPAcKS7iUfcHH3NFA@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <CALCETrWgHQ93G8B8yyX2Ok_rBVJxfsm_2jPAcKS7iUfcHH3NFA@mail.gmail.com>
User-Agent: Mutt/1.6.0.1 (2016-04-01)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1144
Lines: 38

On Fri, Jun 30, 2017 at 08:44:40AM -0700, Andy Lutomirski wrote:
> > A few comments about the first patch:
> >
> >   https://git.kernel.org/pub/scm/linux/kernel/git/luto/linux.git/commit/?h=x86/entry_irq_stack&id=3e2aa2102cc1c5e60d4a8637bff78d0478a55059
> >
> > - It uses a '693:' label instead of '.Lirqs_off_\@:'
> 
> Touché!
> 
> >
> > - There's a comment I don't follow:
> >
> >     "Anything that can interrupt us here without using IST must be
> >     *extremely* careful to limit its stack usage."
> >
> >   What specifically could interrupt there without using IST?
> 
> #DB, later on in the series.  I'll update the comment.
> 
> >
> > - Since do_softirq_own_stack() is a callable function, I think it still
> >   needs to save rbp.
> 
> Whoops.
> 
> >
> > - Why change the "jmp error_exit" to "ret" in
> >   xen_do_hypervisor_callback()?
> 
> To match the other change I made there.  I removed both.

One more thing I forgot to mention: if you could use r10 instead of r11,
that would be helpful because it means one less register undwarf needs
to know about.  (It already deals with r10 because of GCC stack
realignment).

-- 
Josh