All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Edgecombe, Rick P" <rick.p.edgecombe@intel.com>
To: "ripatel@wii.dev" <ripatel@wii.dev>
Cc: "kees@kernel.org" <kees@kernel.org>,
	"x86@kernel.org" <x86@kernel.org>,
	"dave.hansen@linux.intel.com" <dave.hansen@linux.intel.com>,
	"hpa@zytor.com" <hpa@zytor.com>,
	"shuah@kernel.org" <shuah@kernel.org>,
	"mingo@redhat.com" <mingo@redhat.com>,
	"bp@alien8.de" <bp@alien8.de>,
	"tglx@kernel.org" <tglx@kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-kselftest@vger.kernel.org"
	<linux-kselftest@vger.kernel.org>
Subject: Re: [PATCH 1/3] x86/shstk: ban ia32 sigreturn when shadow stack is enabled
Date: Fri, 9 Oct 2026 22:14:02 +0000	[thread overview]
Message-ID: <bb352ccd1939c6e01f1e71a12fc457b8051bc04a.camel@intel.com> (raw)
In-Reply-To: <asjRt49FfzdP4pxk@wii.dev>

On Fri, 2026-10-09 at 11:36 +0000, Richard Patel wrote:
> On Thu, Oct 08, 2026 at 11:11:41PM +0000, Edgecombe, Rick P wrote:
> > On Thu, 2026-10-08 at 22:47 +0000, Richard Patel wrote:
> > > > But today setting EIP to an arbitrary point is fairly easy. But even in a
> > > > future
> > > > case of IBT enabled, shadow stack would still need enhancements for the
> > > > normal
> > > > 64 bit runtime to prevent this. 
> > > 
> > > What do you think of creating a shadow stack frame on signal delivery
> > > and popping that on sigreturn? I suppose that would need siglongjmp
> > > modifications and probably break CRIU. :(
> > 
> > Yes I didn't know they did not parse the shadow stack signal frame
> > appropriately. That is unfortunate. But we can still evolve the shadow stack ABI
> > by adding new modes to the enable prctl if we want.
> 
> In the case of CRIU, they are not only parsing
> 

Ah so the parsing is right, but they don't add the extra frames?

> , but crafting their own
> frames on the shadow stack. I suppose we can keep the kernel's parser
> lenient so it works with older CRIU frames even after IBT enablement.

Yea we would need a backwards compatible frame. The real sigframes have the same
problems to solve.

  reply	other threads:[~2026-10-09 22:14 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-08 20:16 [PATCH 0/3] x86/shstk: ban ia32 sigreturn Richard Patel
2026-10-08 20:16 ` [PATCH 1/3] x86/shstk: ban ia32 sigreturn when shadow stack is enabled Richard Patel
2026-10-08 20:59   ` Edgecombe, Rick P
2026-10-08 21:50     ` Richard Patel
2026-10-08 22:34       ` Edgecombe, Rick P
2026-10-08 22:47         ` Richard Patel
2026-10-08 23:11           ` Edgecombe, Rick P
2026-10-08 23:29             ` Richard Patel
2026-10-09 11:36             ` Richard Patel
2026-10-09 22:14               ` Edgecombe, Rick P [this message]
2026-10-08 20:16 ` [PATCH 2/3] selftests/x86: test shadow stack sigreturn protection Richard Patel
2026-10-08 20:16 ` [PATCH 3/3] selftests/x86: skip shstk tests where perf_event_open() fails Richard Patel
2026-10-08 21:00 ` [PATCH 0/3] x86/shstk: ban ia32 sigreturn Edgecombe, Rick P

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bb352ccd1939c6e01f1e71a12fc457b8051bc04a.camel@intel.com \
    --to=rick.p.edgecombe@intel.com \
    --cc=bp@alien8.de \
    --cc=dave.hansen@linux.intel.com \
    --cc=hpa@zytor.com \
    --cc=kees@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=ripatel@wii.dev \
    --cc=shuah@kernel.org \
    --cc=tglx@kernel.org \
    --cc=x86@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.