Tejun Heo wrote:
> On Sat, Apr 18, 2015 at 03:03:46AM +0900, Tetsuo Handa wrote:
> > packet will be sufficient for finding out whether the packets were lost
> > and/or
> > reordered in flight.
> >
> > printk("Hello");
> > => netconsole sends "00000000 Hello" using UDP
> > printk("netconsole");
> > => netconsole sends "00000001 netconsole" using UDP
> > printk("world\n");
> > => netconsole sends "00000002 world\n" using UDP
> >
> > It might be nice to allow administrator to prefix a sequence number
> > to netconsole messages for those who are using special receiver
> > program (e.g. ncrx) which checks that sequence number.
>
> That said, this is pretty much what the first 12 patches do (except
> for the last printk patch, which can be taken out). We already have
> sequencing and established format to expose them to userland - try cat
> /dev/kmsg, which btw is what local loggers on modern systems use
> anyway. Why introduce netconsole's own version of metadata?
I didn't mean to introduce netconsole's own version of metadata.
I meant we don't need to implement in-kernel retry logic.
If we can assume that scheduler is working, adding a kernel thread that
does
while (1) {
read messages with metadata from /dev/kmsg
send them using UDP network
}
might be easier than modifying netconsole module.
>
> Thanks.
>
> --
> tejun
>
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
