Re: [PATCH v2 3/3] tun: Set num_buffers for virtio 1.0

From: Michael S. Tsirkin
Date: Fri Jan 10 2025 - 05:24:18 EST


On Fri, Jan 10, 2025 at 11:27:13AM +0800, Jason Wang wrote:
> On Thu, Jan 9, 2025 at 2:59 PM Akihiko Odaki <akihiko.odaki@xxxxxxxxxx> wrote:
> >
> > The specification says the device MUST set num_buffers to 1 if
> > VIRTIO_NET_F_MRG_RXBUF has not been negotiated.
>
> Have we agreed on how to fix the spec or not?
>
> As I replied in the spec patch, if we just remove this "MUST", it
> looks like we are all fine?
>
> Thanks

We should replace MUST with SHOULD but it is not all fine,
ignoring SHOULD is a quality of implementation issue.