From mboxrd@z Thu Jan 1 00:00:00 1970 From: Samuel Thibault Subject: Re: [PATCH] Fix xenconsole's "Could not read tty from store" Date: Tue, 18 Dec 2007 18:07:17 +0000 Message-ID: <20071218180717.GQ4244@implementation.uk.xensource.com> References: <20071217112254.GA5240@implementation.uk.xensource.com> <20071217164246.GF4172@totally.trollied.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Return-path: Content-Disposition: inline In-Reply-To: <20071217164246.GF4172@totally.trollied.org.uk> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: John Levon Cc: xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org John Levon, le Mon 17 Dec 2007 16:42:46 +0000, a écrit : > > + if (tcgetattr(master, &term) < 0) { > > + err = errno; > > Please use the slave not the master. It doesn't matter on Linux but it's > critical on Solaris. What is the Solaris reason for this? It really looks we can't afford closing the slave end on Linux (hence leak it) or not open it at all (and then we can't tcgetattr on it). Samuel