From: Hugh Dickins <hughd@google.com>
To: Dave Hansen <dave.hansen@linux.intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>,
linux-kernel@vger.kernel.org, linux-mm@kvack.org,
hughd@google.com, moritz.lipp@iaik.tugraz.at,
daniel.gruss@iaik.tugraz.at, michael.schwarz@iaik.tugraz.at,
richard.fellner@student.tugraz.at, luto@kernel.org,
torvalds@linux-foundation.org, keescook@google.com,
x86@kernel.org
Subject: Re: [PATCH 18/30] x86, kaiser: map virtually-addressed performance monitoring buffers
Date: Tue, 14 Nov 2017 11:10:23 -0800 (PST) [thread overview]
Message-ID: <alpine.LSU.2.11.1711141057510.2433@eggly.anvils> (raw)
In-Reply-To: <30655167-963f-09e3-f88f-600bb95407e8@linux.intel.com>
On Tue, 14 Nov 2017, Dave Hansen wrote:
> On 11/14/2017 10:20 AM, Peter Zijlstra wrote:
> > On Fri, Nov 10, 2017 at 11:31:39AM -0800, Dave Hansen wrote:
> >> static int alloc_ds_buffer(int cpu)
> >> {
> >> + struct debug_store *ds = per_cpu_ptr(&cpu_debug_store, cpu);
> >>
> >> + memset(ds, 0, sizeof(*ds));
> > Still wondering about that memset...
Sorry, my attention is far away at the moment.
>
> My guess is that it was done to mirror the zeroing done by the original
> kzalloc().
You guess right.
> But, I think you're right that it's zero'd already by virtue
> of being static:
>
> static
> DEFINE_PER_CPU_SHARED_ALIGNED_USER_MAPPED(struct debug_store,
> cpu_debug_store);
>
> I'll queue a cleanup, or update it if I re-post the set.
I was about to agree, but now I'm not so sure. I don't know much
about these PMC things, but at a glance it looks like what is reserved
by x86_reserve_hardware() may later be released by x86_release_hardware(),
and then later reserved again by x86_reserve_hardware(). And although
the static per-cpu area would be zeroed the first time, the second time
it will contain data left over from before, so really needs the memset?
Hugh
next prev parent reply other threads:[~2017-11-14 19:10 UTC|newest]
Thread overview: 77+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-10 19:30 [PATCH 00/30] [v3] KAISER: unmap most of the kernel from userspace page tables Dave Hansen
2017-11-10 19:31 ` [PATCH 01/30] x86, mm: do not set _PAGE_USER for init_mm " Dave Hansen
2017-11-10 19:31 ` [PATCH 02/30] x86, tlb: Make CR4-based TLB flushes more robust Dave Hansen
2017-11-10 19:31 ` [PATCH 03/30] x86/mm: Document X86_CR4_PGE toggling behavior Dave Hansen
2017-11-10 19:31 ` [PATCH 04/30] x86, kaiser: disable global pages by default with KAISER Dave Hansen
2017-11-14 19:38 ` Rik van Riel
2017-11-26 14:48 ` Ingo Molnar
2017-11-27 11:37 ` Thomas Gleixner
2017-11-27 13:20 ` [PATCH v2] x86/mm/kaiser: Disable " Ingo Molnar
2017-11-27 13:23 ` Thomas Gleixner
2017-11-27 13:27 ` Ingo Molnar
2017-11-10 19:31 ` [PATCH 05/30] x86, kaiser: prepare assembly for entry/exit CR3 switching Dave Hansen
2017-11-20 12:17 ` Thomas Gleixner
2017-11-10 19:31 ` [PATCH 06/30] x86, kaiser: introduce user-mapped per-cpu areas Dave Hansen
2017-11-10 19:31 ` [PATCH 07/30] x86, kaiser: mark per-cpu data structures required for entry/exit Dave Hansen
2017-11-10 19:31 ` [PATCH 08/30] x86, kaiser: unmap kernel from userspace page tables (core patch) Dave Hansen
2017-11-20 17:21 ` Thomas Gleixner
2017-11-22 22:45 ` Dave Hansen
2017-11-22 22:50 ` Dave Hansen
2017-11-22 22:54 ` Dave Hansen
2017-11-22 23:11 ` Dave Hansen
2017-11-10 19:31 ` [PATCH 09/30] x86, kaiser: only populate shadow page tables for userspace Dave Hansen
2017-11-20 20:12 ` Thomas Gleixner
2017-11-21 7:05 ` Ingo Molnar
2017-11-21 22:09 ` Dave Hansen
2017-11-22 3:44 ` Andy Lutomirski
2017-11-22 23:30 ` Dave Hansen
2017-11-10 19:31 ` [PATCH 10/30] x86, kaiser: allow NX poison to be set in p4d/pgd Dave Hansen
2017-11-10 19:31 ` [PATCH 11/30] x86, kaiser: make sure static PGDs are 8k in size Dave Hansen
2017-11-10 19:31 ` [PATCH 12/30] x86, kaiser: map GDT into user page tables Dave Hansen
2017-11-20 20:22 ` Thomas Gleixner
2017-11-20 20:46 ` Andy Lutomirski
2017-11-20 20:55 ` Thomas Gleixner
2017-11-21 21:19 ` Dave Hansen
2017-11-21 22:46 ` Andy Lutomirski
2017-11-21 23:17 ` Dave Hansen
2017-11-21 23:32 ` Andy Lutomirski
2017-11-21 23:42 ` Dave Hansen
2017-11-22 0:17 ` Andy Lutomirski
2017-11-22 0:37 ` Dave Hansen
2017-11-21 22:12 ` Dave Hansen
2017-11-10 19:31 ` [PATCH 13/30] x86, kaiser: map dynamically-allocated LDTs Dave Hansen
2017-11-10 19:31 ` [PATCH 14/30] x86, kaiser: map espfix structures Dave Hansen
2017-11-10 19:31 ` [PATCH 15/30] x86, kaiser: map entry stack variables Dave Hansen
2017-11-10 19:31 ` [PATCH 16/30] x86, kaiser: map trace interrupt entry Dave Hansen
2017-11-10 19:31 ` [PATCH 17/30] x86, kaiser: map debug IDT tables Dave Hansen
2017-11-20 20:40 ` Thomas Gleixner
2017-11-21 22:16 ` Dave Hansen
2017-11-20 20:44 ` Andy Lutomirski
2017-11-20 20:54 ` Thomas Gleixner
2017-11-10 19:31 ` [PATCH 18/30] x86, kaiser: map virtually-addressed performance monitoring buffers Dave Hansen
2017-11-14 18:20 ` Peter Zijlstra
2017-11-14 18:28 ` Dave Hansen
2017-11-14 19:10 ` Hugh Dickins [this message]
2017-11-14 19:24 ` Andy Lutomirski
2017-11-15 9:41 ` Peter Zijlstra
2017-11-10 19:31 ` [PATCH 19/30] x86, mm: Move CR3 construction functions Dave Hansen
2017-11-10 19:31 ` [PATCH 20/30] x86, mm: remove hard-coded ASID limit checks Dave Hansen
2017-11-20 20:47 ` Thomas Gleixner
2017-11-10 19:31 ` [PATCH 21/30] x86, mm: put mmu-to-h/w ASID translation in one place Dave Hansen
2017-11-10 22:03 ` Andy Lutomirski
2017-11-10 22:09 ` Dave Hansen
2017-11-10 22:10 ` Andy Lutomirski
2017-11-10 19:31 ` [PATCH 22/30] x86, pcid, kaiser: allow flushing for future ASID switches Dave Hansen
2017-11-10 19:31 ` [PATCH 23/30] x86, kaiser: use PCID feature to make user and kernel switches faster Dave Hansen
2017-11-16 19:19 ` Andrea Arcangeli
2017-11-16 19:25 ` Dave Hansen
2017-11-10 19:31 ` [PATCH 24/30] x86, kaiser: disable native VSYSCALL Dave Hansen
2017-11-10 19:31 ` [PATCH 25/30] x86, kaiser: add debugfs file to turn KAISER on/off at runtime Dave Hansen
2017-11-10 19:31 ` [PATCH 26/30] x86, kaiser: add a function to check for KAISER being enabled Dave Hansen
2017-11-10 19:31 ` [PATCH 27/30] x86, kaiser: un-poison PGDs at runtime Dave Hansen
2017-11-10 19:31 ` [PATCH 28/30] x86, kaiser: allow KAISER to be enabled/disabled " Dave Hansen
2017-11-10 19:32 ` [PATCH 29/30] x86, kaiser: add Kconfig Dave Hansen
2017-11-10 19:32 ` [PATCH 30/30] x86, kaiser, xen: Dynamically disable KAISER when running under Xen PV Dave Hansen
2017-11-20 16:02 ` [PATCH 00/30] [v3] KAISER: unmap most of the kernel from userspace page tables Juerg Haefliger
-- strict thread matches above, loose matches on Subject: below --
2017-11-08 19:46 [PATCH 00/30] [v2] " Dave Hansen
2017-11-08 19:47 ` [PATCH 18/30] x86, kaiser: map virtually-addressed performance monitoring buffers Dave Hansen
2017-11-10 12:17 ` Peter Zijlstra
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.LSU.2.11.1711141057510.2433@eggly.anvils \
--to=hughd@google.com \
--cc=daniel.gruss@iaik.tugraz.at \
--cc=dave.hansen@linux.intel.com \
--cc=keescook@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=luto@kernel.org \
--cc=michael.schwarz@iaik.tugraz.at \
--cc=moritz.lipp@iaik.tugraz.at \
--cc=peterz@infradead.org \
--cc=richard.fellner@student.tugraz.at \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®