From: "Egger, Christoph" <chegger@amazon.de>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: George Dunlap <george.dunlap@eu.citrix.com>,
Frediano Ziglio <frediano.ziglio@citrix.com>,
Matthew Daley <mattjd@gmail.com>,
Miguel Clara <miguelmclara@gmail.com>,
xen-devel@lists.xen.org
Subject: Re: [PATCH] gcov: Support gcc 4.7
Date: Mon, 17 Jun 2013 12:49:26 +0200 [thread overview]
Message-ID: <51BEE9B6.7080702@amazon.de> (raw)
In-Reply-To: <1371465779.23802.31.camel@zakaz.uk.xensource.com>
On 17.06.13 12:42, Ian Campbell wrote:
> On Mon, 2013-06-17 at 09:29 +0100, Frediano Ziglio wrote:
>> gcc 4.7 changed format used internally for coverage data.
>> This patch address these changes.
>> The information that gcc generate are mostly the same but to support common
>> sections.
>> The only difference in the blob exported by Xen is that functions have 2
>> different checksums instead of one.
>
> Do we really need to reflect that in the Xen public API though? Could we
> condense it into the things we want/need and try and keep our API
> relatively stable?
>
> I'm a bit concerned that we are going to end up with dozens of variants
> of the API (and the internal code) supporting each new variant of this
> internal-to-gcc ABI.
>
> I think you explained before why this couldn't be exported from the
> hypervisor as an opaque set of bytes to be interpreted by a userspace
> tool, which is a shame.
I think this is possible the same way the hypervisor exports
machine check error telemetry.
Christoph
prev parent reply other threads:[~2013-06-17 10:49 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-17 8:29 [PATCH] gcov: Support gcc 4.7 Frediano Ziglio
2013-06-17 8:58 ` Jan Beulich
2013-06-28 10:18 ` Frediano Ziglio
2013-06-28 10:26 ` Jan Beulich
2013-06-17 9:50 ` George Dunlap
2013-06-17 10:42 ` Ian Campbell
2013-06-17 10:46 ` Frediano Ziglio
2013-06-17 10:49 ` Ian Campbell
2013-06-17 12:25 ` Frediano Ziglio
2013-06-18 14:32 ` Frediano Ziglio
2013-06-18 14:53 ` George Dunlap
2013-06-18 15:04 ` Frediano Ziglio
2013-06-17 10:49 ` Egger, Christoph [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51BEE9B6.7080702@amazon.de \
--to=chegger@amazon.de \
--cc=Ian.Campbell@citrix.com \
--cc=frediano.ziglio@citrix.com \
--cc=george.dunlap@eu.citrix.com \
--cc=mattjd@gmail.com \
--cc=miguelmclara@gmail.com \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.