[PATCH 2/5] serial: expose buf_overrun count through proc interface
Jeremy Kerr
jk at ozlabs.org
Tue Mar 27 12:44:02 AEDT 2018
Hi Greg,
>> diff --git a/drivers/tty/serial/serial_core.c b/drivers/tty/serial/serial_core.c
>> index 8f3dfc8b5307..fc677534b510 100644
>> --- a/drivers/tty/serial/serial_core.c
>> +++ b/drivers/tty/serial/serial_core.c
>> @@ -1780,6 +1780,8 @@ static void uart_line_info(struct seq_file *m, struct uart_driver *drv, int i)
>> seq_printf(m, " brk:%d", uport->icount.brk);
>> if (uport->icount.overrun)
>> seq_printf(m, " oe:%d", uport->icount.overrun);
>> + if (uport->icount.buf_overrun)
>> + seq_printf(m, " bo:%d", uport->icount.buf_overrun);
>
> Hopefully this doesn't break any tools that might want to parse this
> file :)
Given that those existing fe/pe/brk/oe fields only appear when they're
non-zero, existing parsing code needs to be tolerant of different sets
of fields being present/absent. Hopefully that means that a new field
shouldn't be too much of a surprise?
Otherwise, I'm happy to report the buf_overrun count elsewhere, if
there's a more suitable place.
Cheers,
Jeremy
More information about the openbmc
mailing list