On Sat, Oct 27, 2012 at 02:20:28PM +0200, Jiri Slaby wrote: > On 10/23/2012 11:53 AM, Alan Cox wrote: > > 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. > > And that function looks strange alltogether. It's like it's trying to > optimize in case of raw TTY by sending the data directly to ldisc. In a > completely racy way. I think that it should remain marked as BROKEN > until this is converted to tty_prepare_flip_string. That code has been ripped out of the driver now, and it builds fine in my tree, so the BROKEN marking linux-next can be removed. thanks, greg k-h -- 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