On Mon, 14 Jul 2025 at 17:55, Peter Maydell <peter.mayd...@linaro.org> wrote: > > After the bug fix in the previous commit, the length and prev_buf_size > variables are identical, except that prev_buf_size is uint32_t and > length is uint16_t. We can therefore unify them. The only place where > the type makes a difference is that we will truncate the packet > at 64K when sending it; this commit preserves that behaviour > by using a local variable when doing the packet send. > > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > --- > Should the behaviour really be to truncate the packet at 64K > rather than flagging an error if the guest assembles descriptors > that combine to give a too-large packet?
I see Jason has picked this up in a pullreq, which is OK given this isn't a behaviour change -- but it would still be good if somebody with access to the datasheet for this device could confirm what the actual correct behaviour for attempts to tx very large packets should be. thanks -- PMM
