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