Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hostboot build tools utilities are built as 32-bit #3

Closed
jk-ozlabs opened this issue Oct 26, 2014 · 2 comments
Closed

Hostboot build tools utilities are built as 32-bit #3

jk-ozlabs opened this issue Oct 26, 2014 · 2 comments

Comments

@jk-ozlabs
Copy link
Member

Currently, the hostboot utils are built with -m32. For example:

g++ -m32  -c  -g -O0   -I ../../../../src/include/usr -I ../../../../obj/genfiles  -o ../../../../obj/modules/errl/parser/errlparser.o  errlparser.C

This means that 64-bit build machines need an entire 32-bit toolchain. We've worked-around this by requiring 32-bit libc and a multilib-capable compiler in the README, but it'd be nice not to require that.

@jk-ozlabs
Copy link
Member Author

but it'd be nice not to require that.

in fact, gcc-multilib can't be installed alongside the Ubuntu powerpc64le cross compiler. This means that we require two separate build environments for hostboot vs general LE Linux binaries.

@causten
Copy link
Member

causten commented Nov 7, 2014

I hope this isn't off topic, but a goal of mine (and others) is to build the open-power image on the system it runs on... i.e. Palmetto can be built by the OS running on a Palmetto. I suspect there will be some other changes to the build env to make it work. Keep the end goal in mind when coming up with any solution

wghoffa pushed a commit to wghoffa/hostboot that referenced this issue Sep 28, 2017
Sync with powerpc64_petitboot_defconfig
wghoffa pushed a commit that referenced this issue Aug 26, 2019
- Allocate HTM/OCC memory location in order from largest -> smallest.
- Verify memory bar addresses aligned with memory allocation size.

Set #3: Address Joe's and Jenny's comments from set #2

Change-Id: I1000b3d6de371c129ae62ca53be36b632a1c545e
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82121
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Joseph J McGill <jmcgill@us.ibm.com>
Reviewed-by: Jenny Huynh <jhuynh@us.ibm.com>
Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82196
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants