WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] libxl: drop bootloader output if xenconsole file

To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] libxl: drop bootloader output if xenconsole file descriptor is not available for writing
From: Roger Pau Monné <roger.pau@xxxxxxxxxxxxx>
Date: Mon, 10 Oct 2011 16:17:37 +0200
Cc: Ian@xxxxxxxxxx, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Jackson <Ian.Jackson@xxxxxxxxxxxxx>, Ian Campbell <Ian.Campbell@xxxxxxxxxx>
Delivery-date: Mon, 10 Oct 2011 07:18:26 -0700
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=PvUTwk8HS4flQUTxUV5McuUVCdySX7cjKQXnOJI6CxE=; b=NbnEY0KdK/7y6knRE5lLF+9psjbaBoMg+zz+MFL6pW6BFdFOO8H6yEdmDx3QvsReL7 +4h1FLM5nlhuV2RlBi2Hgbafp/usPu2VBaWCI4CJEDzODn2K+c630SjZjbK8vitSjbN4 mxnPmNLmbwxTykrhr0nEhkZ2sNN3R0mYHpnNk=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4E92FD1D.8070400@xxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <1ebcb5d27ead38d6713d.1318255457@loki> <4E92FD1D.8070400@xxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
2011/10/10 Andrew Cooper <andrew.cooper3@xxxxxxxxxx>:
>
>
> On 10/10/11 15:04, Roger Pau Monne wrote:
>> # HG changeset patch
>> # User Roger Pau Monne <roger.pau@xxxxxxxxxxxxx>
>> # Date 1318255452 -7200
>> # Node ID 1ebcb5d27ead38d6713d865704394ca99444f93c
>> # Parent  a5b8d5c8c98e0dd17368eea1801a8edaf1e79e69
>> libxl: drop bootloader output if xenconsole file descriptor is not available 
>> for writing.
>>
>> Drop output from bootloader if the pty buffer is full and we cannot write 
>> more data. Prevents the bootloader from getting stuck when using ptys with 
>> small buffers.
>>
>> Signed-off-by: Roger Pau Monne <roger.pau@xxxxxxxxxxxxx>
>>
>> diff -r a5b8d5c8c98e -r 1ebcb5d27ead tools/libxl/libxl_bootloader.c
>> --- a/tools/libxl/libxl_bootloader.c  Mon Oct 10 11:26:22 2011 +0200
>> +++ b/tools/libxl/libxl_bootloader.c  Mon Oct 10 16:04:12 2011 +0200
>> @@ -169,6 +169,7 @@ static char * bootloader_interact(libxl_
>>
>>      size_t nr_out = 0, size_out = 0;
>>      char *output = NULL;
>> +    struct timeval timeout;
>>
>>      /* input from xenconsole. read on xenconsoled_fd write to bootloader_fd 
>> */
>>      int xenconsoled_prod = 0, xenconsoled_cons = 0;
>> @@ -177,9 +178,14 @@ static char * bootloader_interact(libxl_
>>      int bootloader_prod = 0, bootloader_cons = 0;
>>      char bootloader_buf[BOOTLOADER_BUF_SIZE];
>>
>> +    /* Set timeout to 1s before starting to discard data */
>> +    timeout.tv_sec = 1;
>> +    timeout.tv_usec = 0;
>> +
>>      while(1) {
>>          fd_set wsel, rsel;
>>          int nfds;
>> +        int space;
>
> You appear to introduce an unreferenced variable here.

Yes I've missed that one, sorry, second patch is ok.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel