[PATCH 05/14] tests: Change persistent_group_missed_inv_resp() test
ilan.peer at intel.com
Sun Nov 1 06:54:31 PST 2015
> -----Original Message-----
> From: Jouni Malinen [mailto:j at w1.fi]
> Sent: Saturday, October 31, 2015 23:35
> To: Peer, Ilan
> Cc: hostap at lists.infradead.org; Stern, Avraham
> Subject: Re: [PATCH 05/14] tests: Change
> persistent_group_missed_inv_resp() test
> On Thu, Oct 22, 2015 at 02:09:44PM +0000, Peer, Ilan wrote:
> > > > Support configurations that use a dedicated P2P device interface
> > > > by setting ext_mgmt_frame_handling and waiting for MGMT-RX event
> > > > on the global control interface.
> > Would you prefer to handle this by adding an argument to the mgmt._rx to
> state if this is P2P frame or not and have the method resolve the interface
> name internally based on whether P2P Device is used or not (something
> similar to list_networks()).
> To be honest, I'm not sure I'd want either and I'm not sure I understand the
> need to make this specific test case "work" with the where mac80211_hwsim
> is forced to use a separate P2P Device interface. I would much rather focus
> on making sure there is sufficient coverage for both options in test cases
> without relying to running tests with custom configuration that results in this
> type of issues in the first place. In other words, I'd rather leave this as-is and if
> something is missing from testing coverage with P2P Device interface, that
> should addressed by adding a new test case that adds a dynamic hwsim radio
> with P2P Device interface and then appropriately targeted SET and wait-event
> commands on the interface that handles P2P messages.
We are running the testing framework with support for P2P Device dedicated interface as this is the configuration we are most interested with, so we are trying to modify the framework/tests so they could also be valuable for such configuration. Saying that, I concur with the approach of adding P2P Device interface specific tests when needed, to avoid relying on the custom configurations.
More information about the Hostap