[ofa-general] Re: [PATCH 3/10] IB/core: Add LSO support
Roland Dreier
rdreier at cisco.com
Mon Mar 24 15:21:50 PDT 2008
> @@ -622,7 +624,8 @@ enum ib_wr_opcode {
> + IB_WR_LSO
> @@ -630,7 +633,8 @@ enum ib_send_flags {
> + IB_SEND_UDP_LSO = (1<<5)
What is this send flag used for? It doesn't seem to appear anywhere
else in the patch series and it seems redundant given that you've
defined a new opcode for LSO send requests.
I did a quick grep of the OFED 1.3 kernel tree and IB_SEND_UDP_LSO
only seems to appear once, in the line where it is defined. So I
guess it's OK to drop this chunk of the patch?
- R.
More information about the general
mailing list