Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757229AbYJEQRG (ORCPT ); Sun, 5 Oct 2008 12:17:06 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756657AbYJEQKT (ORCPT ); Sun, 5 Oct 2008 12:10:19 -0400 Received: from earthlight.etchedpixels.co.uk ([81.2.110.250]:48482 "EHLO lxorguk.ukuu.org.uk" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1756599AbYJEQKQ (ORCPT ); Sun, 5 Oct 2008 12:10:16 -0400 From: Alan Cox Subject: [PATCH 35/76] tty: compare the tty winsize To: linux-kernel@vger.kernel.org Date: Sun, 05 Oct 2008 17:10:16 +0100 Message-ID: <20081005161011.1997.66375.stgit@localhost.localdomain> In-Reply-To: <20081005160231.1997.10462.stgit@localhost.localdomain> References: <20081005160231.1997.10462.stgit@localhost.localdomain> User-Agent: StGIT/0.14.2 MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1095 Lines: 27 We always use the real tty one for stuff so the pty one should not be compared. As we propogate window changes to both it doesn't currently matter but will when we tidy up the pty termios logic a bit more --- drivers/char/tty_io.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/char/tty_io.c b/drivers/char/tty_io.c index ac53d7f..9a76db3 100644 --- a/drivers/char/tty_io.c +++ b/drivers/char/tty_io.c @@ -2133,7 +2133,7 @@ int tty_do_resize(struct tty_struct *tty, struct tty_struct *real_tty, /* For a PTY we need to lock the tty side */ mutex_lock(&real_tty->termios_mutex); - if (!memcmp(ws, &tty->winsize, sizeof(*ws))) + if (!memcmp(ws, &real_tty->winsize, sizeof(*ws))) goto done; /* Get the PID values and reference them so we can avoid holding the tty ctrl lock while sending signals */ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/