Re: [RFC PATCH v2 3/4] kvm: Common device control API functions

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

 



On 26 September 2013 20:56, Christoffer Dall
<christoffer.dall@xxxxxxxxxx> wrote:
> Introduces two simple functions:
>     int kvm_device_ioctl(int fd, int type, ...);
>     int kvm_create_device(KVMState *s, uint64_t type, bool test);
>
> These functions wrap the basic ioctl-based interactions with KVM in a
> way similar to other KVM ioctl wrappers.
>
> Signed-off-by: Christoffer Dall <christoffer.dall@xxxxxxxxxx>
> --- a/trace-events
> +++ b/trace-events
> @@ -1163,6 +1163,7 @@ migrate_set_state(int new_state) "new state %d"
>  kvm_ioctl(int type, void *arg) "type %d, arg %p"
>  kvm_vm_ioctl(int type, void *arg) "type %d, arg %p"
>  kvm_vcpu_ioctl(int cpu_index, int type, void *arg) "cpu_index %d, type %d, arg %p"
> +kvm_device_ioctl(int fd, int type, void *arg) "dev fd %d, type %d, arg %p"
>  kvm_run_exit(int cpu_index, uint32_t reason) "cpu_index %d, reason %d"

Following commit 4fe6e9ec this will need rebasing to
avoid the conflict, and you should also update the
format string in line with that commit so it reads
"dev fd %d, type 0x%x, arg %p".
Other than that nit,

Reviewed-by: Peter Maydell <peter.maydell@xxxxxxxxxx>

-- PMM
_______________________________________________
kvmarm mailing list
kvmarm@xxxxxxxxxxxxxxxxxxxxx
https://lists.cs.columbia.edu/cucslists/listinfo/kvmarm




[Index of Archives]     [Linux KVM]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux