public inbox for ecos-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug 1001558] New: GDB stub fails to send some registers.
@ 2012-04-11 20:42 bugzilla-daemon
2012-04-11 20:46 ` [Bug 1001558] " bugzilla-daemon
` (6 more replies)
0 siblings, 7 replies; 9+ messages in thread
From: bugzilla-daemon @ 2012-04-11 20:42 UTC (permalink / raw)
To: ecos-bugs
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001558
Summary: GDB stub fails to send some registers.
Product: eCos
Version: CVS
Platform: All
OS/Version: Other
Status: NEW
Severity: enhancement
Priority: low
Component: GDB stubs
AssignedTo: unassigned@bugs.ecos.sourceware.org
ReportedBy: ilijak@siva.com.mk
CC: jifl@ecoscentric.com, ecos-bugs@ecos.sourceware.org,
john@dallaway.org.uk
Class: Advice Request
GDB stub sometimes fails to send all registers. I encountered this problem in a
course of providing support for Cortex-M4F VFP registers.
The cause is function in hal/common/current/src/thread-packets.c that sends
NUMREGS words that may be not correct if registers are not all same size and/or
some registers have been skipped.
Ilija
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug 1001558] GDB stub fails to send some registers.
2012-04-11 20:42 [Bug 1001558] New: GDB stub fails to send some registers bugzilla-daemon
@ 2012-04-11 20:46 ` bugzilla-daemon
2012-04-11 20:52 ` bugzilla-daemon
` (5 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: bugzilla-daemon @ 2012-04-11 20:46 UTC (permalink / raw)
To: ecos-bugs
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001558
--- Comment #1 from Ilija Kocho <ilijak@siva.com.mk> 2012-04-11 21:46:05 BST ---
(In reply to comment #0)
> GDB stub sometimes fails to send all registers. I encountered this problem in a
> course of providing support for Cortex-M4F VFP registers.
>
> The cause is function in hal/common/current/src/thread-packets.c that sends
> NUMREGS words that may be not correct if registers are not all same size and/or
> some registers have been skipped.
>
> Ilija
Here is more detailed description:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001524#c14
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug 1001558] GDB stub fails to send some registers.
2012-04-11 20:42 [Bug 1001558] New: GDB stub fails to send some registers bugzilla-daemon
2012-04-11 20:46 ` [Bug 1001558] " bugzilla-daemon
@ 2012-04-11 20:52 ` bugzilla-daemon
2012-04-12 16:15 ` bugzilla-daemon
` (4 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: bugzilla-daemon @ 2012-04-11 20:52 UTC (permalink / raw)
To: ecos-bugs
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001558
--- Comment #2 from Ilija Kocho <ilijak@siva.com.mk> 2012-04-11 21:51:52 BST ---
Created an attachment (id=1693)
--> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1693)
thread-packets.c patch
This patch seems like solution but has been tested only with Cortex-M4F target
(VFP4-SP-D16). For the time being i mark it as "works for me".
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug 1001558] GDB stub fails to send some registers.
2012-04-11 20:42 [Bug 1001558] New: GDB stub fails to send some registers bugzilla-daemon
2012-04-11 20:46 ` [Bug 1001558] " bugzilla-daemon
2012-04-11 20:52 ` bugzilla-daemon
@ 2012-04-12 16:15 ` bugzilla-daemon
2012-04-12 16:17 ` bugzilla-daemon
` (3 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: bugzilla-daemon @ 2012-04-12 16:15 UTC (permalink / raw)
To: ecos-bugs
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001558
--- Comment #3 from Jonathan Larmour <jifl@ecoscentric.com> 2012-04-12 17:15:29 BST ---
HAL_STUB_REGISTERS_SIZE is not guaranteed to be defined. See hal_stub.c for
example, which sets it to NUMREGS by default if undefined.
That bit (with its accompanying comment) should be moved to hal_stub.h,
somewhere after the inclusion of the platform stub header.
Apart from that, I think the change is ok.
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug 1001558] GDB stub fails to send some registers.
2012-04-11 20:42 [Bug 1001558] New: GDB stub fails to send some registers bugzilla-daemon
` (2 preceding siblings ...)
2012-04-12 16:15 ` bugzilla-daemon
@ 2012-04-12 16:17 ` bugzilla-daemon
2012-04-25 18:53 ` bugzilla-daemon
` (2 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: bugzilla-daemon @ 2012-04-12 16:17 UTC (permalink / raw)
To: ecos-bugs
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001558
Jonathan Larmour <jifl@ecoscentric.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #1693| |assignment+, review-
Flag| |
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug 1001558] GDB stub fails to send some registers.
2012-04-11 20:42 [Bug 1001558] New: GDB stub fails to send some registers bugzilla-daemon
` (3 preceding siblings ...)
2012-04-12 16:17 ` bugzilla-daemon
@ 2012-04-25 18:53 ` bugzilla-daemon
2012-04-25 19:16 ` bugzilla-daemon
2012-04-25 19:58 ` bugzilla-daemon
6 siblings, 0 replies; 9+ messages in thread
From: bugzilla-daemon @ 2012-04-25 18:53 UTC (permalink / raw)
To: ecos-bugs
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001558
Ilija Kocho <ilijak@siva.com.mk> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #1693|0 |1
is obsolete| |
--- Comment #4 from Ilija Kocho <ilijak@siva.com.mk> 2012-04-25 19:52:40 BST ---
Created an attachment (id=1717)
--> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1717)
hal_stub_120425
(In reply to comment #3)
> HAL_STUB_REGISTERS_SIZE is not guaranteed to be defined. See hal_stub.c for
> example, which sets it to NUMREGS by default if undefined.
>
> That bit (with its accompanying comment) should be moved to hal_stub.h,
> somewhere after the inclusion of the platform stub header.
>
> Apart from that, I think the change is ok.
Here it is. By the way, one warning "set but unused variable" gets fixed.
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug 1001558] GDB stub fails to send some registers.
2012-04-11 20:42 [Bug 1001558] New: GDB stub fails to send some registers bugzilla-daemon
` (4 preceding siblings ...)
2012-04-25 18:53 ` bugzilla-daemon
@ 2012-04-25 19:16 ` bugzilla-daemon
2012-04-25 19:58 ` bugzilla-daemon
6 siblings, 0 replies; 9+ messages in thread
From: bugzilla-daemon @ 2012-04-25 19:16 UTC (permalink / raw)
To: ecos-bugs
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001558
--- Comment #5 from Jonathan Larmour <jifl@ecoscentric.com> 2012-04-25 20:16:03 BST ---
(In reply to comment #4)
> Here it is. By the way, one warning "set but unused variable" gets fixed.
Thanks.
A really trivial thing, but you took out the separator lines in hal_stub.c too:
-//-----------------------------------------------------------------------------
-// Extra eCos data.
-
Also there was a spelling mistake in the original comment: suffucient should
be sufficient. With those, please commit. No need to post an updated patch
here.
Jifl
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug 1001558] GDB stub fails to send some registers.
2012-04-11 20:42 [Bug 1001558] New: GDB stub fails to send some registers bugzilla-daemon
` (5 preceding siblings ...)
2012-04-25 19:16 ` bugzilla-daemon
@ 2012-04-25 19:58 ` bugzilla-daemon
6 siblings, 0 replies; 9+ messages in thread
From: bugzilla-daemon @ 2012-04-25 19:58 UTC (permalink / raw)
To: ecos-bugs
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001558
--- Comment #6 from Ilija Kocho <ilijak@siva.com.mk> 2012-04-25 20:57:49 BST ---
(In reply to comment #5)
> (In reply to comment #4)
> > Here it is. By the way, one warning "set but unused variable" gets fixed.
>
> Thanks.
>
> A really trivial thing, but you took out the separator lines in hal_stub.c too:
>
> -//-----------------------------------------------------------------------------
> -// Extra eCos data.
> -
Oops.
Thanks.
>
> Also there was a spelling mistake in the original comment: suffucient should
> be sufficient. With those, please commit. No need to post an updated patch
> here.
>
Fixed both mistakes.
Then checked in.
Ilija
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug 1001558] New: GDB stub fails to send some registers.
@ 2012-04-11 20:42 bugzilla-daemon
0 siblings, 0 replies; 9+ messages in thread
From: bugzilla-daemon @ 2012-04-11 20:42 UTC (permalink / raw)
To: unassigned
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001558
Summary: GDB stub fails to send some registers.
Product: eCos
Version: CVS
Platform: All
OS/Version: Other
Status: NEW
Severity: enhancement
Priority: low
Component: GDB stubs
AssignedTo: unassigned@bugs.ecos.sourceware.org
ReportedBy: ilijak@siva.com.mk
CC: jifl@ecoscentric.com, ecos-bugs@ecos.sourceware.org,
john@dallaway.org.uk
Class: Advice Request
GDB stub sometimes fails to send all registers. I encountered this problem in a
course of providing support for Cortex-M4F VFP registers.
The cause is function in hal/common/current/src/thread-packets.c that sends
NUMREGS words that may be not correct if registers are not all same size and/or
some registers have been skipped.
Ilija
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2012-04-25 19:58 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-11 20:42 [Bug 1001558] New: GDB stub fails to send some registers bugzilla-daemon
2012-04-11 20:46 ` [Bug 1001558] " bugzilla-daemon
2012-04-11 20:52 ` bugzilla-daemon
2012-04-12 16:15 ` bugzilla-daemon
2012-04-12 16:17 ` bugzilla-daemon
2012-04-25 18:53 ` bugzilla-daemon
2012-04-25 19:16 ` bugzilla-daemon
2012-04-25 19:58 ` bugzilla-daemon
2012-04-11 20:42 [Bug 1001558] New: " bugzilla-daemon
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).