From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3693 invoked by alias); 9 Aug 2012 10:12:26 -0000 Received: (qmail 3675 invoked by uid 22791); 9 Aug 2012 10:12:24 -0000 X-SWARE-Spam-Status: No, hits=-2.8 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED X-Spam-Check-By: sourceware.org Received: from hagrid.ecoscentric.com (HELO mail.ecoscentric.com) (212.13.207.197) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 09 Aug 2012 10:12:05 +0000 Received: from localhost (hagrid.ecoscentric.com [127.0.0.1]) by mail.ecoscentric.com (Postfix) with ESMTP id CA718151A01A for ; Thu, 9 Aug 2012 11:12:04 +0100 (BST) Received: from mail.ecoscentric.com ([127.0.0.1]) by localhost (hagrid.ecoscentric.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ghnFgudZc+hK; Thu, 9 Aug 2012 11:12:03 +0100 (BST) From: bugzilla-daemon@bugs.ecos.sourceware.org To: ecos-patches@ecos.sourceware.org Subject: [Bug 1001607] Cortex-M4F architectural Floating Point Support X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: eCos X-Bugzilla-Component: Patches and contributions X-Bugzilla-Keywords: X-Bugzilla-Severity: enhancement X-Bugzilla-Who: ilijak@siva.com.mk X-Bugzilla-Status: NEEDINFO X-Bugzilla-Priority: low X-Bugzilla-Assigned-To: jifl@ecoscentric.com X-Bugzilla-Target-Milestone: --- X-Bugzilla-Changed-Fields: Attachment #1881 is obsolete In-Reply-To: References: X-Bugzilla-URL: http://bugs.ecos.sourceware.org/ Auto-Submitted: auto-generated Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Date: Thu, 09 Aug 2012 10:12:00 -0000 Message-Id: <20120809101203.CF8EF1519FFB@mail.ecoscentric.com> Mailing-List: contact ecos-patches-help@ecos.sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: ecos-patches-owner@ecos.sourceware.org X-SW-Source: 2012-08/txt/msg00026.txt.bz2 Please do not reply to this email. Use the web interface provided at: http://bugs.ecos.sourceware.org/show_bug.cgi?id=3D1001607 Ilija Kocho changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #1881|0 |1 is obsolete| | --- Comment #23 from Ilija Kocho 2012-08-09 11:11:57 B= ST --- Created an attachment (id=3D1882) --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=3D1882) Enter default VSR from Usagefault VSR if not FPU exception. 120809 (In reply to comment #22) > (In reply to comment #19) > > (In reply to comment #17) >=20 > Looking at the patch you posted in comment #21, do you need to save r2/r3= at > all? In fact since hal_deliver_usagefault_exception doesn't use its 'regs' > argument, should we drop that and only be saving, I guess, r12 and lr?=20 Although > we have to stash r0 into r4 as before, so that too. All the other registe= rs > should be unaffected by hal_deliver_usagefault_exception, since unlike ot= her > user exception handling by default_exception_vsr, we know what this handl= er > does. And basepri should also be unaffected I would have thought. You are right, now that exception's job is predetermined, it can be optimiz= ed. We don't have to save all registers, indeed, and r12 can be omitted (auto-s= aved by exception), also maybe lr - but we need to keep stack aligned to 8 bytes anyway. [snip] > I think with this route then hal_usagefault_exception_vsr and > hal_deliver_usagefault_exception should be renamed to > hal_fpu_usagefault_exception_vsr and hal_deliver_fpu_usagefault_exception > instead.=20 >=20 I renamed hal_deliver_usagefault..., but technically hal_usagefault_exception_vsr still handles all usagefault exceptions, though some of them with help of hal_default_exception_vsr I am going to implement other changes for which which I think that are clea= r, and for others I will reply in another post. =D0=9F=D0=BE=D0=B7=D0=B4=D1=80=D0=B0=D0=B2 =D0=98=D0=BB=D0=B8=D1=98=D0=B0 --=20 Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=3Demail ------- You are receiving this mail because: ------- You are on the CC list for the bug.