New hardware, old problem: stuck beacon when here is WiFi traffic

[Adrian Chadd]

.. how's this never triggered a clang warning on me before?

Just add casts for now or something so it compiles.




Adrian

On 28 April 2013 12:48, Lev Serebryakov <lev at freebsd.org> wrote:
> Hello, Adrian.
> You wrote 28 апреля 2013 г., 23:38:20:
>
> AC> There's some race condition hack that Sam threw in that gets enabled
> AC> only if you compile things with TDMA support enabled. Would you mind
> AC> compiling in TDMA support (add options IEEE80211_SUPPORT_TDMA) to your
> AC> kernel config and rebuild? I'd like to see if that TX queue workaround
> AC> is effective at helping us out here.
>  Oooopps... And, yes, it is really annoying, that there is no good way
>  to specify (u_)?int\d\d?_t in printf() :(
>
> cc -c -O2 -pipe -fno-strict-aliasing  -std=c99  -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual  -Wundef -Wno-pointer-sign -fformat-extensions  -Wmissing-include-dirs -fdiagnostics-show-option  -Wno-error-tautological-compare -Wno-error-empty-body  -Wno-error-parentheses-equality -nostdinc  -I. -I/data/src/sys -I/data/src/sys/contrib/altq -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h  -fno-omit-frame-pointer -mno-aes -mno-avx -mcmodel=kernel -mno-red-zone -mno-mmx -mno-sse -msoft-float  -fno-asynchronous-unwind-tables -ffreestanding -fstack-protector -Werror  /data/src/sys/dev/ath/if_ath_tdma.c -I/data/src/sys/dev/ath
> /data/src/sys/dev/ath/if_ath_tdma.c:442:42: error: format specifies type 'unsigned long long' but the argument has type 'u_int64_t' (aka 'unsigned long') [-Werror,-Wformat]
>             (unsigned long long) rs->rs_tstamp, rstamp, tsf, txtime,
>                                                 ^~~~~~
> /data/src/sys/dev/ath/if_ath_debug.h:97:34: note: expanded from macro 'DPRINTF'
>                 device_printf(sc->sc_dev, fmt, __VA_ARGS__);            \
>                                                ^
> /data/src/sys/dev/ath/if_ath_tdma.c:442:50: error: format specifies type 'unsigned long long' but the argument has type 'u_int64_t' (aka 'unsigned long') [-Werror,-Wformat]
>             (unsigned long long) rs->rs_tstamp, rstamp, tsf, txtime,
>                                                         ^~~
> /data/src/sys/dev/ath/if_ath_debug.h:97:34: note: expanded from macro 'DPRINTF'
>                 device_printf(sc->sc_dev, fmt, __VA_ARGS__);            \
>                                                ^
> /data/src/sys/dev/ath/if_ath_tdma.c:443:6: error: format specifies type 'unsigned long long' but the argument has type 'u_int64_t' (aka 'unsigned long') [-Werror,-Wformat]
>             nextslot, nextslottu, TSF_TO_TU(nextslot >> 32, nextslot));
>             ^~~~~~~~
> /data/src/sys/dev/ath/if_ath_debug.h:97:34: note: expanded from macro 'DPRINTF'
>                 device_printf(sc->sc_dev, fmt, __VA_ARGS__);            \
>                                                ^
> /data/src/sys/dev/ath/if_ath_tdma.c:446:6: error: format specifies type 'unsigned long long' but the argument has type 'uint64_t' (aka 'unsigned long') [-Werror,-Wformat]
>             le64toh(ni->ni_tstamp.tsf),
>             ^~~~~~~~~~~~~~~~~~~~~~~~~~~
> /data/src/sys/sys/endian.h:80:20: note: expanded from macro 'le64toh'
> #define le64toh(x)      ((uint64_t)(x))
>                         ^
> /data/src/sys/dev/ath/if_ath_debug.h:97:34: note: expanded from macro 'DPRINTF'
>                 device_printf(sc->sc_dev, fmt, __VA_ARGS__);            \
>                                                ^
> /data/src/sys/dev/ath/if_ath_tdma.c:447:6: error: format specifies type 'unsigned long long' but the argument has type 'uint64_t' (aka 'unsigned long') [-Werror,-Wformat]
>             le64toh(ni->ni_tstamp.tsf));
>             ^~~~~~~~~~~~~~~~~~~~~~~~~~~
> /data/src/sys/sys/endian.h:80:20: note: expanded from macro 'le64toh'
> #define le64toh(x)      ((uint64_t)(x))
>                         ^
> /data/src/sys/dev/ath/if_ath_debug.h:97:34: note: expanded from macro 'DPRINTF'
>                 device_printf(sc->sc_dev, fmt, __VA_ARGS__);            \
>                                                ^
> /data/src/sys/dev/ath/if_ath_tdma.c:451:6: error: format specifies type 'unsigned long long' but the argument has type 'u_int64_t' (aka 'unsigned long') [-Werror,-Wformat]
>             nexttbtt,
>             ^~~~~~~~
> /data/src/sys/dev/ath/if_ath_debug.h:97:34: note: expanded from macro 'DPRINTF'
>                 device_printf(sc->sc_dev, fmt, __VA_ARGS__);            \
>                                                ^
> /data/src/sys/dev/ath/if_ath_tdma.c:545:7: error: format specifies type 'unsigned long long' but the argument has type 'uint64_t' (aka 'unsigned long') [-Werror,-Wformat]
>                     tsf,
>                     ^~~
> /data/src/sys/dev/ath/if_ath_debug.h:97:34: note: expanded from macro 'DPRINTF'
>                 device_printf(sc->sc_dev, fmt, __VA_ARGS__);            \
>                                                ^
> 7 errors generated.
> *** [if_ath_tdma.o] Error code 1
> 1 error
> *** [buildkernel] Error code 2
> 1 error
> *** [buildkernel] Error code 2
> 1 error
>
>
>
> --
> // Black Lion AKA Lev Serebryakov <lev at FreeBSD.org>
>