Wrong printf-flag in log message

[Adek336]

src/system/kernel/int.c

    219 			} else {
    220 				// this is a shared interrupt vector, we cannot just disable it
-   221 				dprintf("More than 99%% interrupts of vector %d are unhandled\n",
+   221 				dprintf("More than 99\% interrupts of vector %d are unhandled\n",
    222 					vector);
    223 			}

I think this solves the erroneous log messages that go like

More than 99-1nterrupts of...

instead of

More than 99% interrupts of...

[anevilyak]

Mm, no, \% results in the compiler warning about incorrect number of arguments for format.

also %% works correctly with printf from userland code, so something else seems to be wrong here.

[axeld]

Indeed, "%%" is perfectly legal, just our vsnprintf() did not handle it correctly.

Fixed in hrev28323.

[Adek336]

Doesn't yet work:

dprintf("ziom%%ziom\n");
dprintf("%% in\n");
dprintf("yo %% ziom\n");

yields

ziom%ziom
 1701523315n
yo %ziom

where the last two are wrong.

[axeld]

Now that looks nice :-) I'll have another look, thanks for testing!

[axeld]

You obviously tested with a wrong release, this all works flawlessly here.