From: Alok Tiwari <[email protected]> Sent: Wednesday, September 17, 2025 2:00 AM > > The target_cpu_store() function parses the target CPU from the sysfs > buffer using sscanf(). The format string currently uses "%uu", which > is redundant. The compiler ignores the extra "u", so there is no > incorrect parsing at runtime. > > Update the format string to use "%u" for clarity and consistency. > > Signed-off-by: Alok Tiwari <[email protected]> > --- > v1 -> v2 > Rephrase commit message and subject to clarify > that there is no incorrect parsing at runtime. > --- > drivers/hv/vmbus_drv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c > index 5b4f8d009ca5..69591dc7bad2 100644 > --- a/drivers/hv/vmbus_drv.c > +++ b/drivers/hv/vmbus_drv.c > @@ -1742,7 +1742,7 @@ static ssize_t target_cpu_store(struct vmbus_channel > *channel, > u32 target_cpu; > ssize_t ret; > > - if (sscanf(buf, "%uu", &target_cpu) != 1) > + if (sscanf(buf, "%u", &target_cpu) != 1) > return -EIO; > > cpus_read_lock(); > -- > 2.50.1
Reviewed-by: Michael Kelley <[email protected]>
