The HWMP sequence number of received RANN element is compared to decide whether to be
propagated. The sequence number is required to covert from 32bit little endian data into
CPUs endianness for comparison.
Signed-off-by: Chun-Yeow Yeoh <[email protected]>
---
net/mac80211/mesh_hwmp.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/net/mac80211/mesh_hwmp.c b/net/mac80211/mesh_hwmp.c
index 1c6f3d0..2ccb3c9 100644
--- a/net/mac80211/mesh_hwmp.c
+++ b/net/mac80211/mesh_hwmp.c
@@ -748,7 +748,7 @@ static void hwmp_rann_frame_process(struct ieee80211_sub_if_data *sdata,
flags = rann->rann_flags;
root_is_gate = !!(flags & RANN_FLAG_IS_GATE);
orig_addr = rann->rann_addr;
- orig_sn = rann->rann_seq;
+ orig_sn = le32_to_cpu(rann->rann_seq);
hopcount = rann->rann_hopcount;
hopcount++;
metric = rann->rann_metric;
--
1.7.0.4
Hi, Johannes
>
> You should mark struct ieee80211_rann_ie to contain __le32, so that
> sparse will warn about things like this.
>
I try to change u32 to __le32 for the following members in struct
ieee80211_rann_ie.
struct ieee80211_rann_ie {
u8 rann_flags;
u8 rann_hopcount;
u8 rann_ttl;
u8 rann_addr[6];
__le32 rann_seq;
__le32 rann_interval;
__le32 rann_metric;
} __attribute__ ((packed));
But when I compile using "make C=2 CF=-D__CHECK_ENDIAN__", I don't see
any sparse warning on this.
Please advice. Thanks
Regards,
Chun-Yeow
On Sat, 2012-03-17 at 02:43 +0800, Chun-Yeow Yeoh wrote:
> The HWMP sequence number of received RANN element is compared to decide whether to be
> propagated. The sequence number is required to covert from 32bit little endian data into
> CPUs endianness for comparison.
>
> Signed-off-by: Chun-Yeow Yeoh <[email protected]>
> ---
> net/mac80211/mesh_hwmp.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/net/mac80211/mesh_hwmp.c b/net/mac80211/mesh_hwmp.c
> index 1c6f3d0..2ccb3c9 100644
> --- a/net/mac80211/mesh_hwmp.c
> +++ b/net/mac80211/mesh_hwmp.c
> @@ -748,7 +748,7 @@ static void hwmp_rann_frame_process(struct ieee80211_sub_if_data *sdata,
> flags = rann->rann_flags;
> root_is_gate = !!(flags & RANN_FLAG_IS_GATE);
> orig_addr = rann->rann_addr;
> - orig_sn = rann->rann_seq;
> + orig_sn = le32_to_cpu(rann->rann_seq);
You should mark struct ieee80211_rann_ie to contain __le32, so that
sparse will warn about things like this.
johannes
Ok, I am able to generate the warnings. It seems that I need to
resolve the first error below, otherwise don't see the subsequent
warnings.
include/linux/skbuff.h:357:41: error: invalid bitfield specifier for
type restricted __be16.
Will submit patch with different naming since it involves the
rann->metric as well.
Chun-Yeow
On Mon, Mar 19, 2012 at 12:32 AM, Johannes Berg
<[email protected]> wrote:
> Hi,
>
>> > You should mark struct ieee80211_rann_ie to contain __le32, so that
>> > sparse will warn about things like this.
>> >
>>
>> I try to change u32 to __le32 for the following members in struct
>> ieee80211_rann_ie.
>>
>> struct ieee80211_rann_ie {
>> u8 rann_flags;
>> u8 rann_hopcount;
>> u8 rann_ttl;
>> u8 rann_addr[6];
>> __le32 rann_seq;
>> __le32 rann_interval;
>> __le32 rann_metric;
>> } __attribute__ ((packed));
>>
>> But when I compile using "make C=2 CF=-D__CHECK_ENDIAN__", I don't see
>> any sparse warning on this.
>
> Not sure, if I make the above change I get:
>
> CHECK net/mac80211/mesh_hwmp.c
> net/mac80211/mesh_hwmp.c:751:17: warning: incorrect type in assignment (different base types)
> net/mac80211/mesh_hwmp.c:751:17: expected unsigned int [unsigned] [usertype] orig_sn
> net/mac80211/mesh_hwmp.c:751:17: got restricted __le32 [usertype] rann_seq
> net/mac80211/mesh_hwmp.c:754:16: warning: incorrect type in assignment (different base types)
> net/mac80211/mesh_hwmp.c:754:16: expected unsigned int [unsigned] [usertype] metric
> net/mac80211/mesh_hwmp.c:754:16: got restricted __le32 [usertype] rann_metric
>
> johannes
>
Hi,
> > You should mark struct ieee80211_rann_ie to contain __le32, so that
> > sparse will warn about things like this.
> >
>
> I try to change u32 to __le32 for the following members in struct
> ieee80211_rann_ie.
>
> struct ieee80211_rann_ie {
> u8 rann_flags;
> u8 rann_hopcount;
> u8 rann_ttl;
> u8 rann_addr[6];
> __le32 rann_seq;
> __le32 rann_interval;
> __le32 rann_metric;
> } __attribute__ ((packed));
>
> But when I compile using "make C=2 CF=-D__CHECK_ENDIAN__", I don't see
> any sparse warning on this.
Not sure, if I make the above change I get:
CHECK net/mac80211/mesh_hwmp.c
net/mac80211/mesh_hwmp.c:751:17: warning: incorrect type in assignment (different base types)
net/mac80211/mesh_hwmp.c:751:17: expected unsigned int [unsigned] [usertype] orig_sn
net/mac80211/mesh_hwmp.c:751:17: got restricted __le32 [usertype] rann_seq
net/mac80211/mesh_hwmp.c:754:16: warning: incorrect type in assignment (different base types)
net/mac80211/mesh_hwmp.c:754:16: expected unsigned int [unsigned] [usertype] metric
net/mac80211/mesh_hwmp.c:754:16: got restricted __le32 [usertype] rann_metric
johannes