On Thu, Sep 18, 2025 at 03:51:00AM +0000, Michael Kelley wrote: > 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]>
Applied. Thanks.
