On Thu, Mar 24, 2016 at 05:41:43PM +0300, Vladimir Davydov wrote:
> On Fri, Mar 18, 2016 at 09:45:05PM +0300, Cyrill Gorcunov wrote:
> > @@ -2043,8 +2044,7 @@ static ssize_t n_tty_write(struct tty_st
> > }
> > #ifdef CONFIG_VE
> > if (tty_hung_up_p(file) ||
> > - (tty->link && !tty->link->count &&
> > - !(test_bit(TTY_PINNED_BY_OTHER, &tty->link->flags)))) {
> > + (tty->link && !tty->link->count &&
> > !vtty_is_master(tty->link))) {
> > #else
>
> I'd got rid of this ifdef while we are here.
Please no. It's easier to catch merge conflicts having CONFIG_VE.
> > +bool vtty_is_master(struct tty_struct *tty)
> > +{
> > + return tty->driver == vttym_driver;
> > +}
> > +EXPORT_SYMBOL(vtty_is_master);
> > +
>
> No point to export it. Anyway,
Yeah. Kostya, could you simply drop it when merging.
>
> Reviewed-by: Vladimir Davydov <[email protected]>
Cyrill
_______________________________________________
Devel mailing list
[email protected]
https://lists.openvz.org/mailman/listinfo/devel
