#ifdef out the definition for the small packet RX ring. I ended up only

using one RX ring because of the alignment issue, so we may as well save
a few K of memory by not allocating space for it.

1 files changed, 9 insertions, 0 deletions

diff --git a/sys/dev/sf/if_sfreg.h b/sys/dev/sf/if_sfreg.h
index b9a8b9d..58f44a7 100644
--- a/sys/dev/sf/if_sfreg.h
+++ b/sys/dev/sf/if_sfreg.h
@@ -1004,7 +1004,16 @@ struct sf_list_data {
 	struct sf_tx_bufdesc_type0	sf_tx_dlist[SF_TX_DLIST_CNT];
 	struct sf_tx_cmpdesc_type1	sf_tx_clist[SF_TX_CLIST_CNT];
 	struct sf_rx_bufdesc_type0	sf_rx_dlist_big[SF_RX_DLIST_CNT];
+#ifdef notdef
+	/*
+	 * Unfortunately, because the Starfire doesn't allow arbitrary
+	 * byte alignment, we have to copy packets in the RX handler in
+	 * order to align the payload correctly. This means that we
+	 * don't gain anything by having separate large and small descriptor
+	 * lists, so for now we don't bother with the small one.
+	 */
 	struct sf_rx_bufdesc_type0	sf_rx_dlist_small[SF_RX_DLIST_CNT];
+#endif
 	struct sf_rx_cmpdesc_type3	sf_rx_clist[SF_RX_CLIST_CNT];
 };