linux-kernel - [PATCH v4 0/3] rtl8723bs: Optimize error counting logic (remove global variable)

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <tencent_3BA74A77EF2AD7EDD9A776C6A58EC0EAAD09@qq.com>
Date: Mon, 22 Dec 2025 09:22:45 +0800
From: cjz <guagua210311@...com>
To: linux-staging@...r.kernel.org
Cc: gregkh@...uxfoundation.org,
	linux-wireless@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	changjunzheng <guagua210311@...com>
Subject: [PATCH v4 0/3] rtl8723bs: Optimize error counting logic (remove global variable)

This patch series optimizes the IO error counting logic of the rtl8723bs wireless
driver by removing the unnecessary global variable and adopting local error
counting. The series is split into logical patches (per Greg KH's review comment)
to ensure each patch fulfills a single, clear purpose, which greatly improves
reviewability and future maintainability.

Patch Breakdown:
1. 0001: Add independent rtw_check_continual_io_error function (single responsibility, includes old function cleanup)
2. 0002: Use local error_count in sd_read32/sd_write32 (eliminate global dependency)
3. 0003: Remove redundant global continual_io_error variable (clean up struct)

Changelog from v3 to v4:
- Split the monolithic v3 patch into logical, single-purpose patches
- Fixed all coding style errors (trailing spaces, missing assignment spaces, indentation, blank line alignment)
- Added detailed, purpose-driven commit messages for each patch
- Added version changelog as required by kernel documentation
- Added this cover letter to explain the entire patch series
- Optimized the new check function to remove redundant ternary expression (more concise kernel style)

Thank you for your time and review!

Signed-off-by: changjunzheng <guagua210311@...com>
---