Re: [RFC] mac80211: add debugfs file for mic failure

[Johannes Berg <johannes@sipsolutions.net>]

On Tue, 2012-12-04 at 18:49 +0530, Saravana wrote:
> The mic failure count will provide the number of mic failure that 
> have happened without a countermeasure being started. Once the 
> countermeasure is started, the counter will be reset to 0.
> This count will be helpful when the consecutive 
> mic failure occur at intervals greater than 60 seconds regularly(frequently).
> 
> Signed-off-by: Saravana <saravanad@posedge.com>
> ---
>  net/mac80211/debugfs_key.c |   20 ++++++++++++++++++++
>  net/mac80211/key.h         |    3 +++
>  net/mac80211/wpa.c         |    2 ++
>  3 files changed, 25 insertions(+), 0 deletions(-)
> 
> diff --git a/net/mac80211/debugfs_key.c b/net/mac80211/debugfs_key.c
> index 2d42354..1cc262e 100644
> --- a/net/mac80211/debugfs_key.c
> +++ b/net/mac80211/debugfs_key.c
> @@ -199,6 +199,25 @@ static ssize_t key_icverrors_read(struct file *file, char __user *userbuf,
>  }
>  KEY_OPS(icverrors);
>  
> +static ssize_t key_mic_failures_read(struct file *file, char __user *userbuf,
> +				  size_t count, loff_t *ppos)

please fix indentation

> +{
> +	struct ieee80211_key *key = file->private_data;
> +	char buf[20];
> +	int len;
> +
> +	switch (key->conf.cipher) {
> +	case WLAN_CIPHER_SUITE_TKIP:
> +		len = scnprintf(buf, sizeof(buf), "%u\n",
> +				key->u.tkip.mic_failures);
> +		break;
> +	default:
> +		return 0;
> +	}

I think this should just be

if (key->conf.cipher != TKIP)
	return -EINVAL;

len = scnprintf(...)

> +	return simple_read_from_buffer(userbuf, count, ppos, buf, len);

Other than that looks fine.

johannes