linux-kernel - RE: [PATCH v2] wifi: rtw89: Print just once for unknown C2H classes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <e8e68a94bb9940509233153f9764c397@realtek.com>
Date: Wed, 30 Jul 2025 03:42:15 +0000
From: Ping-Ke Shih <pkshih@...ltek.com>
To: Sean Anderson <sean.anderson@...ux.dev>,
        "linux-wireless@...r.kernel.org"
	<linux-wireless@...r.kernel.org>
CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "Bitterblue
 Smith" <rtl8821cerfe2@...il.com>
Subject: RE: [PATCH v2] wifi: rtw89: Print just once for unknown C2H classes

Ping-Ke Shih <pkshih@...ltek.com> wrote:
> Sean Anderson <sean.anderson@...ux.dev> wrote:
> > There are more unsupported functions than just LOWRT_RTY. Improve on
> > commit 3b66519b023b ("wifi: rtw89: phy: add dummy c2h handler to avoid
> > warning message") by printing a message just once when we first
> > encounter an unsupported class.
> 
> Once I encounter an unsupported class/func, I'll check firmware team if the
> C2H events can be ignored. If so, I add a dummy function to avoid the message.
> If not, I should add code to handle the event.
> 
> Do you want to see the message even though it only appears once?
> 
> > Do the same for each unsupported func of
> > the supported classes. This prevents messages like
> >
> > rtw89_8922ae 0000:81:00.0: PHY c2h class 2 not support

Is this a real example? We have handled class 2 (RTW89_PHY_C2H_CLASS_DM), no?

Please point out the class / func you encountered. Then I can look up vendor
driver or contact internal firmware team to know if we should implement or
just add a dummy function.

If we defer it, I don't know when we can do it. 

> >
> > from filling up dmesg.
> >
> > Signed-off-by: Sean Anderson <sean.anderson@...ux.dev>
> > ---
> >
> > Changes in v2:
> > - Also suppress unsupported func messages
> >
> >  drivers/net/wireless/realtek/rtw89/phy.c | 18 +++++++++++++++---
> >  1 file changed, 15 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/net/wireless/realtek/rtw89/phy.c b/drivers/net/wireless/realtek/rtw89/phy.c
> > index f4eee642e5ce..9484d80eea9b 100644
> > --- a/drivers/net/wireless/realtek/rtw89/phy.c
> > +++ b/drivers/net/wireless/realtek/rtw89/phy.c
> > @@ -3535,17 +3535,25 @@ void rtw89_phy_c2h_handle(struct rtw89_dev *rtwdev, struct sk_buff *skb,
> >  {
> >         void (*handler)(struct rtw89_dev *rtwdev,
> >                         struct sk_buff *c2h, u32 len) = NULL;
> > +       static DECLARE_BITMAP(printed_ra, U8_MAX);
> > +       static DECLARE_BITMAP(printed_rfk_log, U8_MAX);
> > +       static DECLARE_BITMAP(printed_rfk_report, U8_MAX);
> > +       static DECLARE_BITMAP(printed_class, U8_MAX);
> > +       unsigned long *printed;
> >
> >         switch (class) {
> >         case RTW89_PHY_C2H_CLASS_RA:
> > +               printed = printed_ra;
> >                 if (func < RTW89_PHY_C2H_FUNC_RA_MAX)
> >                         handler = rtw89_phy_c2h_ra_handler[func];
> >                 break;
> >         case RTW89_PHY_C2H_RFK_LOG:
> > +               printed = printed_rfk_log;
> >                 if (func < ARRAY_SIZE(rtw89_phy_c2h_rfk_log_handler))
> >                         handler = rtw89_phy_c2h_rfk_log_handler[func];
> >                 break;
> >         case RTW89_PHY_C2H_RFK_REPORT:
> > +               printed = printed_rfk_report;
> >                 if (func < ARRAY_SIZE(rtw89_phy_c2h_rfk_report_handler))
> >                         handler = rtw89_phy_c2h_rfk_report_handler[func];
> >                 break;
> > @@ -3554,12 +3562,16 @@ void rtw89_phy_c2h_handle(struct rtw89_dev *rtwdev, struct sk_buff *skb,
> >                         return;
> >                 fallthrough;
> >         default:
> > -               rtw89_info(rtwdev, "PHY c2h class %d not support\n", class);
> > +               if (!test_and_set_bit(class, printed_class))
> > +                       rtw89_info(rtwdev, "PHY c2h class %d not supported\n",
> > +                                  class);
> >                 return;
> >         }
> >         if (!handler) {
> > -               rtw89_info(rtwdev, "PHY c2h class %d func %d not support\n", class,
> > -                          func);
> > +               if (!test_and_set_bit(func, printed))
> > +                       rtw89_info(rtwdev,
> > +                                  "PHY c2h class %d func %d not supported\n",
> > +                                  class, func);
> >                 return;
> >         }
> >         handler(rtwdev, skb, len);
> > --
> > 2.35.1.1320.gc452695387.dirty