Author: adrian
Date: Mon Jul 23 03:52:18 2012
New Revision: 238710
URL: http://svn.freebsd.org/changeset/base/238710

Log:
  Begin separating out the TX DMA setup in preparation for TX EDMA support.
  
  * Introduce TX DMA setup/teardown methods, mirroring what's done in
    the RX path.
  
    Although the TX DMA descriptor is setup via ath_desc_alloc() /
    ath_desc_free(), there TX status descriptor ring will be allocated
    in this path.
  
  * Remove some of the TX EDMA capability probing from the RX path and
    push it into the new TX EDMA path.

Added:
  head/sys/dev/ath/if_ath_tx_edma.c   (contents, props changed)
  head/sys/dev/ath/if_ath_tx_edma.h   (contents, props changed)
Modified:
  head/sys/conf/files
  head/sys/dev/ath/if_ath.c
  head/sys/dev/ath/if_ath_rx_edma.c
  head/sys/dev/ath/if_ath_tx.c
  head/sys/dev/ath/if_ath_tx.h
  head/sys/dev/ath/if_ath_tx_ht.c
  head/sys/dev/ath/if_athvar.h
  head/sys/modules/ath/Makefile

Modified: head/sys/conf/files
==============================================================================
--- head/sys/conf/files Mon Jul 23 02:49:25 2012        (r238709)
+++ head/sys/conf/files Mon Jul 23 03:52:18 2012        (r238710)
@@ -729,6 +729,8 @@ dev/ath/if_ath_led.c                optional ath \
        compile-with "${NORMAL_C} -I$S/dev/ath"
 dev/ath/if_ath_tx.c            optional ath \
        compile-with "${NORMAL_C} -I$S/dev/ath"
+dev/ath/if_ath_tx_edma.c       optional ath \
+       compile-with "${NORMAL_C} -I$S/dev/ath"
 dev/ath/if_ath_tx_ht.c         optional ath \
        compile-with "${NORMAL_C} -I$S/dev/ath"
 dev/ath/if_ath_tdma.c          optional ath \

Modified: head/sys/dev/ath/if_ath.c
==============================================================================
--- head/sys/dev/ath/if_ath.c   Mon Jul 23 02:49:25 2012        (r238709)
+++ head/sys/dev/ath/if_ath.c   Mon Jul 23 03:52:18 2012        (r238710)
@@ -109,6 +109,7 @@ __FBSDID("$FreeBSD$");
 #include <dev/ath/if_ath_keycache.h>
 #include <dev/ath/if_ath_rx.h>
 #include <dev/ath/if_ath_rx_edma.h>
+#include <dev/ath/if_ath_tx_edma.h>
 #include <dev/ath/if_ath_beacon.h>
 #include <dev/ath/if_athdfs.h>
 
@@ -306,8 +307,11 @@ ath_attach(u_int16_t devid, struct ath_s
        if (ath_hal_hasedma(sc->sc_ah)) {
                sc->sc_isedma = 1;
                ath_recv_setup_edma(sc);
-       } else
+               ath_xmit_setup_edma(sc);
+       } else {
                ath_recv_setup_legacy(sc);
+               ath_xmit_setup_legacy(sc);
+       }
 
        /*
         * Check if the MAC has multi-rate retry support.
@@ -367,14 +371,24 @@ ath_attach(u_int16_t devid, struct ath_s
        ath_setcurmode(sc, IEEE80211_MODE_11A);
 
        /*
-        * Allocate tx+rx descriptors and populate the lists.
+        * Allocate TX descriptors and populate the lists.
         */
        error = ath_desc_alloc(sc);
        if (error != 0) {
-               if_printf(ifp, "failed to allocate descriptors: %d\n", error);
+               if_printf(ifp, "failed to allocate TX descriptors: %d\n",
+                   error);
+               goto bad;
+       }
+       error = ath_txdma_setup(sc);
+       if (error != 0) {
+               if_printf(ifp, "failed to allocate TX descriptors: %d\n",
+                   error);
                goto bad;
        }
 
+       /*
+        * Allocate RX descriptors and populate the lists.
+        */
        error = ath_rxdma_setup(sc);
        if (error != 0) {
                if_printf(ifp, "failed to allocate RX descriptors: %d\n",
@@ -858,6 +872,7 @@ ath_attach(u_int16_t devid, struct ath_s
 bad2:
        ath_tx_cleanup(sc);
        ath_desc_free(sc);
+       ath_txdma_teardown(sc);
        ath_rxdma_teardown(sc);
 bad:
        if (ah)

Modified: head/sys/dev/ath/if_ath_rx_edma.c
==============================================================================
--- head/sys/dev/ath/if_ath_rx_edma.c   Mon Jul 23 02:49:25 2012        
(r238709)
+++ head/sys/dev/ath/if_ath_rx_edma.c   Mon Jul 23 03:52:18 2012        
(r238710)
@@ -828,9 +828,6 @@ ath_recv_setup_edma(struct ath_softc *sc
 
        /* Fetch EDMA field and buffer sizes */
        (void) ath_hal_getrxstatuslen(sc->sc_ah, &sc->sc_rx_statuslen);
-       (void) ath_hal_gettxdesclen(sc->sc_ah, &sc->sc_tx_desclen);
-       (void) ath_hal_gettxstatuslen(sc->sc_ah, &sc->sc_tx_statuslen);
-       (void) ath_hal_getntxmaps(sc->sc_ah, &sc->sc_tx_nmaps);
 
        /* Configure the hardware with the RX buffer size */
        (void) ath_hal_setrxbufsize(sc->sc_ah, sc->sc_edma_bufsize -
@@ -838,14 +835,8 @@ ath_recv_setup_edma(struct ath_softc *sc
 
        device_printf(sc->sc_dev, "RX status length: %d\n",
            sc->sc_rx_statuslen);
-       device_printf(sc->sc_dev, "TX descriptor length: %d\n",
-           sc->sc_tx_desclen);
-       device_printf(sc->sc_dev, "TX status length: %d\n",
-           sc->sc_tx_statuslen);
-       device_printf(sc->sc_dev, "TX/RX buffer size: %d\n",
+       device_printf(sc->sc_dev, "RX buffer size: %d\n",
            sc->sc_edma_bufsize);
-       device_printf(sc->sc_dev, "TX buffers per descriptor: %d\n",
-           sc->sc_tx_nmaps);
 
        sc->sc_rx.recv_stop = ath_edma_stoprecv;
        sc->sc_rx.recv_start = ath_edma_startrecv;

Modified: head/sys/dev/ath/if_ath_tx.c
==============================================================================
--- head/sys/dev/ath/if_ath_tx.c        Mon Jul 23 02:49:25 2012        
(r238709)
+++ head/sys/dev/ath/if_ath_tx.c        Mon Jul 23 03:52:18 2012        
(r238710)
@@ -4463,3 +4463,27 @@ ath_addba_response_timeout(struct ieee80
        ath_tx_tid_resume(sc, atid);
        ATH_TXQ_UNLOCK(sc->sc_ac2q[atid->ac]);
 }
+
+static int
+ath_legacy_dma_txsetup(struct ath_softc *sc)
+{
+
+       /* nothing new needed */
+       return (0);
+}
+
+static int
+ath_legacy_dma_txteardown(struct ath_softc *sc)
+{
+
+       /* nothing new needed */
+       return (0);
+}
+
+void
+ath_xmit_setup_legacy(struct ath_softc *sc)
+{
+
+       sc->sc_tx.xmit_setup = ath_legacy_dma_txsetup;
+       sc->sc_tx.xmit_teardown = ath_legacy_dma_txteardown;
+}

Modified: head/sys/dev/ath/if_ath_tx.h
==============================================================================
--- head/sys/dev/ath/if_ath_tx.h        Mon Jul 23 02:49:25 2012        
(r238709)
+++ head/sys/dev/ath/if_ath_tx.h        Mon Jul 23 03:52:18 2012        
(r238710)
@@ -124,4 +124,13 @@ extern     void ath_bar_response(struct ieee
 extern void ath_addba_response_timeout(struct ieee80211_node *ni,
     struct ieee80211_tx_ampdu *tap);
 
+/*
+ * Setup path
+ */
+#define        ath_txdma_setup(_sc)                    \
+       (_sc)->sc_tx.xmit_setup(_sc)
+#define        ath_txdma_teardown(_sc)                 \
+       (_sc)->sc_tx.xmit_teardown(_sc)
+extern void ath_xmit_setup_legacy(struct ath_softc *sc);
+
 #endif

Added: head/sys/dev/ath/if_ath_tx_edma.c
==============================================================================
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/sys/dev/ath/if_ath_tx_edma.c   Mon Jul 23 03:52:18 2012        
(r238710)
@@ -0,0 +1,162 @@
+/*-
+ * Copyright (c) 2012 Adrian Chadd <adr...@freebsd.org>
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer,
+ *    without modification.
+ * 2. Redistributions in binary form must reproduce at minimum a disclaimer
+ *    similar to the "NO WARRANTY" disclaimer below ("Disclaimer") and any
+ *    redistribution must be conditioned upon including a substantially
+ *    similar Disclaimer requirement for further binary redistribution.
+ *
+ * NO WARRANTY
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF NONINFRINGEMENT, MERCHANTIBILITY
+ * AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL
+ * THE COPYRIGHT HOLDERS OR CONTRIBUTORS BE LIABLE FOR SPECIAL, EXEMPLARY,
+ * OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER
+ * IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
+ * THE POSSIBILITY OF SUCH DAMAGES.
+ */
+
+#include <sys/cdefs.h>
+__FBSDID("$FreeBSD$");
+
+/*
+ * Driver for the Atheros Wireless LAN controller.
+ *
+ * This software is derived from work of Atsushi Onoe; his contribution
+ * is greatly appreciated.
+ */
+
+#include "opt_inet.h"
+#include "opt_ath.h"
+/*
+ * This is needed for register operations which are performed
+ * by the driver - eg, calls to ath_hal_gettsf32().
+ *
+ * It's also required for any AH_DEBUG checks in here, eg the
+ * module dependencies.
+ */
+#include "opt_ah.h"
+#include "opt_wlan.h"
+
+#include <sys/param.h>
+#include <sys/systm.h>
+#include <sys/sysctl.h>
+#include <sys/mbuf.h>
+#include <sys/malloc.h>
+#include <sys/lock.h>
+#include <sys/mutex.h>
+#include <sys/kernel.h>
+#include <sys/socket.h>
+#include <sys/sockio.h>
+#include <sys/errno.h>
+#include <sys/callout.h>
+#include <sys/bus.h>
+#include <sys/endian.h>
+#include <sys/kthread.h>
+#include <sys/taskqueue.h>
+#include <sys/priv.h>
+#include <sys/module.h>
+#include <sys/ktr.h>
+#include <sys/smp.h>   /* for mp_ncpus */
+
+#include <machine/bus.h>
+
+#include <net/if.h>
+#include <net/if_dl.h>
+#include <net/if_media.h>
+#include <net/if_types.h>
+#include <net/if_arp.h>
+#include <net/ethernet.h>
+#include <net/if_llc.h>
+
+#include <net80211/ieee80211_var.h>
+#include <net80211/ieee80211_regdomain.h>
+#ifdef IEEE80211_SUPPORT_SUPERG
+#include <net80211/ieee80211_superg.h>
+#endif
+#ifdef IEEE80211_SUPPORT_TDMA
+#include <net80211/ieee80211_tdma.h>
+#endif
+
+#include <net/bpf.h>
+
+#ifdef INET
+#include <netinet/in.h>
+#include <netinet/if_ether.h>
+#endif
+
+#include <dev/ath/if_athvar.h>
+#include <dev/ath/ath_hal/ah_devid.h>          /* XXX for softled */
+#include <dev/ath/ath_hal/ah_diagcodes.h>
+
+#include <dev/ath/if_ath_debug.h>
+#include <dev/ath/if_ath_misc.h>
+#include <dev/ath/if_ath_tsf.h>
+#include <dev/ath/if_ath_tx.h>
+#include <dev/ath/if_ath_sysctl.h>
+#include <dev/ath/if_ath_led.h>
+#include <dev/ath/if_ath_keycache.h>
+#include <dev/ath/if_ath_rx.h>
+#include <dev/ath/if_ath_beacon.h>
+#include <dev/ath/if_athdfs.h>
+
+#ifdef ATH_TX99_DIAG
+#include <dev/ath/ath_tx99/ath_tx99.h>
+#endif
+
+#include <dev/ath/if_ath_tx_edma.h>
+
+/*
+ * some general macros
+  */
+#define        INCR(_l, _sz)           (_l) ++; (_l) &= ((_sz) - 1)
+#define        DECR(_l, _sz)           (_l) --; (_l) &= ((_sz) - 1)
+
+MALLOC_DECLARE(M_ATHDEV);
+
+static int
+ath_edma_dma_txsetup(struct ath_softc *sc)
+{
+
+       /* XXX placeholder */
+       return (0);
+}
+
+static int
+ath_edma_dma_txteardown(struct ath_softc *sc)
+{
+
+       /* XXX placeholder */
+       return (0);
+}
+
+void
+ath_xmit_setup_edma(struct ath_softc *sc)
+{
+
+       /* Fetch EDMA field and buffer sizes */
+       (void) ath_hal_gettxdesclen(sc->sc_ah, &sc->sc_tx_desclen);
+       (void) ath_hal_gettxstatuslen(sc->sc_ah, &sc->sc_tx_statuslen);
+       (void) ath_hal_getntxmaps(sc->sc_ah, &sc->sc_tx_nmaps);
+
+       device_printf(sc->sc_dev, "TX descriptor length: %d\n",
+           sc->sc_tx_desclen);
+       device_printf(sc->sc_dev, "TX status length: %d\n",
+           sc->sc_tx_statuslen);
+       device_printf(sc->sc_dev, "TX buffers per descriptor: %d\n",
+           sc->sc_tx_nmaps);
+
+       sc->sc_tx.xmit_setup = ath_edma_dma_txsetup;
+       sc->sc_tx.xmit_teardown = ath_edma_dma_txteardown;
+}

Added: head/sys/dev/ath/if_ath_tx_edma.h
==============================================================================
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/sys/dev/ath/if_ath_tx_edma.h   Mon Jul 23 03:52:18 2012        
(r238710)
@@ -0,0 +1,36 @@
+/*-
+ * Copyright (c) 2012 Adrian Chadd <adr...@freebsd.org>
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer,
+ *    without modification.
+ * 2. Redistributions in binary form must reproduce at minimum a disclaimer
+ *    similar to the "NO WARRANTY" disclaimer below ("Disclaimer") and any
+ *    redistribution must be conditioned upon including a substantially
+ *    similar Disclaimer requirement for further binary redistribution.
+ *
+ * NO WARRANTY
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF NONINFRINGEMENT, MERCHANTIBILITY
+ * AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL
+ * THE COPYRIGHT HOLDERS OR CONTRIBUTORS BE LIABLE FOR SPECIAL, EXEMPLARY,
+ * OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER
+ * IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
+ * THE POSSIBILITY OF SUCH DAMAGES.
+ *
+ * $FreeBSD$
+ */
+#ifndef        __IF_ATH_TX_EDMA_H__
+#define        __IF_ATH_TX_EDMA_H__
+
+extern void ath_xmit_setup_edma(struct ath_softc *sc);
+
+#endif

Modified: head/sys/dev/ath/if_ath_tx_ht.c
==============================================================================
--- head/sys/dev/ath/if_ath_tx_ht.c     Mon Jul 23 02:49:25 2012        
(r238709)
+++ head/sys/dev/ath/if_ath_tx_ht.c     Mon Jul 23 03:52:18 2012        
(r238710)
@@ -511,6 +511,8 @@ ath_rateseries_setup(struct ath_softc *s
                        series[i].RateFlags |= HAL_RATESERIES_HALFGI;
 
                series[i].Rate = rt->info[rc[i].rix].rateCode;
+               series[i].RateIndex = rc[i].rix;
+               series[i].tx_power_cap = 0x3f;  /* XXX? */
 
                /*
                 * PktDuration doesn't include slot, ACK, RTS, etc timing -

Modified: head/sys/dev/ath/if_athvar.h
==============================================================================
--- head/sys/dev/ath/if_athvar.h        Mon Jul 23 02:49:25 2012        
(r238709)
+++ head/sys/dev/ath/if_athvar.h        Mon Jul 23 03:52:18 2012        
(r238710)
@@ -397,6 +397,11 @@ struct ath_rx_edma {
        struct mbuf     *m_rxpending;
 };
 
+struct ath_tx_methods {
+       int             (*xmit_setup)(struct ath_softc *sc);
+       int             (*xmit_teardown)(struct ath_softc *sc);
+};
+
 struct ath_softc {
        struct ifnet            *sc_ifp;        /* interface common */
        struct ath_stats        sc_stats;       /* interface statistics */
@@ -412,6 +417,8 @@ struct ath_softc {
 
        struct ath_rx_methods   sc_rx;
        struct ath_rx_edma      sc_rxedma[HAL_NUM_RX_QUEUES];   /* HP/LP queues 
*/
+       struct ath_tx_methods   sc_tx;
+
        int                     sc_rx_statuslen;
        int                     sc_tx_desclen;
        int                     sc_tx_statuslen;

Modified: head/sys/modules/ath/Makefile
==============================================================================
--- head/sys/modules/ath/Makefile       Mon Jul 23 02:49:25 2012        
(r238709)
+++ head/sys/modules/ath/Makefile       Mon Jul 23 03:52:18 2012        
(r238710)
@@ -37,7 +37,7 @@ ATH_RATE?=    sample          # tx rate control alg
 KMOD=  if_ath
 SRCS=  if_ath.c if_ath_debug.c if_ath_keycache.c if_ath_sysctl.c
 SRCS+= if_ath_tx.c if_ath_tx_ht.c if_ath_led.c if_ath_rx.c if_ath_tdma.c
-SRCS+= if_ath_beacon.c if_ath_rx_edma.c
+SRCS+= if_ath_beacon.c if_ath_rx_edma.c if_ath_tx_edma.c
 # NB: v3 eeprom support used by both AR5211 and AR5212; just include it
 SRCS+= ah_osdep.c ah.c ah_regdomain.c ah_eeprom_v3.c
 SRCS+= device_if.h bus_if.h pci_if.h opt_inet.h opt_ath.h opt_ah.h opt_wlan.h
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to