..snip..
> > cxl_for_each_cmd(cmd) {
> > const struct cxl_command_info *info = &cmd->info;
> > + int i = 0;
> >
> > - if (copy_to_user(&q->commands[j++], info, sizeof(*info)))
> > + if (copy_to_user(&q->commands[i++], info, sizeof(*info)))
> >
From: Konrad Rzeszutek Wilk
> Sent: 26 February 2021 21:17
>
> .. otherwise people spend extra cycles looking for the
> inner loop and wondering 'why j'?
>
> This was an oversight when initial work was rebased
> so let's fix it here.
>
> Signed-off-by: Konrad Rzeszutek Wilk
> ---
> v1: Initial
.. otherwise people spend extra cycles looking for the
inner loop and wondering 'why j'?
This was an oversight when initial work was rebased
so let's fix it here.
Signed-off-by: Konrad Rzeszutek Wilk
---
v1: Initial posting
v2: Fix per Dan's request
---
drivers/cxl/mem.c | 6 +++---
1 file chan
3 matches
Mail list logo