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

RFC: arm64 support [ci-skip] [not for merge] #1654

Closed
wants to merge 29 commits into from
Closed

Conversation

@mor1
Copy link
Contributor

mor1 commented Apr 18, 2017

This PR is not for merge but is the starting point for ARM64 support.
It has now successfully built a kernel and initrd that have booted a packet.net ARM64 box.

@mor1 mor1 force-pushed the mor1:project-arm64 branch from 483f17e to a297d41 Apr 20, 2017
@mor1 mor1 force-pushed the mor1:project-arm64 branch from 28100a3 to 635ec8f Apr 24, 2017
@GordonTheTurtle GordonTheTurtle removed the dco/no label Apr 24, 2017
@mor1 mor1 force-pushed the mor1:project-arm64 branch from 635ec8f to 6a2b79e Apr 24, 2017
@mor1 mor1 force-pushed the mor1:project-arm64 branch from 6a2b79e to 7b335d7 Apr 24, 2017
@GordonTheTurtle GordonTheTurtle removed the dco/no label Apr 24, 2017
@mor1 mor1 force-pushed the mor1:project-arm64 branch 2 times, most recently from d2837a0 to f825cc0 Apr 24, 2017
@mor1 mor1 force-pushed the mor1:project-arm64 branch from c9df894 to 571ea11 Apr 24, 2017
@GordonTheTurtle GordonTheTurtle removed the dco/no label Apr 24, 2017
@AkihiroSuda
Copy link
Contributor

AkihiroSuda commented Apr 25, 2017

Can we deduplicate kernel/Dockerfile,Makefile,kernel_config across all the platforms?

i.e.

  • kernel/Dockerfile: universal Dockerfile, accepts specifying ARCH via build-arg
  • Makefile: ditto via make ARCH=...
  • kernel_config: common parts across all the archs
  • kernel_config.{x86_64,arm64,...}: architecture-specific part. Concatinated to kernel_config in the build container
@rn
Copy link
Member

rn commented Apr 25, 2017

@AkihiroSuda hopefully today I'll tidy up the kernel build a little and that might make it easier to dedup

@mor1 mor1 force-pushed the mor1:project-arm64 branch from a0fb12d to 4c1824d Apr 25, 2017
@mor1 mor1 force-pushed the mor1:project-arm64 branch from 4c1824d to b009769 Apr 25, 2017
@GordonTheTurtle GordonTheTurtle removed the dco/no label Apr 25, 2017
Makefile Outdated

MOBY?=bin/moby
LINUXKIT?=bin/linuxkit
GOOS=$(shell uname -s | tr '[:upper:]' '[:lower:]')
GOARCH=amd64
GOARCH=arm64

This comment has been minimized.

Copy link
@justincormack

justincormack Apr 26, 2017

Member

you should not be setting this unconditionally, you should use uname to find it

mor1 added 8 commits Mar 27, 2017
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
mor1 added 4 commits Apr 24, 2017
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
@mor1 mor1 force-pushed the mor1:project-arm64 branch from b009769 to 343ce0f Apr 29, 2017
@GordonTheTurtle
Copy link
Collaborator

GordonTheTurtle commented Apr 29, 2017

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "project-arm64" git@github.com:mor1/linuxkit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354472624
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Signed-off-by: Richard Mortier <mort@cantab.net>
@mor1 mor1 force-pushed the mor1:project-arm64 branch from 575c130 to e100e24 Apr 29, 2017
@GordonTheTurtle GordonTheTurtle removed the dco/no label Apr 29, 2017
mor1 added 11 commits May 2, 2017
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
mor1 added 3 commits May 8, 2017
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
Signed-off-by: Richard Mortier <mort@cantab.net>
@mor1 mor1 mentioned this pull request May 17, 2017
3 of 5 tasks complete
Signed-off-by: Richard Mortier <mort@cantab.net>
@zlim
Copy link
Contributor

zlim commented Jun 26, 2017

cc @arm64b

@justincormack
Copy link
Member

justincormack commented Jul 12, 2017

Going to close this as parts of arm64 are now being merged in and this does not apply.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.