From: Keir Fraser <keir@xensource.com>
To: Alex Williamson <alex.williamson@hp.com>
Cc: xen-devel <xen-devel@lists.xensource.com>
Subject: Re: Is patches/linux-2.6.18/xen-hotplug.patch still necessary?
Date: Wed, 09 May 2007 16:32:29 +0100 [thread overview]
Message-ID: <C267A81D.E996%keir@xensource.com> (raw)
In-Reply-To: <1178723240.6512.2.camel@lappy>
On 9/5/07 16:07, "Alex Williamson" <alex.williamson@hp.com> wrote:
>> It has to be handled in top itself. With some versions at least it was the
>> case that top would crash when CPUs where hot-unplugged. If you revert that
>> patch, does your version of top work okay in the hot-unplug case (you have
>> to run top across the unplug operation to see if it will crash or not)?
>
> Top still crashes without that patch, but the patch just hides the
> problem. Top also crashes during a CPU hotplug operation on bare metal.
> I think this patch causes more problems than if fixes. Thanks,
The fact that physical CPU hotplug is an incredibly rare operation, even on
the 0.001% of all systems that it is actually supported on, compared with
virtual CPU hotplug being potentially quite frequent, is a real concern.
Users are actually moderately likely to see this bug when running on Xen. In
fact they used to, and then they complained to us, at which point we applied
the band-aid solution.
-- Keir
next prev parent reply other threads:[~2007-05-09 15:32 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-05-08 22:26 Is patches/linux-2.6.18/xen-hotplug.patch still necessary? Alex Williamson
2007-05-09 7:05 ` Keir Fraser
2007-05-09 13:52 ` Dave Lively
2007-05-09 15:07 ` Alex Williamson
2007-05-09 15:32 ` Keir Fraser [this message]
2007-05-09 15:38 ` Alex Williamson
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=C267A81D.E996%keir@xensource.com \
--to=keir@xensource.com \
--cc=alex.williamson@hp.com \
--cc=xen-devel@lists.xensource.com \
/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.