Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 1 Mar 2022 10:49:05 +0300
From:   Dan Carpenter <dan.carpenter@oracle.com>
To:     john@phrozen.org
Cc:     ath11k@lists.infradead.org, linux-wireless@vger.kernel.org
Subject: [bug report] ath11k: add debugfs for TWT debug calls
Message-ID: <20220301074905.GA13071@kili>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.10.1 (2018-07-13)
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?sVNHp+zl+0JFVrD45saCAlkDQpsn1OtNSiAjppBiBPM+BjvMB+AD6+ycZ8pC?=
 =?us-ascii?Q?f7oPyKlMd/jYkomHPCWYXyVSQHgH0roKDTGMBErRenu6oVDDIX7n5LiTkL7v?=
 =?us-ascii?Q?ejh+D9EVqSubbup4NiFkbZZ6JU3iB5Z2UsK8vpPk/gBz8oXfWnsBPjf2NseZ?=
 =?us-ascii?Q?BpzY47ZCOOsf6cSHAXtgHUoS/o5YwTg3ioqyDhhRLHa0a07FHmr/nqvry5Gw?=
 =?us-ascii?Q?HKyCgt8ogMFKpcsnC+wGbhvlFtFHjweC6xcGAkkaK5QhR2dPB8MoVJRQDZT/?=
 =?us-ascii?Q?vT0O5eAZs8tvEUWRWfHTOqVf142gtVPbHnqp3hqrEm1MzLLIM1goxW4rl0SY?=
 =?us-ascii?Q?CmOT6EZJK4RT7WI/JwSjiVjdMtQoDqxDC7/bjhfZY2K0suxUeEqKWUtUgBHF?=
 =?us-ascii?Q?1jPT+jz0I1rQVZ5DqBe0U/i8i5PT9/nVhsa7BsBjsFr8dn9Ig8PwfUzrrAvT?=
 =?us-ascii?Q?LSkLWgx66h5+HmvvLVVoX25ij2MgbJXMczhJn+qOYJyC4jxOaAYZa3IhJRlc?=
 =?us-ascii?Q?TDnxOKRnUWcSBG6Ot0bAZvSc5xFnZDwLJpZSsrvp+b9kcjQ+v85fJIf0cPtj?=
 =?us-ascii?Q?rJduls+hNGmc1+v0rSvgXDniJSbz//lNpk+Qg0ScKVYjx2+6Xy6g6tRQbA2z?=
 =?us-ascii?Q?27G2lPMlfzXGT6c6R/hx6h2MphhodaSesKBWRp4gAXlib1YT4CB+jNh9VBl6?=
 =?us-ascii?Q?wrTrtJWWiDANJzj19pnfUhl0OY9SunssRJ5JoUxcfHmv2erSpwvIQhRcwi9S?=
 =?us-ascii?Q?s/4Vb9nH5v+p1MnOtD4+HtQ4iVLITPLjjzuzEwHN6LMCA4cu+fKl10ceYle4?=
 =?us-ascii?Q?KDlSF2wTbgEG2oTTSTZBPBHS7MmuJWjoWRQcUS04WuapdZ3mQSWUApf7U3b1?=
 =?us-ascii?Q?ULFB6xYPYrlk+OK7yjFNzTqjBfIQbgi5J6F+bttQdMnOlf1X25XmbxNcJ+Dq?=
 =?us-ascii?Q?wCPJCFztU7bBdhAhneYF4IagawWwauJ7VJ/i9eTEngpUphesgVbJeHXgSTvK?=
 =?us-ascii?Q?kpdqxjeFRDnoahL1n58Nx05oMjtFJVwDRb8rcT4Nstf1MBfVC06g6sKQZVMt?=
 =?us-ascii?Q?vCI5shlyp6hLCQ/5e3GxtynX+6HI5R7UNIF9l9FKz2N5Gk90QHwLKMyt3exM?=
 =?us-ascii?Q?bLN5uMgjH7WXv9vjXALOfQPMEMbNkRVszLv+x6HDIeq3CzkFC34OYwnGpaC7?=
 =?us-ascii?Q?z9CjhFsw3Dm9VeDkteq4+5Ro19IA8tD8Hlx+MxuZFgFExiWHyrr383cJZeUY?=
 =?us-ascii?Q?SoaTA8Ce6leXM6mDjY4qLlmRnr+qusRffbZjMMT8Ro2cgFlbFZ0yuEL6Z70X?=
 =?us-ascii?Q?yobIMU8LHNzT/LmFr+a29syP29V33RN/q17nqpyl4diAE9+NYdb0Scg5NZ2v?=
 =?us-ascii?Q?jshcUCeJEeD+xQzYwbh8v7t+NKrrJHUrEBmMfHW6/5uAGIzusr7N8Jvm+xR9?=
 =?us-ascii?Q?LOvmhBgx0bPydOcxxOxDsDLxxvbnEffNhoDQh88fysCeCkDbNOjZDhj3OafJ?=
 =?us-ascii?Q?vqU4utNMpDdbZndWXbGD6Z4XQOCDFH2D0Q+CDjs0LgowekWMvfONfcxG8iVM?=
 =?us-ascii?Q?0IYRKXB+C71QCLf70qHnToBT77niWVGQNeIl5vo9HZ1TmDYTRAmkOU65Ie4g?=
 =?us-ascii?Q?G/s+dN+bha+6V5skp3udqiR9vj6LAUnwBCG3S7df1x4+v5DGkHCNMuvONmJk?=
 =?us-ascii?Q?NjrOIw=3D=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 7f7fb852-b756-488b-7947-08d9fb57fc81
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Mar 2022 07:49:16.7643
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: LHCOhSNe3RxJjGTZyf8eQHQyFFAA/5Tz+tWcI/cDEjXnF7zQwUjiVxmJ7hQ2Ax4boMY/KWPHJsbvR2preth73nytVxlTkHeDkxuCAwU9Or4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR10MB3779
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10272 signatures=684655
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxlogscore=780 bulkscore=0
 adultscore=0 phishscore=0 spamscore=0 suspectscore=0 malwarescore=0
 mlxscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2201110000 definitions=main-2203010036
X-Proofpoint-GUID: ubSsuQN-g8M1ou5zInrg0TE3wZ9P8Mtr
X-Proofpoint-ORIG-GUID: ubSsuQN-g8M1ou5zInrg0TE3wZ9P8Mtr
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-wireless.vger.kernel.org>
X-Mailing-List: linux-wireless@vger.kernel.org

Hello John Crispin,

The patch fe98a6137d03: "ath11k: add debugfs for TWT debug calls"
from Jan 31, 2022, leads to the following Smatch static checker
warning:

	drivers/net/wireless/ath/ath11k/debugfs.c:1642 ath11k_debugfs_add_interface()
	warn: 'arvif->debugfs_twt' is an error pointer or valid

drivers/net/wireless/ath/ath11k/debugfs.c
    1637 int ath11k_debugfs_add_interface(struct ath11k_vif *arvif)
    1638 {
    1639         if (arvif->vif->type == NL80211_IFTYPE_AP && !arvif->debugfs_twt) {
    1640                 arvif->debugfs_twt = debugfs_create_dir("twt",
    1641                                                         arvif->vif->debugfs_dir);
--> 1642                 if (!arvif->debugfs_twt || IS_ERR(arvif->debugfs_twt)) {
    1643                         ath11k_warn(arvif->ar->ab,
    1644                                     "failed to create directory %p\n",
    1645                                     arvif->debugfs_twt);

The debugfs_create_dir() function never returns NULL.  It's generally
not supposed to checked for errors.  This code here looks like a
layering violation because it's trying to check if debugfs is already
registered.  But the clean up code just unregisters on the first call.
Should it be ref counted or can the !arvif->debugfs_twt check be
removed?

Also if the user deliberately disabled debugfs then this prints an error
message.

    1646                         arvif->debugfs_twt = NULL;
    1647                         return -1;
    1648                 }
    1649 
    1650                 debugfs_create_file("add_dialog", 0200, arvif->debugfs_twt,
    1651                                     arvif, &ath11k_fops_twt_add_dialog);
    1652 
    1653                 debugfs_create_file("del_dialog", 0200, arvif->debugfs_twt,
    1654                                     arvif, &ath11k_fops_twt_del_dialog);
    1655 
    1656                 debugfs_create_file("pause_dialog", 0200, arvif->debugfs_twt,
    1657                                     arvif, &ath11k_fops_twt_pause_dialog);
    1658 
    1659                 debugfs_create_file("resume_dialog", 0200, arvif->debugfs_twt,
    1660                                     arvif, &ath11k_fops_twt_resume_dialog);
    1661         }
    1662         return 0;
    1663 }

regards,
dan carpenter