linux-kernel - Re: 2.6.18-mm1 tiacx-fix-attribute-packed-warnings for wlan

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <20060928115134.92738bb4.rdunlap@xenotime.net>
Date:	Thu, 28 Sep 2006 11:51:34 -0700
From:	Randy Dunlap <rdunlap@...otime.net>
To:	Judith Lebzelter <judith@...l.org>
Cc:	linux-kernel@...r.kernel.org, vda@...ort.com.ua, akpm@...l.org,
	acx100-devel@...ts.sourceforge.net
Subject: Re: 2.6.18-mm1 tiacx-fix-attribute-packed-warnings for wlan

On Thu, 28 Sep 2006 10:34:21 -0700 Judith Lebzelter wrote:

> Hello,
> 
> The patch tiacx-fix-attribute-packed-warnings.patch takes care of many of 
> the 'packed attribute' warnings, but I noticed that some of these warnings like:
> 
> drivers/net/wireless/tiacx/wlan_compat.h:246: warning: 'packed' attribute ignored for field of type 'u8[5u]'
> drivers/net/wireless/tiacx/wlan_hdr.h:476: warning: 'packed' attribute ignored for field of type 'wlanitem_u32_t'
> drivers/net/wireless/tiacx/wlan_mgmt.h:253: warning: 'packed' attribute ignored for field of type 'u8'
> 
> are still present for the i386/allmodconfig build on gcc-4.1.1 for 2.6.18-mm1.  
> So I made a patch for the files where it is still a problem.  This compiled
> without warnings.


Hello,

How are you creating/generating patch files?  (git, quilt, by hand, ...)
[quilt can help a bit if your answer is "by hand" or by custom tools.]

Please see Documentation/SubmittingPatches, e.g.:

section 12) The canonical patch format

Consider including output of "diffstat -p1 -w70" after the
"---" marker so that patch readers can see what files are
changed and how much.

Please add diff option "-p" also.

(all mentioned in SubmittingPatches)

> diff -Nur linux.nodelta/drivers/net/wireless/tiacx/wlan_compat.h linux/drivers/net/wireless/tiacx/wlan_compat.h
> --- linux.nodelta/drivers/net/wireless/tiacx/wlan_compat.h	2006-09-28 09:06:00.628523695 -0700
> +++ linux/drivers/net/wireless/tiacx/wlan_compat.h	2006-09-28 09:50:59.301786662 -0700
> @@ -243,20 +243,20 @@

Thanks.
---
~Randy
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/