Re: [PATCH next-next-2.6 v2] virtio_net: missing sg_init_table

From: David Miller
Date: Wed Mar 31 2010 - 06:16:44 EST


From: "Michael S. Tsirkin" <mst@xxxxxxxxxx>
Date: Wed, 31 Mar 2010 12:20:22 +0300

> On Mon, Mar 29, 2010 at 06:19:15PM -0700, Shirley Ma wrote:
>> Add missing sg_init_table for sg_set_buf in virtio_net which
>> induced in defer skb patch.
>>
>> Reported-by: Thomas Müller <thomas@xxxxxxxxx>
>> Tested-by: Thomas Müller <thomas@xxxxxxxxx>
>> Signed-off-by: Shirley Ma <xma@xxxxxxxxxx>
>
> I'm concerned that the 'big' path might cause a performance regression.
> Let's move sg into virtnet_info so that this needs to be only called
> once?

Yeah that might improve things.

Shirley's change is already in net-next-2.6 so anything implementing
this would need to be submitted relative to that.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/