All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Yatsina, Marina" <marina.yatsina@intel.com>
To: Greg Kroah-Hartman <gregkh@linux-foundation.org>,
	Chandler Carruth <chandlerc@google.com>
Cc: Kees Cook <keescook@google.com>,
	David Woodhouse <dwmw2@infradead.org>,
	"Kreitzer, David L" <david.l.kreitzer@intel.com>,
	"Grischenko, Andrei L" <andrei.l.grischenko@intel.com>,
	"rnk@google.com" <rnk@google.com>,
	"LLVM Developers" <llvm-dev@lists.llvm.org>,
	"ehsan@mozilla.com" <ehsan@mozilla.com>,
	"Tayree, Coby" <coby.tayree@intel.com>,
	Matthias Braun <matze@braunis.de>,
	Dean Michael Berris <dean.berris@gmail.com>,
	"James Y Knight" <jyknight@google.com>,
	Guenter Roeck <linux@roeck-us.net>, X86 ML <x86@kernel.org>,
	LKML <linux-kernel@vger.kernel.org>,
	Alan Cox <gnomes@lxorguk.ukuu.org.uk>,
	Rik van Riel <riel@redhat.com>, Andi Kleen <ak@linux.intel.com>,
	Josh Poimboeuf <jpoimboe@redhat.com>,
	Tom Lendacky <thomas.lendacky@amd.com>,
	Peter Zijlstra <peterz@infradead.org>,
	"Linus Torvalds" <torvalds@linux-foundation.org>,
	Jiri Kosina <jikos@kernel.org>,
	Andy Lutomirski <luto@amacapital.net>,
	"Hansen, Dave" <dave.hansen@intel.com>,
	Tim Chen <tim.c.chen@linux.intel.com>,
	Paul Turner <pjt@google.com>, Stephen Hines <srhines@google.com>,
	Nick Desaulniers <ndesaulniers@google.com>
Subject: RE: clang asm-goto support (Was Re: [PATCH v2] x86/retpoline: Add clang support)
Date: Wed, 14 Feb 2018 10:07:43 +0000	[thread overview]
Message-ID: <A3D6595E4CE1DA4A80538CC077C51A2B84771946@hasmsx107.ger.corp.intel.com> (raw)
In-Reply-To: <20180214095308.GA16956@kroah.com>

I believe we do have the infrastructure in llvm to implement a working patch.
The "asm goto" was introduced as a solution to a problem (optimizing tracepoints/ nop patching).
When we started the discussion in llvm about adding this feature, questions arose if we can design a better solution for this problem than "asm goto".

You can find a more about the discussion in the llvm community here:
http://lists.llvm.org/pipermail/llvm-dev/2017-April/111748.html

* This is the middle of the thread, and there are more messages later on.

Thanks,
Marina

-----Original Message-----
From: Greg Kroah-Hartman [mailto:gregkh@linux-foundation.org] 
Sent: Wednesday, February 14, 2018 11:53
To: Chandler Carruth <chandlerc@google.com>
Cc: Yatsina, Marina <marina.yatsina@intel.com>; Kees Cook <keescook@google.com>; David Woodhouse <dwmw2@infradead.org>; Kreitzer, David L <david.l.kreitzer@intel.com>; Grischenko, Andrei L <andrei.l.grischenko@intel.com>; rnk@google.com; LLVM Developers <llvm-dev@lists.llvm.org>; ehsan@mozilla.com; Tayree, Coby <coby.tayree@intel.com>; Matthias Braun <matze@braunis.de>; Dean Michael Berris <dean.berris@gmail.com>; James Y Knight <jyknight@google.com>; Guenter Roeck <linux@roeck-us.net>; X86 ML <x86@kernel.org>; LKML <linux-kernel@vger.kernel.org>; Alan Cox <gnomes@lxorguk.ukuu.org.uk>; Rik van Riel <riel@redhat.com>; Andi Kleen <ak@linux.intel.com>; Josh Poimboeuf <jpoimboe@redhat.com>; Tom Lendacky <thomas.lendacky@amd.com>; Peter Zijlstra <peterz@infradead.org>; Linus Torvalds <torvalds@linux-foundation.org>; Jiri Kosina <jikos@kernel.org>; Andy Lutomirski <luto@amacapital.net>; Hansen, Dave <dave.hansen@intel.com>; Tim Chen <tim.c.chen@linux.intel.com>; Paul Turner <pjt@google.com>; Stephen Hines <srhines@google.com>; Nick Desaulniers <ndesaulniers@google.com>
Subject: Re: clang asm-goto support (Was Re: [PATCH v2] x86/retpoline: Add clang support)

On Wed, Feb 14, 2018 at 08:41:43AM +0000, Chandler Carruth wrote:
> Marina, Kees:
> 
> I think the combination of LKML and the LLVM developer's list is 
> probably too large a set of folks to have a (somewhat) lengthy 
> exploration of options and how to go about this and find a good path forward.

Why is there any need to discuss "options" here?  What is preventing the implementation of "asm goto" in llvm?  Is it the lack of a working patch, or something else?

thanks,

greg k-h
---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.

  reply	other threads:[~2018-02-14 10:07 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-14  0:28 clang asm-goto support (Was Re: [PATCH v2] x86/retpoline: Add clang support) Kees Cook
2018-02-14  8:33 ` Yatsina, Marina
2018-02-14  8:41   ` Chandler Carruth
2018-02-14  9:10     ` Peter Zijlstra
2018-02-14 10:44       ` Thomas Gleixner
2018-02-14  9:53     ` Greg Kroah-Hartman
2018-02-14 10:07       ` Yatsina, Marina [this message]
2018-02-14 10:17         ` Chandler Carruth
2018-02-14  9:08   ` Peter Zijlstra
2018-02-14  9:52     ` Yatsina, Marina
2018-02-14 10:34       ` Peter Zijlstra
2018-02-14 12:04         ` Ingo Molnar
2018-02-14 23:07           ` Ingo Molnar
2018-02-14 17:39         ` Linus Torvalds
2018-02-14 17:31     ` Kees Cook
2018-02-14 18:39       ` Peter Zijlstra
2018-02-14 15:31   ` James Y Knight
2018-02-14 17:21     ` Andy Lutomirski
2018-02-14 17:43     ` Linus Torvalds

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=A3D6595E4CE1DA4A80538CC077C51A2B84771946@hasmsx107.ger.corp.intel.com \
    --to=marina.yatsina@intel.com \
    --cc=ak@linux.intel.com \
    --cc=andrei.l.grischenko@intel.com \
    --cc=chandlerc@google.com \
    --cc=coby.tayree@intel.com \
    --cc=dave.hansen@intel.com \
    --cc=david.l.kreitzer@intel.com \
    --cc=dean.berris@gmail.com \
    --cc=dwmw2@infradead.org \
    --cc=ehsan@mozilla.com \
    --cc=gnomes@lxorguk.ukuu.org.uk \
    --cc=gregkh@linux-foundation.org \
    --cc=jikos@kernel.org \
    --cc=jpoimboe@redhat.com \
    --cc=jyknight@google.com \
    --cc=keescook@google.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=llvm-dev@lists.llvm.org \
    --cc=luto@amacapital.net \
    --cc=matze@braunis.de \
    --cc=ndesaulniers@google.com \
    --cc=peterz@infradead.org \
    --cc=pjt@google.com \
    --cc=riel@redhat.com \
    --cc=rnk@google.com \
    --cc=srhines@google.com \
    --cc=thomas.lendacky@amd.com \
    --cc=tim.c.chen@linux.intel.com \
    --cc=torvalds@linux-foundation.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.