[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4C7A263F.1060407@redhat.com>
Date: Sun, 29 Aug 2010 12:19:59 +0300
From: Avi Kivity <avi@...hat.com>
To: Xiao Guangrong <xiaoguangrong@...fujitsu.com>
CC: Marcelo Tosatti <mtosatti@...hat.com>,
LKML <linux-kernel@...r.kernel.org>, KVM <kvm@...r.kernel.org>
Subject: Re: [PATCH 4/4] KVM: MMU: lower the aduit frequency
On 08/28/2010 03:03 PM, Xiao Guangrong wrote:
> The audit is very high overhead, so we need lower the frequency to assure the guest running
>
>
> */
>
> #include<linux/debugfs.h>
> +#include<linux/ratelimit.h>
>
> static struct dentry *debugfs_file;
> static bool mmu_debug;
> @@ -233,6 +234,11 @@ static void audit_vcpu_spte(struct kvm_vcpu *vcpu)
>
> static void kvm_mmu_audit(void *ignore, struct kvm_vcpu *vcpu, const char *msg)
> {
> + static DEFINE_RATELIMIT_STATE(ratelimit_state, 5 * HZ, 10);
> +
> + if (!__ratelimit(&ratelimit_state))
> + return;
> +
> audit_msg = msg;
> audit_all_active_sps(vcpu->kvm);
> audit_vcpu_spte(vcpu);
This means we see a bug long after it happened, so we can't correlate it
to the cause.
It's fine as an option (even the default) but I'd like to be able to
audit after every operation. Perhaps a partial audit that only looks at
the gfns and vaddrs that were affected in the last operation?
I have to admit, it's been a very long time since I last used audit.
--
error compiling committee.c: too many arguments to function
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists