linux-kernel - Re: [PATCH v2] Bluetooth: 6lowpan: use DEV_STAT

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-prev] [day] [month] [year] [list]

Date: Mon, 27 May 2024 09:03:12 +0200
From: Paul Menzel <pmenzel@...gen.mpg.de>
To: jiangyunshui@...inos.cn
Cc: linux-kernel@...r.kernel.org, linux-bluetooth@...r.kernel.org,
 marcel@...tmann.org, johan.hedberg@...il.com, luiz.dentz@...il.com
Subject: Re: [PATCH v2] Bluetooth: 6lowpan: use DEV_STAT_INC() to avoid races

Dear yunshui,


Thank you for your patch. One formal request:

Am 27.05.24 um 08:52 schrieb yunshui:
> syzbot/KCSAN reported that races happen when multiple cpus
> updating dev->stats.tx_error concurrently.
> 
> Adopt SMP safe DEV_STATS_INC() to update dev->stats fields.
> 
> Signed-off-by: yunshui <jiangyunshui@...inos.cn>

Could you please use your full name? Maybe:

     $ git config --global user.name "Jiang Yunshui"
     $ git commit --amend --author="Jiang Yunshui <jiangyunshui@...inos.cn>"

[…]


Kind regards,

Paul