public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: "Ulrich Weigand" <uweigand@de.ibm.com>
To: arnez@linux.vnet.ibm.com (Andreas Arnez)
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH 2/2] S390: Fix gdbserver support for TDB
Date: Mon, 01 Dec 2014 18:40:00 -0000	[thread overview]
Message-ID: <201412011840.sB1Ie6As009411@d03av02.boulder.ibm.com> (raw)
In-Reply-To: <87iohvp77u.fsf@br87z6lw.de.ibm.com> from "Andreas Arnez" at Dec 01, 2014 07:15:17 PM

Andreas Arnez wrote:

> Here's an updated version of the patch.  Instead of clearing the buffer,
> this version passes a NULL pointer to the store function.  And the store
> function for iWMMXt is adjusted accordingly.  Note that I've tested this
> on S390, but not on ARM.

I agree that this version is preferable to the first one.

The patch looks good to me, but I'd like to give other maintainers
(in particular for ARM) a chance to comment before approving it.

Thanks,
Ulrich

-- 
  Dr. Ulrich Weigand
  GNU/Linux compilers and toolchain
  Ulrich.Weigand@de.ibm.com

  reply	other threads:[~2014-12-01 18:40 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-26 11:56 [PATCH 0/2] S390: Fixes for gdbserver on targets with TDB Andreas Arnez
2014-11-26 11:56 ` [PATCH 2/2] S390: Fix gdbserver support for TDB Andreas Arnez
2014-12-01 18:15   ` Andreas Arnez
2014-12-01 18:40     ` Ulrich Weigand [this message]
2014-12-02 15:21     ` Pedro Alves
2014-12-02 19:18       ` Andreas Arnez
2014-12-03 18:18         ` Andreas Arnez
2014-12-04 13:33           ` Pedro Alves
2014-12-09 16:10             ` Andreas Arnez
2014-12-04 13:39         ` Pedro Alves
2014-11-26 11:56 ` [PATCH 1/2] S390: Fix 'expedite' for s390-te-linux64 Andreas Arnez
2014-12-01 18:34   ` Ulrich Weigand

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=201412011840.sB1Ie6As009411@d03av02.boulder.ibm.com \
    --to=uweigand@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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).