Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Luis Machado <lgustavo@codesourcery.com>
To: Andreas Arnez <arnez@linux.vnet.ibm.com>
Cc: gdb-patches@sourceware.org, Ulrich.Weigand@de.ibm.com
Subject: Re: [ping 2] [RFA][PATCH v4 0/5] Add TDB regset support
Date: Mon, 15 Jul 2013 13:27:00 -0000	[thread overview]
Message-ID: <51E3F8B3.10109@codesourcery.com> (raw)
In-Reply-To: <874nbwtdgk.fsf_-_@br87z6lw.de.ibm.com>

Hi,

On 07/15/2013 05:49 AM, Andreas Arnez wrote:
> Did I miss anything, or is the patch set OK now?
>
> Andreas Arnez <arnez@linux.vnet.ibm.com> writes:
>
>> Is this good to go now?  Any remaining concerns with the patch set?
>>
>> Andreas Arnez <arnez@linux.vnet.ibm.com> writes:
>>
>>> This is another iteration of GDB support for the "transaction diagnostic
>>> block", TDB, which is provided by the transactional-execution facility
>>> in newer z/Architecture systems.
>>>
>>> [...]
>
>
>

I didn't go through your last update of the patch, but FTR i still think 
we should make the core file sections static and store them in some form 
of array instead of hardcoding their contents in numerous function calls.

Luis



  reply	other threads:[~2013-07-15 13:27 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-03 16:57 Andreas Arnez
2013-07-03 17:00 ` [RFA][PATCH v4 1/5] S/390 regmap rework Andreas Arnez
2013-07-15 14:46   ` Luis Machado
2013-07-15 17:15     ` Andreas Arnez
2013-07-03 17:02 ` [RFA][PATCH v4 2/5] S/390: Add TDB regset Andreas Arnez
2013-07-03 19:25   ` Eli Zaretskii
2013-07-03 17:04 ` [RFA][PATCH v4 3/5] Dynamic core regset sections support Andreas Arnez
2013-07-03 17:11 ` [RFA][PATCH v4 4/5] S/390: Exploit dynamic core regset sections Andreas Arnez
2013-07-03 17:21 ` [RFA][PATCH v4 5/5] PowerPC: " Andreas Arnez
2013-07-08 15:44 ` [ping] [RFA][PATCH v4 0/5] Add TDB regset support Andreas Arnez
2013-07-15  8:49   ` [ping 2] " Andreas Arnez
2013-07-15 13:27     ` Luis Machado [this message]
2013-07-15 15:34       ` Andreas Arnez
2013-07-15 15:40         ` Luis Machado
2013-07-15 16:30           ` Andreas Arnez
2013-07-16 15:26             ` Andreas Arnez
2013-07-16 16:03               ` Ulrich Weigand
2013-07-16 17:06                 ` Andreas Arnez
2013-07-15 15:58         ` Mark Kettenis

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=51E3F8B3.10109@codesourcery.com \
    --to=lgustavo@codesourcery.com \
    --cc=Ulrich.Weigand@de.ibm.com \
    --cc=arnez@linux.vnet.ibm.com \
    --cc=gdb-patches@sourceware.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox