All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rik van Riel <riel@redhat.com>
To: Richard Davies <richard@arachsys.com>
Cc: Avi Kivity <avi@redhat.com>, qemu-devel@nongnu.org, kvm@vger.kernel.org
Subject: Re: 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

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