On Tue, 23 Oct 2012 10:30:54 +0800 Fengguang Wu <fengguang.wu@xxxxxxxxx> wrote: > Hi Jiri, > > FYI, kernel build failed on > > tree: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git tty-next > head: ecbbfd44a08fa80e0d664814efd4c187721b85f6 > commit: ba2e68ac6157004ee4922fb39ebd9459bbae883e [17/23] TTY: move ldisc data from tty_struct: read_* and echo_* and canon_* stuff > config: x86_64-allmodconfig # make ARCH=x86_64 allmodconfig > > All error/warnings: > > drivers/staging/dgrp/dgrp_net_ops.c: In function 'dgrp_input': > drivers/staging/dgrp/dgrp_net_ops.c:216:27: error: 'struct tty_struct' has no member named 'real_raw' > drivers/staging/dgrp/dgrp_net_ops.c:229:8: error: 'struct tty_struct' has no member named 'read_cnt' > drivers/staging/dgrp/dgrp_net_ops.c:229:8: error: 'struct tty_struct' has no member named 'read_cnt' > drivers/staging/dgrp/dgrp_net_ops.c:261:30: error: 'struct tty_struct' has no member named 'real_raw' > drivers/staging/dgrp/dgrp_net_ops.c:276:28: error: 'struct tty_struct' has no member named 'real_raw' > > vim +229 drivers/staging/dgrp/dgrp_net_ops.c > > 0b52b749 Bill Pemberton 2012-09-20 228 /* take into consideration length of ldisc */ > 0b52b749 Bill Pemberton 2012-09-20 @229 len = min(len, (N_TTY_BUF_SIZE - 1) - tty->read_cnt); This is broken and unsafe. It's always been broken and unsafe. Probably the report wants directing to whoever signed up to fix it all in staging. Alan -- 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