Re: [PATCH 0/3] Fixes for kvmtool virtio-net part

From: Andre Przywara <andre.przywara@arm.com>
To: Fan Du <fan.du@intel.com>
Cc: kvm@vger.kernel.org, penberg@kernel.org,
	Will Deacon <will.deacon@arm.com>,
	Marc Zyngier <Marc.Zyngier@arm.com>
Subject: Re: [PATCH 0/3] Fixes for kvmtool virtio-net part
Date: Mon, 20 Jul 2015 15:42:56 +0100	[thread overview]
Message-ID: <55AD08F0.1030205@arm.com> (raw)
In-Reply-To: <1437384527-22987-1-git-send-email-fan.du@intel.com>

Hi,
> 
> This changset provide some fix and small enhancement
> for kvm tool when I play it with virtio part.

have you seen the new kvmtool repository Will created, which is a
stand-alone version outside of the Linux tree?

https://git.kernel.org/cgit/linux/kernel/git/will/kvmtool.git/

It seems like Pekka's old repository doesn't see many patches these days
any more.
I quickly checked and could apply your patches to the new repository by
just removing the tools/kvm path prefix.
So do you care to rebase your patches on top of Will's repository and
send them again (minus patch 2, which we have already in our branch
[1]). Please also see my comment on patch 1/3.

Thanks,
Andre.

[1]
https://git.kernel.org/cgit/linux/kernel/git/will/kvmtool.git/commit/?id=f83dc816a9c76f87ad90723f366700077fb367ea

> 
> Fan Du (3):
>   kvm-tool: Introduce downscript option
>   kvm-tool: Pass address of pointer to ioctl for tap
>   kvm-tool: Restrict queue number to 1 when vhost on
> 
>  tools/kvm/include/kvm/virtio-net.h |  1 +
>  tools/kvm/virtio/net.c             | 34 +++++++++++++++++++++++++++++++++-
>  2 files changed, 34 insertions(+), 1 deletion(-)
> 