linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ben Greear <greearb@candelatech.com>
To: Wojciech Dubowik <Wojciech.Dubowik@neratec.com>,
	linux-wireless@vger.kernel.org
Cc: kvalo@codeaurora.org
Subject: Re: [RFC 0/1] ath9k: Frame corruption simulator
Date: Fri, 20 Jan 2017 06:45:25 -0800	[thread overview]
Message-ID: <58822285.30008@candelatech.com> (raw)
In-Reply-To: <1484922570-23659-1-git-send-email-Wojciech.Dubowik@neratec.com>



On 01/20/2017 06:29 AM, Wojciech Dubowik wrote:
> I have been debugging customer reported timeout and loss of
> communication and I have relaized that I don't have such a lossy
> environment available in the lab. To speed up debugging I have
> written frame corruption simulator which will allow me to
> totally loose specific types of packets. I have been mostly
> using it with the mask 0x5000 which drops some EAPOL
> and deauthentication frames. This way I was able to test better
> timeouts and fail paths.
> At the moment only management, null function and EAPOL frames
> are supported. One can add more if necessary.

Would it be worth having a unique percentage configurable for each
of the selected packet types?

How about moving this up into mac80211 so other drivers could
be supported as well?  Couldn't you just drop the frames instead
of corrupting their checksum?  That would work with things like ath10k
as well.

I would like to have something like this, but with the added ability
to corrupt specific things like information-elements in management
frames to better test the receiver's packet parsing and error checking
logic.  For this feature, checksum would not be corrupted.

Thanks,
Ben

>
> Wojciech Dubowik (1):
>    ath9k: Add frame corruption simulator
>
>   drivers/net/wireless/ath/ath9k/Kconfig |  15 +++++
>   drivers/net/wireless/ath/ath9k/ath9k.h |   7 +++
>   drivers/net/wireless/ath/ath9k/debug.c |  49 +++++++++++++++
>   drivers/net/wireless/ath/ath9k/xmit.c  | 106 +++++++++++++++++++++++++++++++++
>   4 files changed, 177 insertions(+)
>

-- 
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com

  parent reply	other threads:[~2017-01-20 14:45 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-20 14:29 [RFC 0/1] ath9k: Frame corruption simulator Wojciech Dubowik
2017-01-20 14:29 ` [RFC 1/2] ath9k: Add tx descriptor flag to corrupt frame fcs Wojciech Dubowik
2017-01-20 14:29 ` [RFC 2/2] ath9k: Add frame corruption simulator Wojciech Dubowik
2017-01-20 15:07   ` Kalle Valo
2017-01-20 15:13     ` Wojciech Dubowik
2017-01-20 14:45 ` Ben Greear [this message]
2017-01-20 15:26   ` [RFC 0/1] ath9k: Frame " Wojciech Dubowik
2017-01-23 19:57     ` Ben Greear

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=58822285.30008@candelatech.com \
    --to=greearb@candelatech.com \
    --cc=Wojciech.Dubowik@neratec.com \
    --cc=kvalo@codeaurora.org \
    --cc=linux-wireless@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).