From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21450 invoked by alias); 14 Jun 2012 10:14:41 -0000 Received: (qmail 21441 invoked by uid 22791); 14 Jun 2012 10:14:40 -0000 X-SWARE-Spam-Status: No, hits=-7.5 required=5.0 tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,KHOP_THREADED,RCVD_IN_DNSWL_HI,RCVD_IN_HOSTKARMA_W,SPF_HELO_PASS,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 14 Jun 2012 10:14:20 +0000 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id q5EAE76A027222 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Thu, 14 Jun 2012 06:14:07 -0400 Received: from [127.0.0.1] (ovpn01.gateway.prod.ext.ams2.redhat.com [10.39.146.11]) by int-mx02.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id q5EAE400016611; Thu, 14 Jun 2012 06:14:05 -0400 Message-ID: <4FD9B96C.9020908@redhat.com> Date: Thu, 14 Jun 2012 10:14:00 -0000 From: Pedro Alves User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120430 Thunderbird/12.0.1 MIME-Version: 1.0 To: Ulrich Weigand CC: Terry Guo , "'Jonathan Larmour'" , Yao Qi , gdb-patches@sourceware.org, tromey@redhat.com, Richard Earnshaw , Joey Ye Subject: Re: [RFC] Enable GDB handle compressed target.xml returned by GDB stub References: <201206131312.q5DDCUfK028160@d06av02.portsmouth.uk.ibm.com> In-Reply-To: <201206131312.q5DDCUfK028160@d06av02.portsmouth.uk.ibm.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2012-06/txt/msg00462.txt.bz2 On 06/13/2012 02:12 PM, Ulrich Weigand wrote: > Terry Guo wrote: > >> Yes, we need to consider xi:includes which means we have to involve a >> global state. > > Not necessarily; the way I had intended my suggestion to work was that > GDB always adds ".gz" (or some other suffix if we actually are not > compatible with the .gz file format) to *every* file it fetches, not > just to the initial target.xml, but also to other files fetched via > xi:include statements ... > > If the compressed version of the file is not available, GDB would > then fall back to the original file name (on a file-by-file basis). That sounds fine. It makes gdb roundtrip to the target twice as much for tdescs in the limit, but maybe that doesn't matter in practice. But note this scheme only works because we're fetching named xml files. What of other xml objects that aren't filename based? I guess it's plausible that we'll find other situations where compressing xml data would be useful. In that perspective, something like the 'try qXfer:features:zread:target.xml first, then qXfer:features:read:target.xml, etc.' alternative sounded attractive. Or should we not bother? -- Pedro Alves