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>] [thread-next>] [day] [month] [year] [list]
Message-ID: <tencent_1726B023EBD3D2ED3BBDF506F6E01D282907@qq.com>
Date: Sat, 10 Jan 2026 15:47:39 +0800
From: cjz <guagua210311@...com>
To: gregkh@...uxfoundation.org
Cc: linux-kernel@...r.kernel.org,
	linux-wireless@...r.kernel.org,
	changjunzheng <guagua210311@...com>
Subject: [PATCH v4 0/3] rtl8723bs: Remove redundant atomic op and replace global io error var

This patch series optimizes the IO error counting logic of the rtl8723bs wireless
driver by removing redundant atomic_t operations (unnecessary for non-concurrent scenarios) and replacing the global continual_io_error variable with 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, remove atomic ops)
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>
---


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ