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] Re: Error reporting capabilities for libxc

To: Ian Pratt <m+Ian.Pratt@xxxxxxxxxxxx>
Subject: Re: [Xen-devel] Re: Error reporting capabilities for libxc
From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
Date: Mon, 23 Oct 2006 21:57:30 +0100
Cc: Anthony Liguori <aliguori@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 23 Oct 2006 13:57:54 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <3AAA99889D105740BE010EB6D5A5A3B205073B@xxxxxxxxxxxxxxxxxxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <453D1554.8000202@xxxxxxxxxx> <20061023192753.GG25795@xxxxxxxxxx> <3AAA99889D105740BE010EB6D5A5A3B205073B@xxxxxxxxxxxxxxxxxxxxxxxxxx>
Reply-to: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.4.1i
On Mon, Oct 23, 2006 at 09:54:12PM +0100, Ian Pratt wrote:
> > One further issue I've thought of is that the __thread annotation does
> > not provide any way to do cleanup when a thread exists. Fine if the
> > per-thread variable is a simple scalar, but not so good if its a char
> *
> > since I think it'll leak memory. So I think I may have no choice by to
> > re-write with pthread_getspecific(), since that allows registration
> > of a cleanup function to free memory.
> 
> Do we actually need pointers? I'd much rather go with __thread.

Well, we need a char buffer to store the error message, since that has much
more useful info than the error code alone. I could always just do a fixed
200 byte buffer, and truncate anything longer than this which would also
actually remove the annoying OOM problem in copying the error message. 

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 

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