Re: [PATCH] trace "exit to userspace" event

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



 On 10/14/2010 12:29 PM, Gleb Natapov wrote:
On Thu, Oct 14, 2010 at 12:27:15PM +0200, Avi Kivity wrote:
>   On 10/10/2010 05:46 PM, Gleb Natapov wrote:
>  >>
>  >>   We should log both errno and exit_reason.  If we want to be clever,
>  >>   we can display strerror(errno) if it's nonzero, and exit_reason
>  >>   otherwise (easy to do in a trace-cmd plugin).
>  >>
>  >For starters we should remove KVM_EXIT_INTR exit reason. Looking into
>  >qemu-kvm history it was never used and there is at least one code path
>  >that returns -EINTR and does not set KVM_EXIT_INTR, so exit_reason field
>  >contains stale info on exit.
>  >
>
>  The two issues are unrelated.
>
So what do you propose? I see no issue with my original patch.

Record both errno and exit_reason. While they're never both valid at the same time, they're both necessary.

--
error compiling committee.c: too many arguments to function

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux