[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20231005082217.GA12575@lst.de>
Date: Thu, 5 Oct 2023 10:22:17 +0200
From: Christoph Hellwig <hch@....de>
To: Kees Cook <keescook@...omium.org>
Cc: James Smart <james.smart@...adcom.com>,
Christoph Hellwig <hch@....de>,
Sagi Grimberg <sagi@...mberg.me>,
Chaitanya Kulkarni <kch@...dia.com>,
linux-nvme@...ts.infradead.org,
"Gustavo A. R. Silva" <gustavoars@...nel.org>,
Nathan Chancellor <nathan@...nel.org>,
Nick Desaulniers <ndesaulniers@...gle.com>,
Tom Rix <trix@...hat.com>, linux-kernel@...r.kernel.org,
linux-hardening@...r.kernel.org, llvm@...ts.linux.dev
Subject: Re: [PATCH] nvmet-fc: Annotate struct nvmet_fc_tgt_queue with
__counted_by
On Tue, Oct 03, 2023 at 04:25:11PM -0700, Kees Cook wrote:
> struct workqueue_struct *work_q;
> struct kref ref;
> struct rcu_head rcu;
> - struct nvmet_fc_fcp_iod fod[]; /* array of fcp_iods */
> + struct nvmet_fc_fcp_iod fod[] __counted_by(sqsize); /* array of fcp_iods */
Please move the comment up to not spill over the end of the line.
Otherwise this looks good.
Powered by blists - more mailing lists