Re: [PATCH 0/5] firewire: cdev: refactor for bus reset event filling

From: Takashi Sakamoto

Date: Sun Sep 20 2026 - 22:32:51 EST


On Sun, Sep 20, 2026 at 04:14:56PM +0900, Takashi Sakamoto wrote:
> Hi,
>
> In the cdev layer, there are two places where bus reset events are filed;
> the update path for the associated fw_device instance and the get_info
> ioctl call. There is still some code that can be refactored in these
> paths.
>
> This series refactors the code to make the memory ordering and locking
> scope more explicit, and adds a KUnit test for the bus reset event UAPI
> structure.
>
> Takashi Sakamoto (5):
> firewire: cdev: use memory barrier to access fw_device members
> firewire: cdev: refactor using scoped_guard() for get_info ioctl call
> firewire: cdev: refactor copy_to_user() in get_info ioctl call
> firewire: cdev: add annotations and comments about race condition of
> bus_reset_closure
> firewire: cdev: add KUnit test for bus reset event UAPI structure
>
> drivers/firewire/core-cdev.c | 63 +++++++++++++++++++++++-------------
> drivers/firewire/uapi-test.c | 20 ++++++++++++
> 2 files changed, 60 insertions(+), 23 deletions(-)

Applied to for-next branch.


Regards

Takashi Sakamoto