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
>
next prev parent 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.