#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:


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 Mar 28, 2007 at 5:57:29 PM.
Patch for CVS head and 4.7

Download all attachments as: .zip

Change History (2)

Changed on Mar 28, 2007 at 5:57:29 PM by Joel Sherrill

Attachment: pr1234.diff added

Patch for CVS head and 4.7

comment:1 Changed on Mar 28, 2007 at 5:58:52 PM by Joel Sherrill

Resolution: fixed
Status: newclosed

Applied patch to 4.7 branch and head.

Note: See TracTickets for help on using tickets.