On 09/23/2014 05:54 AM, Kalle Valo wrote:
greearb@xxxxxxxxxxxxxxx writes:
From: Ben Greear <greearb@xxxxxxxxxxxxxxx>
This can allow more than 32 stations to be supported
without over-running the bitmap.
Signed-off-by: Ben Greear <greearb@xxxxxxxxxxxxxxx>
[...]
- ar->monitor_vdev_id = bit - 1;
+ ar->monitor_vdev_id = bit;
[...]
- arvif->vdev_id = bit - 1;
+ ath10k_warn(ar, "Creating vdev id: %i map: %llu\n",
+ bit, ar->free_vdev_map);
+
+ arvif->vdev_id = bit;
Why remove the "- 1"? Are you sure that's not going to break any
assumptions somewhere?
I have been testing this patch in one form or another for almost a year
and have not noticed any problems with it.
The return value is subtly different when using the 64-bit version,
so you don't need the -1.
Thanks,
Ben
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html