All of lore.kernel.org
 help / color / mirror / Atom feed
From: Borislav Petkov <bp@alien8.de>
To: Peter Zijlstra <peterz@infradead.org>
Cc: linux-kernel@vger.kernel.org, linux-tip-commits@vger.kernel.org,
	Michael Matz <matz@suse.de>, Dave Jiang <dave.jiang@intel.com>,
	Borislav Petkov <bp@suse.de>, Tony Luck <tony.luck@intel.com>,
	x86 <x86@kernel.org>
Subject: Re: [tip: x86/pasid] x86/asm: Carve out a generic movdir64b() helper for general usage
Date: Wed, 7 Oct 2020 23:13:27 +0200	[thread overview]
Message-ID: <20201007211327.GN5607@zn.tnic> (raw)
In-Reply-To: <20201007170835.GM2628@hirez.programming.kicks-ass.net>

On Wed, Oct 07, 2020 at 07:08:35PM +0200, Peter Zijlstra wrote:
> (%rdx), %rax, surely?

Right, later. Already tagged the branch so that Vinod can base stuff ontop.

> Also, that's a horrible convention, but I suppose (%rdx), (%rax) was
> out?

See the end of this mail:

https://lkml.kernel.org/r/alpine.LSU.2.20.2009241356020.20802@wotan.suse.de

> Can we pretty please get a binutils version that knows about this
> instruction, such that we know when we can get rid of the silly .byte
> encoded mess?

It looks like support for this insn got introduced in this binutils commit:

c0a30a9f0ab4 ("Enable Intel MOVDIRI, MOVDIR64B instructions")

So I guess from 2.31 onwards:

$ git tag --contains c0a30a9f0ab48
binutils-2_31
binutils-2_31_1
binutils-2_32
binutils-2_33
binutils-2_33_1
binutils-2_34
binutils-2_35
binutils-2_35_1
gdb-8.2-release
gdb-8.2.1-release
gdb-8.3-release
gdb-8.3.1-release
gdb-9.1-release
gdb-9.2-release
...

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette

  reply	other threads:[~2020-10-07 21:13 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-05 15:11 [PATCH v7 0/5] Add shared workqueue support for idxd driver Dave Jiang
2020-10-05 15:11 ` [PATCH v7 1/5] x86/asm: Carve out a generic movdir64b() helper for general usage Dave Jiang
2020-10-07 16:14   ` [tip: x86/pasid] " tip-bot2 for Dave Jiang
2020-10-07 17:08     ` Peter Zijlstra
2020-10-07 21:13       ` Borislav Petkov [this message]
2020-10-08  6:49         ` Peter Zijlstra
2020-10-05 15:11 ` [PATCH v7 2/5] x86/asm: Add an enqcmds() wrapper for the ENQCMDS instruction Dave Jiang
2020-10-05 15:11 ` [PATCH v7 3/5] dmaengine: idxd: Add shared workqueue support Dave Jiang
2020-10-05 15:11 ` [PATCH v7 4/5] dmaengine: idxd: Clean up descriptors with fault error Dave Jiang
2020-10-05 15:11 ` [PATCH v7 5/5] dmaengine: idxd: Add ABI documentation for shared wq Dave Jiang
2020-10-07  7:01 ` [PATCH v7 0/5] Add shared workqueue support for idxd driver Vinod Koul
2020-10-07  8:48   ` Borislav Petkov
2020-10-07  9:53     ` Vinod Koul
2020-10-07 10:04       ` Borislav Petkov
2020-10-07 14:57         ` Vinod Koul
2020-10-07 16:16           ` Borislav Petkov
2020-10-07 16:50   ` Dave Jiang

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=20201007211327.GN5607@zn.tnic \
    --to=bp@alien8.de \
    --cc=bp@suse.de \
    --cc=dave.jiang@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-tip-commits@vger.kernel.org \
    --cc=matz@suse.de \
    --cc=peterz@infradead.org \
    --cc=tony.luck@intel.com \
    --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.