On 9/14/2025 9:57 AM, Michael Kelley wrote:
From: Alok Tiwari<[email protected]> Sent: Saturday, September 13, 2025 
12:25 PM
The target_cpu_store() function parses the target CPU from the sysfs
buffer using sscanf(). The format string currently uses "%uu", which
is invalid and causes incorrect parsing.
The %uu format string definitely looks invalid, but I'm not seeing
incorrect parsing.  For example, this command works:

# echo 5 >/sys/bus/vmbus/devices/<uuid>/channels/<nn>/cpu

and the target cpu is indeed changed to "5".  A two-digit value also
works:

# echo 14 >/sys/bus/vmbus/devices/<uuid>/channels/<nn>/cpu

What are the details of the incorrect parsing that you are seeing?

Michael

Thanks Michael,

You are right, The compiler ignores the extra "u", so there is no incorrect parsing at runtime. My wording was misleading. This change should be treated as a cleanup/cosmetic change rather than a real fix.


Thanks,
Alok

Reply via email to