From owner-freebsd-wireless@FreeBSD.ORG Sun Apr 28 19:49:39 2013 Return-Path: Delivered-To: freebsd-wireless@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id DA59ACB9; Sun, 28 Apr 2013 19:49:39 +0000 (UTC) (envelope-from adrian.chadd@gmail.com) Received: from mail-wi0-x235.google.com (mail-wi0-x235.google.com [IPv6:2a00:1450:400c:c05::235]) by mx1.freebsd.org (Postfix) with ESMTP id 4556712DC; Sun, 28 Apr 2013 19:49:39 +0000 (UTC) Received: by mail-wi0-f181.google.com with SMTP id c10so2143461wiw.8 for ; Sun, 28 Apr 2013 12:49:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:x-received:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=HC0wgzl7tdwq9habT9crrzjr1qLKWRhFyEjvS9mW1Yo=; b=mNClSUyif+SjySHVrpX2nclV8Y6y/0eo1zkIOm1jTgiEp9m1kh+dT/tln9G00vsSIN JwUFmDb1oSK/iFaI4isFgrfWdub9O7bkSAbb9HOgPh7ZTf+1l61VZgGsWDa0YaVvIsI9 x0f8u8yzukhKdMLamwlmnif309x6qEtuN2WtvzHEEExopRAF5DdSdpiCnqXA4Xr4L3fX u7LwAp4F03B9yd89PAGPyUShTcihwd+CGzXP2nhQbe49hFxCaaE602aV/nsbBfLpdUkp Rb5ovNZnVt75o+B1ZSR6vJg07SUywmK8UybLc9uZ8JWOqnqGJ+rbwOwmXpQaPWdg3D8s 5cLA== MIME-Version: 1.0 X-Received: by 10.194.119.33 with SMTP id kr1mr11527606wjb.36.1367178578359; Sun, 28 Apr 2013 12:49:38 -0700 (PDT) Sender: adrian.chadd@gmail.com Received: by 10.217.58.138 with HTTP; Sun, 28 Apr 2013 12:49:38 -0700 (PDT) In-Reply-To: <15710249844.20130428234801@serebryakov.spb.ru> References: <2810538978.20130423164137@serebryakov.spb.ru> <1813905823.20130423184528@serebryakov.spb.ru> <184105677.20130424002002@serebryakov.spb.ru> <1936997795.20130424003555@serebryakov.spb.ru> <886711115.20130424004702@serebryakov.spb.ru> <6010292503.20130426001447@serebryakov.spb.ru> <99510815.20130426122508@serebryakov.spb.ru> <1467048277.20130428225006@serebryakov.spb.ru> <15710249844.20130428234801@serebryakov.spb.ru> Date: Sun, 28 Apr 2013 12:49:38 -0700 X-Google-Sender-Auth: WFvcZD78utCYvvmRSFXWVb6gEj4 Message-ID: Subject: Re: New hardware, old problem: stuck beacon when here is WiFi traffic From: Adrian Chadd To: lev@freebsd.org Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Cc: freebsd-wireless@freebsd.org X-BeenThere: freebsd-wireless@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Discussions of 802.11 stack, tools device driver development." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 28 Apr 2013 19:49:39 -0000 .. 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 wrote: > Hello, Adrian. > You wrote 28 =D0=B0=D0=BF=D1=80=D0=B5=D0=BB=D1=8F 2013 =D0=B3., 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 you= r > AC> kernel config and rebuild? I'd like to see if that TX queue workaroun= d > 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=3Dc99 -Wall -Wredundant-decls= -Wnested-externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith= -Winline -Wcast-qual -Wundef -Wno-pointer-sign -fformat-extensions -Wmis= sing-include-dirs -fdiagnostics-show-option -Wno-error-tautological-compar= e -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_HE= ADERS -include opt_global.h -fno-omit-frame-pointer -mno-aes -mno-avx -mcm= odel=3Dkernel -mno-red-zone -mno-mmx -mno-sse -msoft-float -fno-asynchrono= us-unwind-tables -ffreestanding -fstack-protector -Werror /data/src/sys/de= v/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 l= ong') [-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 'DP= RINTF' > 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 l= ong') [-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 'DP= RINTF' > 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 lo= ng') [-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 'DP= RINTF' > 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 lon= g') [-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 'DP= RINTF' > 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 lon= g') [-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 'DP= RINTF' > 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 lo= ng') [-Werror,-Wformat] > nexttbtt, > ^~~~~~~~ > /data/src/sys/dev/ath/if_ath_debug.h:97:34: note: expanded from macro 'DP= RINTF' > 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 lon= g') [-Werror,-Wformat] > tsf, > ^~~ > /data/src/sys/dev/ath/if_ath_debug.h:97:34: note: expanded from macro 'DP= RINTF' > 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 >