Re: [dpdk-dev] [EXTERNAL] [PATCH 2/6] net/bnxt: fix potential data race

[dpdk-dev] [PATCH 0/6] net/bnxt: bounds checking patches · Stephen Hemminger <stephen@networkplumber.org> · 2020-03-03
[dpdk-dev] [PATCH 1/6] eal: add portable way to check for math overflow · Stephen Hemminger <stephen@networkplumber.org> · 2020-03-03
Re: [dpdk-dev] [PATCH 1/6] eal: add portable way to check for math overflow · Dmitry Kozlyuk <hidden> · 2020-03-03
[dpdk-dev] [PATCH 2/6] net/bnxt: fix potential data race · Stephen Hemminger <stephen@networkplumber.org> · 2020-03-03
Re: [dpdk-dev] [EXTERNAL] [PATCH 2/6] net/bnxt: fix potential data race · Christopher Ertl <hidden> · 2020-03-05
Re: [dpdk-dev] [EXTERNAL] [PATCH 2/6] net/bnxt: fix potential data race · Stephen Hemminger <stephen@networkplumber.org> · 2020-03-03
[dpdk-dev] [PATCH 3/6] net/bnxt: avoid potential out of bounds read · Stephen Hemminger <stephen@networkplumber.org> · 2020-03-03
[dpdk-dev] [PATCH 4/6] net/bnxt: check for integer overflow in buffer sizing · Stephen Hemminger <stephen@networkplumber.org> · 2020-03-03
[dpdk-dev] [PATCH 5/6] net/bnxt: add integer underflow check · Stephen Hemminger <stephen@networkplumber.org> · 2020-03-03
[dpdk-dev] [PATCH 6/6] net/bnxt: sanitize max_l2_ctx · Stephen Hemminger <stephen@networkplumber.org> · 2020-03-03
Re: [dpdk-dev] [PATCH 0/6] net/bnxt: bounds checking patches · Ferruh Yigit <hidden> · 2020-03-31
Re: [dpdk-dev] [PATCH 0/6] net/bnxt: bounds checking patches · Ajit Khaparde <ajit.khaparde@broadcom.com> · 2020-03-31
Re: [dpdk-dev] [PATCH 0/6] net/bnxt: bounds checking patches · Stephen Hemminger <stephen@networkplumber.org> · 2020-03-31
Re: [dpdk-dev] [PATCH 0/6] net/bnxt: bounds checking patches · Thomas Monjalon <hidden> · 2020-10-19

From: Stephen Hemminger <stephen@networkplumber.org>
Date: 2020-03-03 18:16:21

On Tue, 3 Mar 2020 18:13:22 +0000
Christopher Ertl [off-list ref] wrote:

Can we add the `volatile` qualifier to the `hwrm_cmd_resp_addr` member (in drivers/net/bnxt/bnxt.h) to get a stronger guarantee that the compiler won't insert TOCTOU races in the output?

Christopher Ertl | MSRC Vulnerabilities & Mitigations | Microsoft Limited | +44 7773976589 | Christopher.Ertl@microsoft.com
 
Microsoft Limited (company number 01624297) is a company registered in England and Wales whose registered office is at Microsoft Campus, Thames Valley Park, Reading. RG6 1WG

Should this be done for all the registers in that structure?

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help