[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: Re: [virtio-comment] Re: [PATCH v18] virtio-net: support inner header hash
On Wed, Jun 21, 2023 at 07:28:33PM +0000, Parav Pandit wrote: > > > > From: Michael S. Tsirkin <mst@redhat.com> > > Sent: Wednesday, June 21, 2023 3:26 PM > > > > On Wed, Jun 21, 2023 at 05:52:28PM +0000, Parav Pandit wrote: > > > > > > > From: Heng Qi <hengqi@linux.alibaba.com> > > > > Sent: Wednesday, June 21, 2023 12:46 PM > > > > > > > > > > SET carries an additional 32 bits of information. But if you think > > > > this will make the overall structure more concise, I'm ok. > > > > > > > If it is placed in single structure than it needs to be reworded to remove WO > > or RO notion. > > > > Not really. all of structure is RO and WO. > > > For the device, what is the meaning of driver writing supported_tunnel_types in SET command? > Ignore? It doesn't make sense to pass something to ignore. It can be helpful for debugging. E.g. if driver sets a bit that device does not recognize, it can figure out that driver has it set in supported mask. Or if it seems like a waste we can just ask drivers to put 0 there, works for me too. > Two structures are cleaner. Frankly what's cleaner is what RSS is doing, supported mask in config space. Was this discussed in some version? I don't remember. > > > This also requires additional sw to indicate dma attributes to be RW when > > mapping this area. > > > And extra text to indicate that supported_hash_tunnel_types to be ignored > > on set command. > > > > > > Two structures are more cleaner serving its purpose. > > > > No because all of structure is RO or WO. > > 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/
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]