qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Rik van Riel <riel@redhat.com>
To: Richard Davies <richard@arachsys.com>
Cc: Avi Kivity <avi@redhat.com>, kvm@vger.kernel.org, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Windows slow boot: contractor wanted
Date: Sat, 25 Aug 2012 14:11:48 -0400	[thread overview]
Message-ID: <50391564.30401@redhat.com> (raw)
In-Reply-To: <20120825174550.GA8619@alpha.arachsys.com>

On 08/25/2012 01:45 PM, Richard Davies wrote:

> Are you talking about these patches?
>
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commit;h=c67fe3752abe6ab47639e2f9b836900c3dc3da84
> http://marc.info/?l=linux-mm&m=134521289221259
>
> If so, I believe those are in 3.6.0-rc3, so I tested with that.
>
> Unfortunately, I can still get the slow boots and perf top showing
> _raw_spin_lock_irqsave.
>
>
> Here are two perf top traces on 3.6.0-rc3. They do look a bit different from
> 3.5.2, but _raw_spin_lock_irqsave is still at the top:
>
>     PerfTop:   35272 irqs/sec  kernel:98.1%  exact:  0.0% [4000Hz cycles],  (all, 16 CPUs)
> ------------------------------------------------------------------------------------------------------------------
>
>      61.85%  [kernel]          [k] _raw_spin_lock_irqsave
>       7.18%  [kernel]          [k] sub_preempt_count
>       5.03%  [kernel]          [k] isolate_freepages_block
>       2.49%  [kernel]          [k] yield_to
>       2.05%  [kernel]          [k] memcmp
>       2.01%  [kernel]          [k] compact_zone
>       1.76%  [kernel]          [k] add_preempt_count
>       1.52%  [kernel]          [k] _raw_spin_lock
>       1.31%  [kernel]          [k] kvm_vcpu_on_spin
>       0.92%  [kernel]          [k] svm_vcpu_run

However, the compaction code is not as prominent as before.

Can you get a backtrace to that _raw_spin_lock_irqsave, to see
from where it is running into lock contention?

It would be good to know whether it is isolate_freepages_block,
yield_to, kvm_vcpu_on_spin or something else...

-- 
All rights reversed

  reply	other threads:[~2012-08-25 18:11 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-16 10:47 [Qemu-devel] Windows slow boot: contractor wanted Richard Davies
2012-08-16 11:39 ` Avi Kivity
2012-08-17 12:36   ` Richard Davies
2012-08-17 13:02     ` Robert Vineyard
2012-08-18 14:44       ` Richard Davies
2012-08-19  5:02     ` Brian Jackson
2012-08-20  8:16       ` Richard Davies
2012-08-19  8:40     ` Avi Kivity
2012-08-19  8:51       ` Richard Davies
2012-08-19 14:04     ` Avi Kivity
2012-08-20 13:56       ` Richard Davies
2012-08-21  9:00         ` Avi Kivity
2012-08-21 15:21           ` Richard Davies
2012-08-21 15:39             ` Troy Benjegerdes
2012-08-22  9:08             ` Avi Kivity
2012-08-22 12:40               ` Richard Davies
2012-08-22 12:44                 ` Avi Kivity
2012-08-22 14:41                   ` Richard Davies
2012-08-22 14:53                     ` Avi Kivity
2012-08-22 15:26                       ` Richard Davies
2012-08-22 17:22                       ` Troy Benjegerdes
2012-08-25 17:51                         ` Richard Davies
2012-08-22 15:21                     ` Rik van Riel
2012-08-22 15:34                       ` Richard Davies
2012-08-25 17:45                       ` Richard Davies
2012-08-25 18:11                         ` Rik van Riel [this message]
2012-08-26 10:58                           ` Richard Davies
2012-09-06  9:20                             ` Richard Davies
2012-09-12 10:56                               ` [Qemu-devel] Windows VM slow boot Richard Davies
2012-09-12 12:25                                 ` Mel Gorman
2012-09-12 16:46                                   ` Richard Davies
2012-09-13  9:50                                     ` Mel Gorman
2012-09-13 19:47                                     ` [Qemu-devel] [PATCH 1/2] Revert "mm: have order > 0 compaction start near a pageblock with free pages" Rik van Riel
2012-09-13 19:48                                     ` [Qemu-devel] [PATCH 2/2] make the compaction "skip ahead" logic robust Rik van Riel
2012-09-13 19:54                                       ` [Qemu-devel] [PATCH -v2 " Rik van Riel
2012-09-15 15:55                                         ` Richard Davies
2012-09-16 19:12                                           ` Richard Davies
2012-09-17 12:26                                           ` Mel Gorman
2012-09-18  8:14                                             ` Richard Davies
2012-09-18 11:21                                               ` Mel Gorman
2012-09-18 17:58                                                 ` Richard Davies
2012-09-17 13:50                                           ` Rik van Riel
2012-09-17 14:07                                             ` Mel Gorman
2012-08-16 14:10 ` [Qemu-devel] Windows slow boot: contractor wanted Benoît Canet
2012-08-16 15:53 ` Troy Benjegerdes
2012-09-18 15:12 ` [Qemu-devel] Windows slow boot Michael Tokarev

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=50391564.30401@redhat.com \
    --to=riel@redhat.com \
    --cc=avi@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=qemu-devel@nongnu.org \
    --cc=richard@arachsys.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 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 NNTP newsgroup(s).