On 2017年01月25日 12:02, John Fastabend wrote:

On 17-01-24 07:23 PM, Michael S. Tsirkin wrote:

quoted

On Wed, Jan 25, 2017 at 10:57:12AM +0800, Jason Wang wrote:

quoted

On 2017年01月25日 04:08, Michael S. Tsirkin wrote:

quoted

On Tue, Jan 24, 2017 at 02:43:28PM -0500, David Miller wrote:

quoted

From: "Michael S. Tsirkin"<mst@redhat.com>
Date: Mon, 23 Jan 2017 23:08:35 +0200

quoted

On Tue, Jan 17, 2017 at 02:19:50PM -0800, John Fastabend wrote:

quoted

In the small buffer case during driver unload we currently use
put_page instead of dev_kfree_skb. Resolve this by adding a check
for virtnet mode when checking XDP queue type. Also name the
function so that the code reads correctly to match the additional
check.

Fixes: bb91accf2733 ("virtio-net: XDP support for small buffers")
Signed-off-by: John Fastabend<redacted>
Acked-by: Jason Wang<jasowang@redhat.com>

Acked-by: Michael S. Tsirkin<mst@redhat.com>

I think we definitely want this one in -net as it's
a bugfix.

This whole series is a bug fix, we must have adjust_header XDP
support in the virtio_net driver before v4.10 goes out, it is
a requires base feature for XDP.

I have to say device resets outside probe have a huge potential
to uncover hypervisor bugs.

Maybe not if it reuses most of current codes? Since we've already used them
in sleep or hibernation?

Thanks

Except almost no one uses sleep or hybernate with VMs.  I'm not saying
it's a bad idea, just that it needs a lot of testing before release and
we won't get enough if we merge at this point.

Then it would seem like a good thing to have another user of these paths and
find the bugs versus letting them sit there for some poor folks who do use
sleep/hybernate.

Yes, and uncovering hypervisor bugs now is better than uncovering it in 
the future.

Thanks