(In case anyone is interested)
This is todays 2.6.25-rc8+git on ppc (not powerpc) arch (32-bit),
gcc version 4.3.1 20080401 (prerelease) (Debian 4.3.0-3)
drivers/net/wireless/iwlwifi/iwl3945-base.c: In function 'iwl3945_enqueue_hcmd':
drivers/net/wireless/iwlwifi/iwl3945-base.c:719: warning: integer overflow in expression
drivers/net/wireless/iwlwifi/iwl-3945.c: In function 'iwl3945_add_radiotap':
drivers/net/wireless/iwlwifi/iwl-3945.c:326: warning: integer overflow in expression
drivers/net/wireless/iwlwifi/iwl4965-base.c: In function 'iwl4965_enqueue_hcmd':
drivers/net/wireless/iwlwifi/iwl4965-base.c:723: warning: integer overflow in expression
drivers/net/wireless/iwlwifi/iwl-4965.c: In function 'iwl4965_add_radiotap':
drivers/net/wireless/iwlwifi/iwl-4965.c:3617: warning: integer overflow in expression
--
Meelis Roos ([email protected])
> > drivers/net/wireless/iwlwifi/iwl3945-base.c: In function
> > 'iwl3945_enqueue_hcmd':
> > drivers/net/wireless/iwlwifi/iwl3945-base.c:719: warning: integer
> > overflow in expression drivers/net/wireless/iwlwifi/iwl-3945.c: In
> > function 'iwl3945_add_radiotap':
> > drivers/net/wireless/iwlwifi/iwl-3945.c:326: warning: integer
> > overflow in expression drivers/net/wireless/iwlwifi/iwl4965-base.c:
> > In function 'iwl4965_enqueue_hcmd':
> > drivers/net/wireless/iwlwifi/iwl4965-base.c:723: warning: integer
> > overflow in expression drivers/net/wireless/iwlwifi/iwl-4965.c: In
> > function 'iwl4965_add_radiotap':
> > drivers/net/wireless/iwlwifi/iwl-4965.c:3617: warning: integer
> > overflow in expression
> >
>
> (adding Johannes and John)
>
> These problems have been fixed by Johannes with his patch "iwlwifi:fix
> some warnings". If this is an issue in 2.6.25 then that kernel may take
> this patch to address the problem.
That patch never was applied though and Tomas had reservations. I didn't
have time to look at it again.
johannes
On Fri, Apr 11, 2008 at 02:16:44PM +0200, Johannes Berg wrote:
>
> > > drivers/net/wireless/iwlwifi/iwl3945-base.c: In function
> > > 'iwl3945_enqueue_hcmd':
> > > drivers/net/wireless/iwlwifi/iwl3945-base.c:719: warning: integer
> > > overflow in expression drivers/net/wireless/iwlwifi/iwl-3945.c: In
> > > function 'iwl3945_add_radiotap':
> > > drivers/net/wireless/iwlwifi/iwl-3945.c:326: warning: integer
> > > overflow in expression drivers/net/wireless/iwlwifi/iwl4965-base.c:
> > > In function 'iwl4965_enqueue_hcmd':
> > > drivers/net/wireless/iwlwifi/iwl4965-base.c:723: warning: integer
> > > overflow in expression drivers/net/wireless/iwlwifi/iwl-4965.c: In
> > > function 'iwl4965_add_radiotap':
> > > drivers/net/wireless/iwlwifi/iwl-4965.c:3617: warning: integer
> > > overflow in expression
> > >
> >
> > (adding Johannes and John)
> >
> > These problems have been fixed by Johannes with his patch "iwlwifi:fix
> > some warnings". If this is an issue in 2.6.25 then that kernel may take
> > this patch to address the problem.
>
> That patch never was applied though and Tomas had reservations. I didn't
> have time to look at it again.
Actually, I factored-out the parts Tomas didn't like and applied it.
--
John W. Linville
[email protected]
On Friday, April 11, 2008 5:17 AM, Johannes Berg wrote:
>>> drivers/net/wireless/iwlwifi/iwl3945-base.c: In function
>>> 'iwl3945_enqueue_hcmd':
>>> drivers/net/wireless/iwlwifi/iwl3945-base.c:719: warning: integer
>>> overflow in expression drivers/net/wireless/iwlwifi/iwl-3945.c: In
>>> function 'iwl3945_add_radiotap':
>>> drivers/net/wireless/iwlwifi/iwl-3945.c:326: warning: integer
>>> overflow in expression drivers/net/wireless/iwlwifi/iwl4965-base.c:
>>> In function 'iwl4965_enqueue_hcmd':
>>> drivers/net/wireless/iwlwifi/iwl4965-base.c:723: warning: integer
>>> overflow in expression drivers/net/wireless/iwlwifi/iwl-4965.c: In
>>> function 'iwl4965_add_radiotap':
>>> drivers/net/wireless/iwlwifi/iwl-4965.c:3617: warning: integer
>>> overflow in expression
>>>
>>
>> (adding Johannes and John)
>>
>> These problems have been fixed by Johannes with his patch
>> "iwlwifi:fix some warnings". If this is an issue in 2.6.25 then that
>> kernel may take this patch to address the problem.
>
> That patch never was applied though and Tomas had
> reservations. I didn't
> have time to look at it again.
I did find it in the current wireless-testing as commit
cb08e71a7bc92a5ad518758584f077ed81d62b5c
Reinette
On , Meelis Roos wrote:
> (In case anyone is interested)
>
> This is todays 2.6.25-rc8+git on ppc (not powerpc) arch (32-bit),
> gcc version 4.3.1 20080401 (prerelease) (Debian 4.3.0-3)
>
> drivers/net/wireless/iwlwifi/iwl3945-base.c: In function
> 'iwl3945_enqueue_hcmd':
> drivers/net/wireless/iwlwifi/iwl3945-base.c:719: warning: integer
> overflow in expression drivers/net/wireless/iwlwifi/iwl-3945.c: In
> function 'iwl3945_add_radiotap':
> drivers/net/wireless/iwlwifi/iwl-3945.c:326: warning: integer
> overflow in expression drivers/net/wireless/iwlwifi/iwl4965-base.c:
> In function 'iwl4965_enqueue_hcmd':
> drivers/net/wireless/iwlwifi/iwl4965-base.c:723: warning: integer
> overflow in expression drivers/net/wireless/iwlwifi/iwl-4965.c: In
> function 'iwl4965_add_radiotap':
> drivers/net/wireless/iwlwifi/iwl-4965.c:3617: warning: integer
> overflow in expression
>
(adding Johannes and John)
These problems have been fixed by Johannes with his patch "iwlwifi:fix
some warnings". If this is an issue in 2.6.25 then that kernel may take
this patch to address the problem.
Reinette