From: Nitesh Narayan Lal <nitesh@redhat.com>
To: linux-kernel@vger.kernel.org, kvm@vger.kernel.org,
tglx@linutronix.de, alexandre.chartre@oracle.com,
peterz@infradead.org, pbonzini@redhat.com, w90p710@gmail.com,
sean.j.christopherson@intel.com, vkuznets@redhat.com
Subject: Possible regression in cpuacct.stats system time
Date: Mon, 4 Jan 2021 16:24:30 -0500 [thread overview]
Message-ID: <12a1b9d4-8534-e23a-6bbd-736474928e6b@redhat.com> (raw)
Hi,
Last year I reported an issue of "suspicious RCU usage" [1] with the debug
kernel which was fixed with the patch:
87fa7f3e98 "x86/kvm: Move context tracking where it belongs"
Recently I have come across a possible regression because of this
patch in the cpuacct.stats system time.
With the latest upstream kernel (5.11-rc2) when we set up a VM and start
observing the system time value from cpuacct.stat then it is significantly
higher than value reported with the kernel that doesn't have the
previously mentioned patch.
For instance, the following are the values of cpuacct.stats right after the
VM bring up completion for two cases:
with a kernel that has the patch-
user 471
system 6094
with the patch reverted-
user 498
system 1873
FWIU the reason behind this increase is the moving of guest_exit_irqoff()
to its proper location (near vmexit). This leads to the accounting
of instructions that were previously accounted into the guest context as a
part of the system time.
IMO this should be an expected behavior after the previously mentioned
change. Is that a right conclusion or I am missing something here?
Another question that I have is about the patch
d7a08882a0 "KVM: x86: Unconditionally enable irqs in guest context"
considering we are enabling irqs early now in the code path, do we still
need this patch?
[1] https://lore.kernel.org/lkml/ece36eb1-253a-8ec6-c183-309c10bb35d5@redhat.com/
--
Thanks
Nitesh
next reply other threads:[~2021-01-04 21:26 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-04 21:24 Nitesh Narayan Lal [this message]
2021-01-05 7:27 ` Possible regression in cpuacct.stats system time Paolo Bonzini
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=12a1b9d4-8534-e23a-6bbd-736474928e6b@redhat.com \
--to=nitesh@redhat.com \
--cc=alexandre.chartre@oracle.com \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pbonzini@redhat.com \
--cc=peterz@infradead.org \
--cc=sean.j.christopherson@intel.com \
--cc=tglx@linutronix.de \
--cc=vkuznets@redhat.com \
--cc=w90p710@gmail.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).