All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ben Guthro <bguthro@virtualiron.com>
To: Keir Fraser <keir.fraser@eu.citrix.com>
Cc: xen-devel <xen-devel@lists.xensource.com>,
	Robert Phillips <rphillips@virtualiron.com>
Subject: Re: [PATCH] scrub pages on guest termination
Date: Fri, 23 May 2008 13:01:16 -0400	[thread overview]
Message-ID: <4836F85C.1010609@virtualiron.com> (raw)
In-Reply-To: <C45CA9BA.18E7A%keir.fraser@eu.citrix.com>


[-- Attachment #1.1: Type: text/plain, Size: 1494 bytes --]

Yes, sorry -  should have removed our terminology from the description.
Node=physical machine
VS=HVM guest w/ pv-on-hvm drivers
Looking back at the original bug report - it seems to indicate it was 
migrating from a system with 2 processors to one with 8

Specifcally - from
Dell Precision WorkStation 380
Processor:    Intel(R) Pentium(R) D CPU 2.80GHz
# of CPUs:    2
Speed:    2.8GHz

to

Supermicro X7DB8
Processor:    Genuine Intel(R) CPU @ 2.13GHz
# of CPUs:    8
Speed:    2.133 GHz




Keir Fraser wrote:
> The aim of the loop was to scrub enough pages in a batch that lock 
> contention is kept tolerably low. Even if 16 pages is not sufficient 
> for that, I'm surprised a 'node' (you mean a whole system, 
> presumably?) would appear to lock up. Maybe pages would be scrubbed 
> slower than we'd like, but still CPUs should be able to get the 
> spinlock often enough to evaluate whether they have spent 1ms in the 
> loop and hence get out of there.
>
> What sort of system were you seeing the lockup on? Does it have very 
> many physical CPUs?
>
>  -- Keir
>
> On 23/5/08 16:00, "Ben Guthro" <bguthro@virtualiron.com> wrote:
>
>     This patch solves the following problem.  When a large VS
>     terminates, the node locks
>     up. The node locks up because the page_scrub_kick routine sends a
>     softirq to
>     all processors instructing them to run the page scrub code.  There
>     they interfere
>     with each other as they serialize behind the page_scrub_lock.
>
>


[-- Attachment #1.2: Type: text/html, Size: 2418 bytes --]

[-- Attachment #2: Type: text/plain, Size: 138 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

  reply	other threads:[~2008-05-23 17:01 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-23 15:00 [PATCH] scrub pages on guest termination Ben Guthro
2008-05-23 16:04 ` Keir Fraser
2008-05-23 17:01   ` Ben Guthro [this message]
2008-05-23 17:19     ` Keir Fraser

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=4836F85C.1010609@virtualiron.com \
    --to=bguthro@virtualiron.com \
    --cc=keir.fraser@eu.citrix.com \
    --cc=rphillips@virtualiron.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.