[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [FYI] debian-installer for SONY PS3

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
+++ 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 -
 # Used by yaboot-installer
+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

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 @@

-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=$?

 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

Reply to: