[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v4 1/3] public/io/netif.h: document transmit and receive wire formats separately
> -----Original Message----- [snip] > > Â * Guest transmit > > Â * ============== > > Â * > > + * This is the 'wire' format for packets: > > + *ÂÂFragment 1: netif_tx_request_tÂÂ- flags = NETTXF_* > > + *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂsize = total packet size > > + * [Extra 1: netif_extra_info_t]ÂÂÂÂ- (only if fragment 1 flags include > > + *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂNETTXF_extra_info) > > + * [Extra N: netif_extra_info_t]ÂÂÂÂ- (only if extra N-1 flags include > > + *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂXEN_NETIF_EXTRA_MORE) > > + *ÂÂ... > > + *ÂÂFragment N: netif_tx_request_tÂÂ- (only if fragment N-1 flags include > > + *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂNETTXF_more_data) > > For Fragment 2 is it the Flags of Fragment N-1 = 1 which are relevant, or > the flags in the optional Extra N which may be in the middle (i.e. the > immediately preceding slot)? It's Fragment N-1's flags. The flags on the Extras are not relevant. In fact the only Extra flag defined is the one that says there's another Extra :-) > > Am I correct in remembering that in the presence of NETTXF_more_data the > only way to know the actual size of Fragment 1 is to take away the total of > all the extras from Frag 1's size? That's right. > > > + *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂflags = 0 > > + *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂsize = fragment size > > + * > > Â * Ring slot size is 12 octets, however not all request/response > > Â * structs use the full size. > > Â * > > @@ -202,6 +202,19 @@ > > Â * Guest receive > > Â * ============= > > Â * > > + * This is the 'wire' format for packets: > > + *ÂÂFragment 1: netif_rx_request_tÂÂ- flags = NETRXF_* > > + *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂsize = fragment size > > + * [Extra 1: netif_extra_info_t]ÂÂÂÂ- (only if fragment 1 flags include > > + *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂNETRXF_extra_info) > > + * [Extra N: netif_extra_info_t]ÂÂÂÂ- (only if extra N-1 flags include > > + *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂXEN_NETIF_EXTRA_MORE) > > + *ÂÂ... > > + *ÂÂFragment N: netif_rx_request_tÂÂ- (only if fragment N-1 flags include > > + *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂNETRXF_more_data) > > + *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂflags = 0 > > + *ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂsize = fragment size > > Same Q re which NETRXF_more_data is relevant. > Same answer :-) > In this path there is no indication of the total packet size other than > adding everything up? > Correct. > Given that they differ in a subtle way would a quick but explicit "NOTE: RX > and TX differ" be a useful addition do you think? > Yes, that's probably a good plan. I'll stick an extra comment in to that effect. Paul > > + * > > Â * Ring slot size is 8 octets. > > Â * > > Â * rx request (netif_rx_request_t) _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |