655603de68
The sysfs handler should return the number of bytes consumed, which in the
case of a successful write is the entire buffer. Also fix a bug where
param.data_len was being set to (count - (2 * sizeof(u32))) instead of just
(count - sizeof(u32)). The latter is correct because we skip over the
leading u32 which is our param.type, but we were also incorrectly
subtracting sizeof(u32) on the line where we were actually setting
param.data_len:
param.data_len = count - sizeof(u32);
This meant that for our example event.kernel_software_watchdog with total
length 10 bytes, param.data_len was just 2 prior to this change.
To test, successfully append an event to the log with gsmi sysfs.
This sample event is for a "Kernel Software Watchdog"
> xxd -g 1 event.kernel_software_watchdog
|
||
|---|---|---|
| .. | ||
| coreboot_table.c | ||
| coreboot_table.h | ||
| framebuffer-coreboot.c | ||
| gsmi.c | ||
| Kconfig | ||
| Makefile | ||
| memconsole-coreboot.c | ||
| memconsole-x86-legacy.c | ||
| memconsole.c | ||
| memconsole.h | ||
| vpd_decode.c | ||
| vpd_decode.h | ||
| vpd.c | ||