Hi Bill,

On Fri, 24 Oct 2014 09:21:36 -0500 Bill Fischofer <[email protected]> 
wrote:
> Headroom (and tailroom) are meta data associated with packets and are not
> hard-coded like this.  See the ODP Packet Design doc for details.  Based on
> Tuesday's call I'm revising the doc to be clearer as to how this works so
> look for a follow-on post here later today.

Thanks.

Currently, 'odp_pktio_set_headroom' is not implemented for linux-generic.

Thus, ETHBUF_ALIGN is an (existing) implmentation detail used by
linux-generic's internal 'setup_pkt_sock':
It assists in calculating 'pkt_sock->frame_offset' for the socket.

This is later used when arming buffers for the recv call, e.g. in
recv_pkt_sock_basic:

                l2_hdr = pkt_buf + pkt_sock->frame_offset;

                recv_bytes = recvfrom(sockfd, l2_hdr,
                                      pkt_sock->max_frame_len, MSG_DONTWAIT,
                                      (struct sockaddr *)&sll, &addrlen);

So my motivation was modifying existing internal implementation detail
of linux-generic/odp_packet_socket.c ;)

Will linux-generic/odp_packet_socket.c be revised once
'odp_pktio_set_headroom' gets implemented for linux-generic?

Regards,
Shmulik

_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to