Re: [PATCH net-next v4 08/15] virtio_net: sq support premapped mode

2024-06-12 Thread Xuan Zhuo
On Wed, 12 Jun 2024 19:41:51 -0400, "Michael S. Tsirkin" wrote: > On Tue, Jun 11, 2024 at 07:41:40PM +0800, Xuan Zhuo wrote: > > If the xsk is enabling, the xsk tx will share the send queue. > > But the xsk requires that the send queue use the premapped mode. > > So the send queue must support pr

Re: [PATCH net-next v4 08/15] virtio_net: sq support premapped mode

2024-06-12 Thread Xuan Zhuo
On Wed, 12 Jun 2024 16:23:37 -0700, Jakub Kicinski wrote: > On Tue, 11 Jun 2024 19:41:40 +0800 Xuan Zhuo wrote: > > +static int virtnet_sq_set_premapped(struct send_queue *sq, bool premapped) > > Could you try to add __maybe_unused or some such and then remove it > in the patch which calls this fu

Re: [PATCH net-next v4 08/15] virtio_net: sq support premapped mode

2024-06-12 Thread Michael S. Tsirkin
On Tue, Jun 11, 2024 at 07:41:40PM +0800, Xuan Zhuo wrote: > If the xsk is enabling, the xsk tx will share the send queue. > But the xsk requires that the send queue use the premapped mode. > So the send queue must support premapped mode when it is bound to > af-xdp. > > * virtnet_sq_set_premapped

Re: [PATCH net-next v4 08/15] virtio_net: sq support premapped mode

2024-06-12 Thread Jakub Kicinski
On Tue, 11 Jun 2024 19:41:40 +0800 Xuan Zhuo wrote: > +static int virtnet_sq_set_premapped(struct send_queue *sq, bool premapped) Could you try to add __maybe_unused or some such and then remove it in the patch which calls this function? Having warnings during bisection is not great.

[PATCH net-next v4 08/15] virtio_net: sq support premapped mode

2024-06-11 Thread Xuan Zhuo
If the xsk is enabling, the xsk tx will share the send queue. But the xsk requires that the send queue use the premapped mode. So the send queue must support premapped mode when it is bound to af-xdp. * virtnet_sq_set_premapped(sq, true) is used to enable premapped mode. In this mode, the dri