Remove unnecessary list_head entry from recv_frame union

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Extending size of recv_frame does not guarantee alignment

Why oh why???? Setting a minimum size of the struct (which is purely an
internal kernel struct fwiw) does not guarantee alignment to the
minimum size when the struct is carved out of a big block of
memory from a single large allocation.

Remove this pointless entry in union recv_frame, which allows us to
remove the union completely and return to a plain struct (in the
follow-on patch).

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Remove union wrapping of recv_frame

This removes the pointless code obfuscation that was used to wrap
struct recv_frame.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Remove unused pkt_to_recv{frame,data,mem} calls

These functions all used the broken rxmem_to_recvframe() function,
which simply masked out the lower bits of the pointer based on invalid
assumptions.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Remove unused get_rxbuf_desc()

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Remove get_recvframe_len() call

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Remove unnecessary get_recvframe_data() function

All callers of get_recvframe_data() dereferences the recv_frame
pointer without checking it either just before or just after the
call, thus no point in carrying a wrapper function checking the
pointer before trying to dereferencing it.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>