All of lore.kernel.org
 help / color / mirror / Atom feed
From: James Prestwood <prestwoj@gmail.com>
To: Denis Kenzior <denkenz@gmail.com>, iwd@lists.linux.dev
Subject: Re: [PATCH 1/3] hwsim: remove 'optimization' sending to only known MACs
Date: Wed, 28 Jun 2023 10:22:20 -0700	[thread overview]
Message-ID: <587f150a-e05c-378d-862c-103d8fcaac2a@gmail.com> (raw)
In-Reply-To: <44b16fe4-064f-dcc1-88e8-3e238c74c4a5@gmail.com>

Hi Denis,

On 6/28/23 9:57 AM, Denis Kenzior wrote:
> Hi James,
> 
>>> Then this may be a conversation you have to start on linux-wireless.  
>>> I would have thought that there should only be a single wmediumd 
>>> instance on a system regardless of namespaces, but maybe I'm wrong here.
>>
>> Yeah, I'll do that as well. This would simplify things if the 
>> namespaces didn't come into play.
> 
> Check
> 100cb9ff40e0 ("mac80211_hwsim: Allow managing radios from non-initial 
> namespaces")
> f21e4d8ed16b ("mac80211_hwsim: Allow wmediumd to attach to radios 
> created in its netns")

Yeah, the second one does mention frames specifically, which I am 
getting from radios outside the namespace. But I don't see a difference 
in how ADD/DEL_MAC_ADDR works. But yeah, this is something I'll look into.

> 
>  From those two commits I get the impression that the intent is to allow 
> a single wmediumd across namespaces.  If for some reason the 
> ADD_MAC_ADDR messages are not being sent, then I would look into fixing 
> that first.
> 
>>
>> Nevertheless, was the distributed hwsim concept something you'd be 
>> interested in accepting upstream? Like I said, I'm already doing it 
>> and I'm happy to re-use hwsim and extend to support this external 
>> socket concept.
> 
> I'm not against it, but as you point out, it sounds super complicated.  
> So I'd have to look into it more once you are ready to share something.

Ok fair enough.

> 
> Regards,
> -Denis
> 

  reply	other threads:[~2023-06-28 17:22 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-04 21:52 [PATCH 1/3] hwsim: remove 'optimization' sending to only known MACs James Prestwood
2023-05-04 21:52 ` [PATCH 2/3] test-runner: allow hwsim in namespaces James Prestwood
2023-05-04 21:52 ` [PATCH 3/3] test-runner: fix __str__ for namespace processes James Prestwood
2023-05-07 23:21 ` [PATCH 1/3] hwsim: remove 'optimization' sending to only known MACs Denis Kenzior
2023-05-08 13:43   ` James Prestwood
2023-05-08 18:05     ` Denis Kenzior
2023-05-08 18:55       ` James Prestwood
2023-05-08 19:00         ` Denis Kenzior
2023-05-08 19:03         ` James Prestwood
2023-05-08 19:01           ` Denis Kenzior
2023-06-21 21:05             ` James Prestwood
2023-06-27  2:31               ` Denis Kenzior
2023-06-27 15:15                 ` James Prestwood
2023-06-27 18:00                   ` Denis Kenzior
2023-06-27 18:56                     ` James Prestwood
2023-06-27 19:23                       ` Denis Kenzior
2023-06-27 20:09                     ` James Prestwood
2023-06-28 14:49                       ` Denis Kenzior
2023-06-28 15:33                         ` James Prestwood
2023-06-28 15:40                           ` Denis Kenzior
2023-06-28 16:14                             ` James Prestwood
2023-06-28 16:25                               ` Denis Kenzior
2023-06-28 16:47                                 ` James Prestwood
2023-06-28 16:57                                   ` Denis Kenzior
2023-06-28 17:22                                     ` James Prestwood [this message]
2023-06-28 23:19                               ` Andrew Zaborowski
2023-06-28 23:28                                 ` 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=587f150a-e05c-378d-862c-103d8fcaac2a@gmail.com \
    --to=prestwoj@gmail.com \
    --cc=denkenz@gmail.com \
    --cc=iwd@lists.linux.dev \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.