From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-59575-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 17246 invoked by alias); 29 Oct 2008 07:41:29 -0000
Received: (qmail 17237 invoked by uid 22791); 29 Oct 2008 07:41:28 -0000
X-Spam-Check-By: sourceware.org
Received: from sibelius.xs4all.nl (HELO sibelius.xs4all.nl) (82.92.89.47)     by sourceware.org (qpsmtpd/0.31) with ESMTP; Wed, 29 Oct 2008 07:40:53 +0000
Received: from brahms.sibelius.xs4all.nl (kettenis@localhost.sibelius.xs4all.nl [127.0.0.1]) 	by brahms.sibelius.xs4all.nl (8.14.3/8.14.3) with ESMTP id m9T7ekEL006656; 	Wed, 29 Oct 2008 08:40:46 +0100 (CET)
Received: (from kettenis@localhost) 	by brahms.sibelius.xs4all.nl (8.14.3/8.14.3/Submit) id m9T7eja2032714; 	Wed, 29 Oct 2008 08:40:45 +0100 (CET)
Date: Wed, 29 Oct 2008 16:45:00 -0000
Message-Id: <200810290740.m9T7eja2032714@brahms.sibelius.xs4all.nl>
From: Mark Kettenis <mark.kettenis@xs4all.nl>
To: hjl.tools@gmail.com
CC: drow@false.org, gdb-patches@sourceware.org
In-reply-to: <6dc9ffc80810281915g318fa5cfm4646f1edb910d55e@mail.gmail.com> 	(hjl.tools@gmail.com)
Subject: Re: PATCH: Extend gdb remote protocol for AVX
References: <200810021026.m92AQMqC006955@brahms.sibelius.xs4all.nl> 	 <20081004221325.GA6856@caradoc.them.org> 	 <6dc9ffc80810050737r56b0d044vcf8e8f1368d2d03d@mail.gmail.com> 	 <200810062135.m96LZI0H016325@brahms.sibelius.xs4all.nl> 	 <6dc9ffc80810071221r10439dd2m886637a1dc7327ab@mail.gmail.com> 	 <200810121337.m9CDbu7Y031268@brahms.sibelius.xs4all.nl> 	 <6dc9ffc80810121517n1581323cl21808a5f99cce13d@mail.gmail.com> 	 <6dc9ffc80810272258m60f6d1c6w54b012ee90346b21@mail.gmail.com> 	 <20081028141021.GA21659@caradoc.them.org> 	 <200810281626.m9SGQfbl011775@brahms.sibelius.xs4all.nl> <6dc9ffc80810281915g318fa5cfm4646f1edb910d55e@mail.gmail.com>
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
X-SW-Source: 2008-10/txt/msg00691.txt.bz2

> Date: Tue, 28 Oct 2008 19:15:51 -0700
> From: "H.J. Lu" <hjl.tools@gmail.com>
> 
> On Tue, Oct 28, 2008 at 9:26 AM, Mark Kettenis <mark.kettenis@xs4all.nl> wrote:
> >> Date: Tue, 28 Oct 2008 10:10:21 -0400
> >> From: Daniel Jacobowitz <drow@false.org>
> >>
> >> On Mon, Oct 27, 2008 at 10:58:36PM -0700, H.J. Lu wrote:
> >> > Are there any objections to my AVX patch?
> >>
> >> There were objections - just read the thread you are replying to.  If
> >> you have not changed the patch, then there are still objections.
> >
> > I also think it would be good to seperate out the changes to GDB's
> > internal register file from the extensions you want to make to the
> 
> Here is the patch to extend gdb remote protocol for AVX.
> 
> > remote protocol.  And I'd really like to see how an implementation of
> > i387_supply_xsave() and i387_collect_xsave() would look in that
> > context.
> 
> It is hard to implement them without an OS running on AVX
> H/W. At this moment, we only need to define a remote protocol
> for AVX.  We can work on native AVX support when H/W is
> available.

No we cannot.  You're making changes to how GDB deals internally with
these registers.  You need take native AUVX support into account when
you want to change those too.