Extend the Book-E support into 64-bits. Fix drivers that make 32-bit
assumptions.
Details
Details
- Reviewers
nwhitehorn marcel stevek - Group Reviewers
PowerPC - Commits
- rS315464: Introduce 64-bit PowerPC Book-E support
Continuously testing on a P5020 board (Amiga X5000/20)
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 7176 Build 7346: arc lint + arc unit
Event Timeline
Comment Actions
Remove driver updates, to be done separately. Add powerpc 32-bit binary support.
Tested with running /sbin/init (64-bit and 32-bit) from an embedded md.
| sys/powerpc/booke/locore.S | ||
|---|---|---|
| 548–552 | Shouldn't that just be the following (since you have the #defines for it): | |
| sys/powerpc/booke/locore.S | ||
|---|---|---|
| 548–552 | Yes it should. | |
| sys/powerpc/booke/locore.S | ||
|---|---|---|
| 945–955 | Given this and other usage elsewhere. Would it make more sense to create a macro for the use of .llong or .long based on __powerpc64__ defined or not? | |
| sys/powerpc/booke/pmap.c | ||
| 601 | Typo in comment. Should be... /* Free corresponding ptbl buf. */ | |
| 759 | Comment says "this will sleep", yet you have provisions for nosleep flag. | |
| 1344 | Why is this needed? It doesn't seem to be used in these additions. | |
| 1452 | Line seems a bit long, candidate for wrapping. | |
| 4054 | Shouldn't the wasted space part be using PRI0ptrX also? | |
| 4086 | It would be better to have #defines for the VA and PA in this tlb1_set_entry, instead of bare values. | |
| sys/powerpc/booke/locore.S | ||
|---|---|---|
| 945–955 | Good idea. | |
| sys/powerpc/booke/pmap.c | ||
| 601 | Oops, typo copy. | |
| 759 | This was copied from Juniper code. I forgot to clean up the comments when adapting it to head. | |
| 1344 | Oops, this was for debug purposes. Need to remove this and one other. | |
| 1452 | Probably a better candidate for macroizing. | |
| 4054 | Yes it should. Good catch. | |
| 4086 | This was also debug. It'll be committed separately with EARLY_PRINTF changes. | |