From: Thomas Gleixner <tglx@linutronix.de>
To: speck@linutronix.de
Subject: Re: [PATCH 3/4] INVERTv2 3
Date: Wed, 8 Aug 2018 21:43:57 +0200 (CEST) [thread overview]
Message-ID: <alpine.DEB.2.21.1808082143400.1533@nanos.tec.linutronix.de> (raw)
In-Reply-To: <20180808193159.GZ4238@tassilo.jf.intel.com>
On Wed, 8 Aug 2018, speck for Andi Kleen wrote:
> On Wed, Aug 08, 2018 at 09:23:17AM +0200, speck for Thomas Gleixner wrote:
> > On Tue, 7 Aug 2018, speck for Andi Kleen wrote:
> >
> > > From: Andi Kleen <ak@linux.intel.com>
> > > Subject: x86/speculation/l1tf: Make the mmio tracer invert
> > >
> > > The mmio tracer sets io mapping PTEs and PMDs to non present when enabled.
> > > Make it use the right low level macros to actually invert them
> > > to protect against L1TF.
> > >
> > > Unfortunately I wasn't able to test this, since the procedure described
> > > in Documentation/trace/mmiotrace.rst doesn't result in any tracing
> > > activity, with or without my patch, when starting X.
> >
> > You need to make sure that the GFX driver is loaded _after_ enabling MMIO
> > tracing. I'll omit that for now, until we have confirmation about testing.
>
> It seems very difficult to load the gfx driver late. After wasting a lot of
> time trying to work around this, I managed to get the testmmiotrace module to
> work, and it appears the trace is working even with my patches.
>
> So the testing paragraph can be dropped.
Ok. Thanks!
next prev parent reply other threads:[~2018-08-08 19:43 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-06 23:29 [MODERATED] [PATCH 0/4] INVERTv1 2 Andi Kleen
2018-08-06 23:29 ` [MODERATED] [PATCH 1/4] INVERTv1 1 Andi Kleen
2018-08-06 23:36 ` [MODERATED] " Linus Torvalds
2018-08-07 0:15 ` Andi Kleen
2018-08-06 23:29 ` [MODERATED] [PATCH 2/4] INVERTv1 4 Andi Kleen
2018-08-06 23:29 ` [MODERATED] [PATCH 3/4] INVERTv1 3 Andi Kleen
2018-08-06 23:29 ` [MODERATED] [PATCH 4/4] INVERTv1 0 Andi Kleen
[not found] ` <cover.1533679665.git.ak@linux.intel.com>
2018-08-07 22:09 ` [MODERATED] [PATCH 1/4] INVERTv2 1 Andi Kleen
2018-08-07 22:09 ` [MODERATED] [PATCH 2/4] INVERTv2 4 Andi Kleen
2018-08-07 22:09 ` [MODERATED] [PATCH 3/4] INVERTv2 3 Andi Kleen
2018-08-08 7:23 ` Thomas Gleixner
2018-08-08 19:31 ` [MODERATED] " Andi Kleen
2018-08-08 19:43 ` Thomas Gleixner [this message]
2018-08-07 22:09 ` [MODERATED] [PATCH 4/4] INVERTv2 0 Andi Kleen
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=alpine.DEB.2.21.1808082143400.1533@nanos.tec.linutronix.de \
--to=tglx@linutronix.de \
--cc=speck@linutronix.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).