On 08/19/16 02:50, Pyun YongHyeon wrote:
Modified: head/sys/dev/usb/net/if_axgereg.h
==============================================================================
--- head/sys/dev/usb/net/if_axgereg.h   Fri Aug 19 00:03:41 2016        
(r304438)
+++ head/sys/dev/usb/net/if_axgereg.h   Fri Aug 19 00:50:32 2016        
(r304439)
@@ -156,19 +156,20 @@ enum {
 struct axge_frame_txhdr {
 #if BYTE_ORDER == LITTLE_ENDIAN
        uint32_t                len;
-#define        AXGE_TXLEN_MASK         0x0001FFFF
-#define        AXGE_VLAN_INSERT        0x20000000
-#define        AXGE_CSUM_DISABLE       0x80000000
        uint32_t                mss;
-#define        AXGE_MSS_MASK           0x00003FFF
-#define        AXGE_PADDING            0x80008000
-#define        AXGE_VLAN_TAG_MASK      0xFFFF0000
 #else
        uint32_t                mss;
        uint32_t                len;
 #endif
 } __packed;


Hi,

Is it correct to switch the order of mss and len variables for bit/little endian? Looks buggy to me.

--HPS
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to