public inbox for linux-rdma@vger.kernel.org
 help / color / mirror / Atom feed
* re: RDMA/nes: Add support for MPAv2 Enhanced RDMA Negotiation
@ 2012-01-13 10:05 Dan Carpenter
  0 siblings, 0 replies; only message in thread
From: Dan Carpenter @ 2012-01-13 10:05 UTC (permalink / raw)
  To: Tatyana.E.Nikolova-ral2JQCrhuEAvxtiuMwx3w
  Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA

Hello Tatyana Nikolova,

The patch 615eb715ae10: "RDMA/nes: Add support for MPAv2 Enhanced 
RDMA Negotiation" from Sep 25, 2011, leads to a number of endian
warnings.

For example, struct ietf_rtr_msg is declared as holding big endian
values, but it's used to hold cpu endian values.

+struct ietf_rtr_msg {
+       __be16 ctrl_ird;
+       __be16 ctrl_ord;
+};

Sparse can find many of these problems automatically.  Lwn had an
article on how that works.  http://lwn.net/Articles/205624/

regards,
dan carpenter

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2012-01-13 10:05 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-13 10:05 RDMA/nes: Add support for MPAv2 Enhanced RDMA Negotiation Dan Carpenter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox