Re: [PATCH] cris: lower the printk level in cris serial driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> >        if (info->recv_cnt + recvl > 65536) {
> > -               printk(KERN_CRIT
> > +               printk(KERN_INFO
> >                       "%s: Too much pending incoming serial data! Dropping %u bytes.\n", __func__, recvl);
> >                return 0;
> >        }
> 
> I think dropped data warrants KERN_WARN.

If you have an overflow you want to push the relevant tty flag into the
queue. I'm not sure a printk is that useful - end users of embedded
devices (or even PC desktops) won't see it anyway and if you have serial
console it can actually make things worse

Plus of course it's not rate limited here so its a log wiping vector ...
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux