Skip site navigation (1)Skip section navigation (2) 
Date:      Sat, 31 Dec 2011 20:15:46 +0000 (UTC)
From:      Matthew D Fleming <mdf@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-8@freebsd.org
Subject:   svn commit: r229154 - stable/8/sys/dev/ixgbe
Message-ID:  <201112312015.pBVKFkpd081427@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help 
Author: mdf
Date: Sat Dec 31 20:15:46 2011
New Revision: 229154
URL: http://svn.freebsd.org/changeset/base/229154

Log:
  MFC r228440:
  
  Consistently use types in ixgbe driver code:
  
   - {ixgbe,ixv}_header_split is passed to TUNABLE_INT, so delcare it
     int, not bool.
   - {ixgbe,ixv}_tx_ctx_setup() returns a boolean value, so declare it
     bool, not int.
   - {ixgbe,ixv}_tso_setup() returns a bool, so declare it bool, not boolean_t.
   - {ixgbe,ixv}_txeof() returns a bool, so declare it bool, not boolean_t.
   - Do not re-define bool if the symbol already exists.

Modified:
  stable/8/sys/dev/ixgbe/ixgbe.c
  stable/8/sys/dev/ixgbe/ixgbe_osdep.h
  stable/8/sys/dev/ixgbe/ixv.c
Directory Properties:
  stable/8/sys/   (props changed)

Modified: stable/8/sys/dev/ixgbe/ixgbe.c
==============================================================================
--- stable/8/sys/dev/ixgbe/ixgbe.c	Sat Dec 31 20:12:26 2011	(r229153)
+++ stable/8/sys/dev/ixgbe/ixgbe.c	Sat Dec 31 20:15:46 2011	(r229154)
@@ -264,7 +264,7 @@ TUNABLE_INT("hw.ixgbe.enable_msix", &ixg
  * it can be a performance win in some workloads, but
  * in others it actually hurts, its off by default. 
  */
-static bool ixgbe_header_split = FALSE;
+static int ixgbe_header_split = FALSE;
 TUNABLE_INT("hw.ixgbe.hdr_split", &ixgbe_header_split);
 
 /*
@@ -2990,7 +2990,7 @@ ixgbe_free_transmit_buffers(struct tx_ri
  *
  **********************************************************************/
 
-static boolean_t
+static bool
 ixgbe_tx_ctx_setup(struct tx_ring *txr, struct mbuf *mp)
 {
 	struct adapter *adapter = txr->adapter;
@@ -3108,7 +3108,7 @@ ixgbe_tx_ctx_setup(struct tx_ring *txr, 
  *  adapters using advanced tx descriptors
  *
  **********************************************************************/
-static boolean_t
+static bool
 ixgbe_tso_setup(struct tx_ring *txr, struct mbuf *mp, u32 *paylen)
 {
 	struct adapter *adapter = txr->adapter;
@@ -3272,7 +3272,7 @@ ixgbe_atr(struct tx_ring *txr, struct mb
  *  tx_buffer is put back on the free queue.
  *
  **********************************************************************/
-static boolean_t
+static bool
 ixgbe_txeof(struct tx_ring *txr)
 {
 	struct adapter	*adapter = txr->adapter;

Modified: stable/8/sys/dev/ixgbe/ixgbe_osdep.h
==============================================================================
--- stable/8/sys/dev/ixgbe/ixgbe_osdep.h	Sat Dec 31 20:12:26 2011	(r229153)
+++ stable/8/sys/dev/ixgbe/ixgbe_osdep.h	Sat Dec 31 20:15:46 2011	(r229154)
@@ -95,7 +95,9 @@ typedef uint16_t	u16;
 typedef uint32_t	u32;
 typedef int32_t		s32;
 typedef uint64_t	u64;
+#ifndef __bool_true_false_are_defined
 typedef boolean_t	bool;
+#endif
 
 #define le16_to_cpu 
 

Modified: stable/8/sys/dev/ixgbe/ixv.c
==============================================================================
--- stable/8/sys/dev/ixgbe/ixv.c	Sat Dec 31 20:12:26 2011	(r229153)
+++ stable/8/sys/dev/ixgbe/ixv.c	Sat Dec 31 20:15:46 2011	(r229154)
@@ -206,7 +206,7 @@ TUNABLE_INT("hw.ixv.flow_control", &ixv_
  * it can be a performance win in some workloads, but
  * in others it actually hurts, its off by default.
  */
-static bool ixv_header_split = FALSE;
+static int ixv_header_split = FALSE;
 TUNABLE_INT("hw.ixv.hdr_split", &ixv_header_split);
 
 /*
@@ -2336,7 +2336,7 @@ ixv_free_transmit_buffers(struct tx_ring
  *
  **********************************************************************/
 
-static boolean_t
+static bool
 ixv_tx_ctx_setup(struct tx_ring *txr, struct mbuf *mp)
 {
 	struct adapter *adapter = txr->adapter;
@@ -2459,7 +2459,7 @@ ixv_tx_ctx_setup(struct tx_ring *txr, st
  *  adapters using advanced tx descriptors
  *
  **********************************************************************/
-static boolean_t
+static bool
 ixv_tso_setup(struct tx_ring *txr, struct mbuf *mp, u32 *paylen)
 {
 	struct adapter *adapter = txr->adapter;
@@ -2548,7 +2548,7 @@ ixv_tso_setup(struct tx_ring *txr, struc
  *  tx_buffer is put back on the free queue.
  *
  **********************************************************************/
-static boolean_t
+static bool
 ixv_txeof(struct tx_ring *txr)
 {
 	struct adapter	*adapter = txr->adapter;

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201112312015.pBVKFkpd081427>