Bug 53571

Summary: FLASH address handling inadequate
Product: [Retired] eCos Reporter: George Thomas <gthomas>
Component: RedBootAssignee: ecc-bugs-int
Status: CLOSED CURRENTRELEASE QA Contact: ecc-bugs-int
Severity: medium Docs Contact:
Priority: medium    
Version: CVS   
Target Milestone: ---   
Target Release: ---   
Hardware: i386   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2001-09-12 01:04:28 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description George Thomas 2001-09-12 01:02:52 UTC 
Description of Problem:

The current code in RedBoot for manipulating FLASH addresses falls
over in the case where the end of FLASH wraps the available address
size.


Version-Release number of selected component (if applicable):


How Reproducible:


Steps to Reproduce:
1. Imagine where FLASH base is 0xFF800000 and the size is 0x800000.
2. 
3. 

Actual Results:


Expected Results:


Additional Information:
Comment 1 George Thomas 2001-09-12 01:04:24 UTC 
Changing how RedBoot keeps track of the end of FLASH by letting it
be the "last" byte instead of the first byte after (and all of the
repercussions of this) would fix the problem.
Comment 2 George Thomas 2001-09-12 02:54:17 UTC 
All uses of FLASH addresses in RedBoot are now either (void *) or
CYG_ADDRESS which should avoid any addressing problems in the future.

Also, the 'end' address is kept internally as the last usable address
which avoids the wrap around problem.