Linux wireless drivers development
 help / color / mirror / Atom feed
From: James Prestwood <james.prestwood@linux.intel.com>
To: Johannes Berg <johannes@sipsolutions.net>,
	linux-wireless@vger.kernel.org
Subject: Re: [PATCH v2 1/2] mac80211_hwsim: allow setting iftype support
Date: Wed, 17 Oct 2018 14:00:23 -0700	[thread overview]
Message-ID: <d7b8072e7648942738cac427b0b1f7005befc0c5.camel@linux.intel.com> (raw)
In-Reply-To: <48e12e055637af139094b6c1163f61c45b5ebb54.camel@sipsolutions.net>

On Wed, 2018-10-17 at 21:53 +0200, Johannes Berg wrote:
> On Wed, 2018-10-17 at 12:33 -0700, James Prestwood wrote:
> > The mac80211_hwsim driver hard codes its supported interface types.
> > For
> > testing purposes it would be valuable to allow changing these
> > supported
> > types in order to simulate actual drivers than support a limited
> > set of
> 
> typo: than -> that
> 
> > In order to accomplish this, the ieee80211_iface_limit structure
> > needed
> > to be built dynamically to only include limit rules for iftypes
> > that
> > the user requested to enable.
> 
> That makes me wonder if you'd also want to support limiting the # of
> interfaces/channels to mimic another device?

Yeah it seems like it would be pretty easy to add that on top of this
change, although I don't really see us actually using it. I can add it,
but my only concern is I would not test it up to the same level I
tested iftype/ciphers. Maybe this is also because I am not entirely
sure what num_different_channels is doing. Is this only relevant when
multiple interfaces exist on a radio? Like setting how many channels
can be use simultaneously?

> 
> johannes
> 

  reply	other threads:[~2018-10-17 20:56 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-17 19:33 [PATCH v2 0/2] Configuring iftype/cipher support James Prestwood
2018-10-17 19:33 ` [PATCH v2 1/2] mac80211_hwsim: allow setting iftype support James Prestwood
2018-10-17 19:43   ` James Prestwood
2018-10-17 19:53   ` Johannes Berg
2018-10-17 21:00     ` James Prestwood [this message]
2018-10-17 20:58       ` Johannes Berg
2018-10-17 21:05         ` James Prestwood
2018-10-17 21:08           ` Johannes Berg
2018-10-17 21:15             ` James Prestwood
2018-10-17 19:33 ` [PATCH v2 2/2] mac80211_hwsim: allow configurable cipher types James Prestwood

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=d7b8072e7648942738cac427b0b1f7005befc0c5.camel@linux.intel.com \
    --to=james.prestwood@linux.intel.com \
    --cc=johannes@sipsolutions.net \
    --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