[virtio-comment] Re: [virtio] Re: [virtio-comment] [PATCH requirements 7/7] net-features: Add header data split requirements

[David Edmondson <david.edmondson@oracle.com>]

On Monday, 2023-08-14 at 13:28:52 UTC, Parav Pandit wrote:
>> From: David Edmondson <david.edmondson@oracle.com>
>> Sent: Monday, August 14, 2023 6:40 PM
>
>> >> > +5. The device should be able to zero pad the header buffer when the
>> received
>> >> > +   header is shorter than cpu cache line size.
>> >>
>> >> What's the use case for this (item 5)?
>> >
>> > Without zero padding, each header write results in a
>> > read-modify-write, possibly over PCIe. That can significantly depress
>> > throughput.
>> 
>> Understood. So it could be anything padding, we just want to write a full cache
>> line.
> Yes. if the data descriptor is partial, need to zero fill too.
> I will double check if I covered already or not.

Perhaps the requirement, then, should be that the device is permitted
(and encouraged) to write full cache lines, with an aside that zero
padding should be used to achieve this.
-- 
No proper time of day.

This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.

In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.

Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/