Kenshi Muto wrote: > Takeshi Yaegashi, who is my friend, succeeded to install Debian > on SONY PlayStation 3 (PS3). PS3 is PPC64 architecture and > some Linux distributions and NetBSD are already ported for PS3. Very excellent, I've added a link to this to the Debian wiki too. > SVN diff against current d-i trunk (still under development, > he is carefully changing d-i to keep small): > http://www.keshi.org/moin/moin.cgi/PS3/Debian/Installer/Porting > > Although I haven't PS3, it's very interesting that d-i merges > his PS3 PPC specific patch for post-Etch. I agree, and maybe some more of us will have ps3's by then. :-) If Takeshi is interested in commit access to the d-i subversion repository, I feel that this easily justifies it. Then he could keep his patches in a branch and work on merging things in as appropriate. I've taken an initial look at the patch, and I feel the following hunks could be applied as-is without problems to d-i trunk, although we might prefer to wait until there is a post-etch branch to avoid destabilisation. Not that I feel there's much risk, most of these changes are minor. +++ installer/build/boot/powerpc/kboot.conf +++ installer/build/config/powerpc/ps3pf/monolithic.cfg +++ installer/build/config/powerpc.cfg (would need a minor modification to have ps3pf commented out for now) +++ packages/hw-detect/hw-detect.sh +++ packages/rootskel/src/lib/debian-installer-startup.d/S02ps3pf-linux-powerpc +++ packages/rootskel/src/lib/debian-installer-startup.d/Makefile +++ packages/libdebian-installer/src/system/subarch-powerpc-linux.c +++ packages/arch/powerpc/kboot-installer/* (can't be uploaded yet, and probably not yet a candidate for translation, but fine to check it into our svn) +++ packages/partman/partman-base/definitions.sh (second hunk only) +++ packages/base-installer/kernel/powerpc.sh (it would be good to also add a test case with a copy of the ps3's cpuinfo file) I don't understand the following changes: --- installer/build/pkg-lists/monolithic/powerpc.cfg (revision 43755) +++ installer/build/pkg-lists/monolithic/powerpc.cfg (working copy) @@ -8,5 +8,5 @@ dhcp3-client-udeb - eject-udeb # Used by yaboot-installer -parted-udeb -os-prober +parted-udeb ? +os-prober ? As far as I can see, these should be available for the ps3 image too, so no need to mark them as optional? +++ packages/partman/partman-base/definitions.sh (working copy) @@ -771,6 +771,8 @@ enable_swap () { local swaps dev num id size type fs path name method local startdir="$(pwd)" + # XXX + return 0 # do swapon only when we will be able to swapoff afterwards [ -f /proc/swaps ] || return 0 swaps='' If there's no swap on this machine, there are probably better ways to handle that. :-) --- packages/debian-installer-utils/apt-install (revision 43755) +++ packages/debian-installer-utils/apt-install (working copy) @@ -30,7 +30,7 @@ export DEBIAN_FRONTEND ERRCODE=0 -log-output -t apt-install chroot /target apt-get -y --no-remove install $@ < /dev/null || ERRCODE=$? +log-output -t apt-install chroot /target apt-get -y --force-yes --no-remove install $@ < /dev/null || ERRCODE=$? chroot_cleanup if [ "$ERRCODE" != 0 ]; then This change seems odd, especially since the patch does not use apt-install. -- see shy jo
Attachment:
signature.asc
Description: Digital signature