linux/net/wireless/tests/chan.c
Johannes Berg b82730bf57 wifi: cfg80211/mac80211: move puncturing into chandef
Aloka originally suggested that puncturing should be part of
the chandef, so that it's treated correctly. At the time, I
disagreed and it ended up not part of the chandef, but I've
now realized that this was wrong. Even for clients, the RX,
and perhaps more importantly, CCA configuration needs to take
puncturing into account.

Move puncturing into the chandef, and adjust all the code
accordingly. Also add a few tests for puncturing in chandef
compatibility checking.

Link: https://lore.kernel.org/linux-wireless/20220214223051.3610-1-quic_alokad@quicinc.com/
Suggested-by: Aloka Dixit <quic_alokad@quicinc.com>
Link: https://msgid.link/20240129194108.307183a5d2e5.I4d7fe2f126b2366c1312010e2900dfb2abffa0f6@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
2024-02-08 15:00:39 +01:00

229 lines
4.9 KiB
C

// SPDX-License-Identifier: GPL-2.0-only
/*
* KUnit tests for channel helper functions
*
* Copyright (C) 2023-2024 Intel Corporation
*/
#include <net/cfg80211.h>
#include <kunit/test.h>
MODULE_IMPORT_NS(EXPORTED_FOR_KUNIT_TESTING);
static struct ieee80211_channel chan_6ghz_1 = {
.band = NL80211_BAND_6GHZ,
.center_freq = 5955,
};
static struct ieee80211_channel chan_6ghz_5 = {
.band = NL80211_BAND_6GHZ,
.center_freq = 5975,
};
static struct ieee80211_channel chan_6ghz_105 = {
.band = NL80211_BAND_6GHZ,
.center_freq = 6475,
};
static const struct chandef_compat_case {
const char *desc;
/* leave c1 empty for tests for identical */
struct cfg80211_chan_def c1, c2;
/* we test both ways around, so c2 should always be the compat one */
bool compat;
} chandef_compat_cases[] = {
{
.desc = "identical non-HT",
.c2 = {
.width = NL80211_CHAN_WIDTH_20_NOHT,
.chan = &chan_6ghz_1,
.center_freq1 = 5955,
},
.compat = true,
},
{
.desc = "identical 20 MHz",
.c2 = {
.width = NL80211_CHAN_WIDTH_20,
.chan = &chan_6ghz_1,
.center_freq1 = 5955,
},
.compat = true,
},
{
.desc = "identical 40 MHz",
.c2 = {
.width = NL80211_CHAN_WIDTH_40,
.chan = &chan_6ghz_1,
.center_freq1 = 5955 + 10,
},
.compat = true,
},
{
.desc = "identical 80 MHz",
.c2 = {
.width = NL80211_CHAN_WIDTH_80,
.chan = &chan_6ghz_1,
.center_freq1 = 5955 + 10 + 20,
},
.compat = true,
},
{
.desc = "identical 160 MHz",
.c2 = {
.width = NL80211_CHAN_WIDTH_160,
.chan = &chan_6ghz_1,
.center_freq1 = 5955 + 10 + 20 + 40,
},
.compat = true,
},
{
.desc = "identical 320 MHz",
.c2 = {
.width = NL80211_CHAN_WIDTH_320,
.chan = &chan_6ghz_1,
.center_freq1 = 5955 + 10 + 20 + 40 + 80,
},
.compat = true,
},
{
.desc = "20 MHz in 320 MHz\n",
.c1 = {
.width = NL80211_CHAN_WIDTH_20,
.chan = &chan_6ghz_1,
.center_freq1 = 5955,
},
.c2 = {
.width = NL80211_CHAN_WIDTH_320,
.chan = &chan_6ghz_1,
.center_freq1 = 5955 + 10 + 20 + 40 + 80,
},
.compat = true,
},
{
.desc = "different 20 MHz",
.c1 = {
.width = NL80211_CHAN_WIDTH_20,
.chan = &chan_6ghz_1,
.center_freq1 = 5955,
},
.c2 = {
.width = NL80211_CHAN_WIDTH_20,
.chan = &chan_6ghz_5,
.center_freq1 = 5975,
},
},
{
.desc = "different primary 160 MHz",
.c1 = {
.width = NL80211_CHAN_WIDTH_320,
.chan = &chan_6ghz_105,
.center_freq1 = 6475 + 150,
},
.c2 = {
.width = NL80211_CHAN_WIDTH_320,
.chan = &chan_6ghz_105,
.center_freq1 = 6475 - 10,
},
},
{
/* similar to previous test but one has lower BW */
.desc = "matching primary 160 MHz",
.c1 = {
.width = NL80211_CHAN_WIDTH_160,
.chan = &chan_6ghz_105,
.center_freq1 = 6475 + 70,
},
.c2 = {
.width = NL80211_CHAN_WIDTH_320,
.chan = &chan_6ghz_105,
.center_freq1 = 6475 - 10,
},
.compat = true,
},
{
.desc = "matching primary 160 MHz & punctured secondary 160 Mhz",
.c1 = {
.width = NL80211_CHAN_WIDTH_160,
.chan = &chan_6ghz_105,
.center_freq1 = 6475 + 70,
},
.c2 = {
.width = NL80211_CHAN_WIDTH_320,
.chan = &chan_6ghz_105,
.center_freq1 = 6475 - 10,
.punctured = 0xf,
},
.compat = true,
},
{
.desc = "matching primary 160 MHz & punctured matching",
.c1 = {
.width = NL80211_CHAN_WIDTH_160,
.chan = &chan_6ghz_105,
.center_freq1 = 6475 + 70,
.punctured = 0xc0,
},
.c2 = {
.width = NL80211_CHAN_WIDTH_320,
.chan = &chan_6ghz_105,
.center_freq1 = 6475 - 10,
.punctured = 0xc000,
},
.compat = true,
},
{
.desc = "matching primary 160 MHz & punctured not matching",
.c1 = {
.width = NL80211_CHAN_WIDTH_160,
.chan = &chan_6ghz_105,
.center_freq1 = 6475 + 70,
.punctured = 0x80,
},
.c2 = {
.width = NL80211_CHAN_WIDTH_320,
.chan = &chan_6ghz_105,
.center_freq1 = 6475 - 10,
.punctured = 0xc000,
},
},
};
KUNIT_ARRAY_PARAM_DESC(chandef_compat, chandef_compat_cases, desc)
static void test_chandef_compat(struct kunit *test)
{
const struct chandef_compat_case *params = test->param_value;
const struct cfg80211_chan_def *ret, *expect;
struct cfg80211_chan_def c1 = params->c1;
/* tests with identical ones */
if (!params->c1.chan)
c1 = params->c2;
KUNIT_EXPECT_EQ(test, cfg80211_chandef_valid(&c1), true);
KUNIT_EXPECT_EQ(test, cfg80211_chandef_valid(&params->c2), true);
expect = params->compat ? &params->c2 : NULL;
ret = cfg80211_chandef_compatible(&c1, &params->c2);
KUNIT_EXPECT_PTR_EQ(test, ret, expect);
if (!params->c1.chan)
expect = &c1;
ret = cfg80211_chandef_compatible(&params->c2, &c1);
KUNIT_EXPECT_PTR_EQ(test, ret, expect);
}
static struct kunit_case chandef_compat_test_cases[] = {
KUNIT_CASE_PARAM(test_chandef_compat, chandef_compat_gen_params),
{}
};
static struct kunit_suite chandef_compat = {
.name = "cfg80211-chandef-compat",
.test_cases = chandef_compat_test_cases,
};
kunit_test_suite(chandef_compat);