Opened on 03/28/07 at 17:55:39

Closed on 03/28/07 at 17:58:52

#1234 closed defect (fixed)

Add __assert based upon printk

Reported by: Joel Sherrill Owned by: Joel Sherrill
Priority: normal Milestone: 4.7
Component: score Version: 4.7
Severity: normal Keywords:
Cc: Blocked By:
Blocking:

Description

The assert implementation in newlib writes to stderr using an iprintf() variant. This results in a lot of infrastructure being pulled into an executable which may or may not be desired or supported by a particular application's configuration. By using printk() in assert() we get an implementation of ~1200 bytes on the SPARC/ERC32 and a reduction of ~12K overall on the minimum executable.

All BSPs should provide (some) support for printk now.

Attachments (1)

pr1234.diff​ (1.5 KB) - added by Joel Sherrill on 03/28/07 at 17:57:29.
Patch for CVS head and 4.7

Download all attachments as: .zip

Change History (2)

Changed on 03/28/07 at 17:57:29 by Joel Sherrill

Attachment: pr1234.diff​ added

Patch for CVS head and 4.7

comment:1 Changed on 03/28/07 at 17:58:52 by Joel Sherrill

Resolution: → fixed
Status: new → closed

Applied patch to 4.7 branch and head.

Note: See TracTickets for help on using tickets.