From owner-freebsd-wireless@FreeBSD.ORG Mon Jul 29 17:58:27 2013 Return-Path: Delivered-To: freebsd-wireless@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id B774CF35; Mon, 29 Jul 2013 17:58:27 +0000 (UTC) (envelope-from hans.petter.selasky@bitfrost.no) Received: from mta.bitpro.no (mta.bitpro.no [92.42.64.202]) by mx1.freebsd.org (Postfix) with ESMTP id 420F625B9; Mon, 29 Jul 2013 17:58:27 +0000 (UTC) Received: from mail.lockless.no (mail.lockless.no [46.29.221.38]) by mta.bitpro.no (Postfix) with ESMTP id 311BD7A10F; Mon, 29 Jul 2013 19:58:25 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by mail.lockless.no (Postfix) with ESMTP id 7D6388F08E9; Mon, 29 Jul 2013 19:58:30 +0200 (CEST) X-Virus-Scanned: by amavisd-new-2.6.4 (20090625) (Debian) at lockless.no Received: from mail.lockless.no ([127.0.0.1]) by localhost (mail.lockless.no [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id QHqor4BVr9Hw; Mon, 29 Jul 2013 19:58:29 +0200 (CEST) Received: from mail.lockless.no (localhost [127.0.0.1]) by mail.lockless.no (Postfix) with ESMTP id 73D1A8F08E8; Mon, 29 Jul 2013 19:58:29 +0200 (CEST) Subject: RE: My WLI-UC-GNM up crash From: =?utf-8?Q?Hans_Petter_Selasky?= To: =?utf-8?Q?Warner_Losh?= , =?utf-8?Q?Adrian_Chadd?= Date: Mon, 29 Jul 2013 19:58:29 +0200 Mime-Version: 1.0 In-Reply-To: <8AFE4FCA-BCAA-460C-ABFE-EC7FC2991B8C@bsdimp.com> References: X-Priority: 3 (Normal) X-Mailer: Zarafa 7.1.4-41394 Message-Id: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.14 Cc: =?utf-8?Q?freebsd-arm?= , =?utf-8?Q?freebsd-wireless=40freebsd=2Eorg?= 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: Mon, 29 Jul 2013 17:58:27 -0000 Hi,=0D=0A=0D=0AThe aligned will make sure that the structure gets padded = properly to the size specified. Only on ARM/MIPS etc, structures get auto= matically aligned according to the element in the structure requiring the= greatest alignment. I've test-compiled the USB WLAN drivers, and the ali= gned makes a difference. The problem is that the radiotap header skews so= me following elements, so that they are no longer aligned. The radiotap h= eader itself is packed, and this is not a problem.=0D=0A=0D=0A--HPS=0D=0A= =0D=0A=20=0D=0A-----Original message-----=0D=0A> From:Warner Losh >=0D=0A> Sent: Monday 29th July 2013 17:= 04=0D=0A> To: Adrian Chadd >=0D=0A> Cc: Hans Petter Selasky >; freebsd-arm >; freebsd-wireless@freebsd.org =20=0D=0A> Subject: Re: My WLI-UC-GNM up = crash=0D=0A>=20=0D=0A> Aren't structures already aligned to 4 bytes when = placed inside other structures (unless marked __packed)=3F=0D=0A>=20=0D=0A= > Warner=0D=0A>=20=0D=0A> On Jul 28, 2013, at 11:50 AM, Adrian Chadd wrot= e:=0D=0A>=20=0D=0A> > As long as that results in the radiotap structures = being 4 or 8 byte=0D=0A> > padded when it's embedded in the softc - then = yes, indeed.=0D=0A> >=20=0D=0A> > Xiao, can you try=3F=0D=0A> >=20=0D=0A>= >=20=0D=0A> > -adrian=0D=0A> >=20=0D=0A> > On 28 July 2013 03:35, Hans P= etter Selasky > wrote:=0D=0A> >= > Hi,=0D=0A> >>=20=0D=0A> >> Can you try the attached patch=3F=0D=0A> >>=20= =0D=0A> >> --HPS=0D=0A> > _______________________________________________= =0D=0A> > freebsd-arm@freebsd.org maili= ng list=0D=0A> > http://lists.freebsd.org/mailman/listinfo/freebsd-arm =20=0D=0A> > To unsu= bscribe, send any mail to "freebsd-arm-unsubscribe@freebsd.org "=0D=0A>=20=0D=0A>=20=0D=0A=0D=0A From owner-freebsd-wireless@FreeBSD.ORG Mon Jul 29 18:10:58 2013 Return-Path: Delivered-To: freebsd-wireless@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 84B73215; Mon, 29 Jul 2013 18:10:58 +0000 (UTC) (envelope-from wblock@wonkity.com) Received: from wonkity.com (wonkity.com [67.158.26.137]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 35DC6265B; Mon, 29 Jul 2013 18:10:58 +0000 (UTC) Received: from wonkity.com (localhost [127.0.0.1]) by wonkity.com (8.14.7/8.14.7) with ESMTP id r6TIAbx3045678; Mon, 29 Jul 2013 12:10:37 -0600 (MDT) (envelope-from wblock@wonkity.com) Received: from localhost (wblock@localhost) by wonkity.com (8.14.7/8.14.7/Submit) with ESMTP id r6TIAZjO045675; Mon, 29 Jul 2013 12:10:35 -0600 (MDT) (envelope-from wblock@wonkity.com) Date: Mon, 29 Jul 2013 12:10:35 -0600 (MDT) From: Warren Block To: Adrian Chadd Subject: Re: [iwn] Review split 3 In-Reply-To: Message-ID: References: <007601ce8c73$2b508cf0$81f1a6d0$@info> User-Agent: Alpine 2.00 (BSF 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.4.3 (wonkity.com [127.0.0.1]); Mon, 29 Jul 2013 12:10:37 -0600 (MDT) 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: Mon, 29 Jul 2013 18:10:58 -0000 On Mon, 29 Jul 2013, Adrian Chadd wrote: > Hi! > > Cool! ok, the style niggles. > > * Your #define IWN_blah needs a TAB between #define and the IWN_blah > value. Right now you have spaces. There is whitespace at the end of some of the comment lines also. textproc/igor will find problems like that: igor -R filename | less -RS