From: Keir Fraser <keir.xen@gmail.com>
To: Andrew Cooper <andrew.cooper3@citrix.com>
Cc: Ian Jackson <Ian.Jackson@eu.citrix.com>, Tim Deegan <tim@xen.org>,
Ian Campbell <Ian.Campbell@citrix.com>,
Jan Beulich <JBeulich@suse.com>,
Xen-devel <xen-devel@lists.xen.org>
Subject: Re: [PATCH 0/3] per_cpu() fixes
Date: Thu, 03 Oct 2013 20:52:06 +0100 [thread overview]
Message-ID: <CE738976.3795A%keir.xen@gmail.com> (raw)
In-Reply-To: <524DAF8D.7050003@citrix.com>
On 03/10/2013 18:55, "Andrew Cooper" <andrew.cooper3@citrix.com> wrote:
> On 26/09/13 10:49, Andrew Cooper wrote:
>> This patch series is two independent but related changes to the use of
>> per_cpu() with offline pcpus.
>>
>> The first patch is a fix for use of get_cpu_idle_time() whereby toolstack
>> hypercalls could cause an effective NULL structure dereference.
>>
>> The second patch is a forward looking fix to try and prevent similar issues
>> in
>> the future. It causes uses of per_cpu() against an offline pcpu to cause a
>> #GF rather than to try and dereference a very low address.
>>
>> The third patch is debugging code to demonstrate the effects of patch 2.
>>
>> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
>> CC: Keir Fraser <keir@xen.org>
>> CC: Jan Beulich <JBeulich@suse.com>
>> CC: Tim Deegan <tim@xen.org>
>> CC: Ian Campbell <Ian.Campbell@citrix.com>
>> CC: Ian Jackson <Ian.Jackson@eu.citrix.com>
>>
>
> Ping? Any thoughts at all?
Sensible patches.
Acked-by: Keir Fraser <keir@xen.org>
> ~Andrew
prev parent reply other threads:[~2013-10-03 19:52 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-26 9:49 [PATCH 0/3] per_cpu() fixes Andrew Cooper
2013-09-26 9:49 ` [PATCH 1/3] x86/idle: Fix get_cpu_idle_time()'s interaction with offline pcpus Andrew Cooper
2013-09-26 9:49 ` [PATCH 2/3] x86/percpu: Force INVALID_PERCPU_AREA into the non-canonical address region Andrew Cooper
2013-09-26 10:04 ` Andrew Cooper
2013-09-26 9:49 ` [PATCH 3/3] DO NOT APPLY - debugging code for gpf when accessing invalid per_cpu() data Andrew Cooper
2013-10-03 17:55 ` [PATCH 0/3] per_cpu() fixes Andrew Cooper
2013-10-03 19:52 ` Keir Fraser [this message]
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=CE738976.3795A%keir.xen@gmail.com \
--to=keir.xen@gmail.com \
--cc=Ian.Campbell@citrix.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=JBeulich@suse.com \
--cc=andrew.cooper3@citrix.com \
--cc=tim@xen.org \
--cc=xen-devel@lists.xen.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.