Bug#512740: Sparc disk labels broken on LDOM and Parallel installs
On Tue, Feb 09, 2010 at 06:54:39PM +0100, Frans Pop wrote: Below are instructions for building a custom D-I image that includes a patched libparted and partman. It's not that hard. I've built a mini iso image according to these instructions. It is available from http://www.wooyd.org/debian/512740, along with intermediate packages. Minor issues with the build and the resulting image: 1. dh-di needed for build of partman-base is not pulled in as a result of 'apt-get build-dep debian-installer', probably should be? 2. Firmware loading is broken in the mini iso image, error is 'mountmedia not found'. 3. The installation is still affected by the kernel issue where it misreports the architecture and SILO installs the incorrect second-stage bootloader as a result. I've ran a couple of installs with this image, zero-filling beginning of the disk with zeros during one of them, and did not see any regressions. Mike, can you try downloading the mini.iso and testing that it fixes the issue for you? If your machine needs firmware loaded for the disks to be detected, you'll need to do it manually. Also, you will need to drop into installer shell at the end of the installation, chroot into /target, mount /proc and run 'silo -u -f' to unbreak the booloader. Let me know if you would like more specific instructions. Thanks. -- Jurij Smakov ju...@wooyd.org Key: http://www.wooyd.org/pgpkey/ KeyID: C99E03CC -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/20100214155149.ga5...@droopy.oc.cox.net
Bug#512740: Sparc disk labels broken on LDOM and Parallel installs
On Sunday 14 February 2010, Jurij Smakov wrote: 1. dh-di needed for build of partman-base is not pulled in as a result of 'apt-get build-dep debian-installer', probably should be? No: base-installer is a separate source package and thus has its own build deps. I just forgot to include a step to install those (but luckily DDs are smart enough to figure such minor omissions out by themselves ;-) 2. Firmware loading is broken in the mini iso image, error is 'mountmedia not found'. Added now for future builds. Please 'svn up'. 3. The installation is still affected by the kernel issue where it misreports the architecture and SILO installs the incorrect second-stage bootloader as a result. Correct. The installer still uses 2.6.30 and the bug is related to the kernel version that's running when silo is installed. It'll get solved when we switch to 2.6.32, which should happen soon after the (delayed) D-I alpha1 release gets out. -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/201002141742.20722.elen...@planet.nl
Bug#512740: Sparc disk labels broken on LDOM and Parallel installs
Hello, On Tue, Feb 9, 2010 at 5:52 AM, Stefano Zacchiroli z...@pps.jussieu.fr wrote: On Mon, Feb 08, 2010 at 10:04:05PM +, Jurij Smakov wrote: My main concern is whether it affects installer in any adverse way, so if you could arrange for an installer image to be built with the modified parted, we could do some testing. I don't have access to a machine which supports LDOMs, but I could at least confirm that it does not introduce any regressions. Personally, I'm not very proficient in d-i (read: not at all), so I'm getting -boot into the loop forwarding your request, for help. Bottom line for d-i, to test the patch proposed to fix #512740, we need a d-i sparc image to check for regression on sparc. The patch is attached for your convenience, see bug log for full context. We need this patch against GNU Parted 2.1 (that can be found in experimental) since this is the current codebase. Besides, it would be good if we push it upstream, we can work on that after this has been ported to 2.1 version. Can someone from Sparc take a look and port it? -- Otavio Salvador O.S. Systems E-mail: ota...@ossystems.com.br http://www.ossystems.com.br Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#512740: Sparc disk labels broken on LDOM and Parallel installs
On Tuesday 09 February 2010, Stefano Zacchiroli wrote:
On Mon, Feb 08, 2010 at 10:04:05PM +, Jurij Smakov wrote:
My main concern is whether it affects installer in any adverse way, so
if you could arrange for an installer image to be built with the
modified parted, we could do some testing. I don't have access to a
machine which supports LDOMs, but I could at least confirm that it
does not introduce any regressions.
Personally, I'm not very proficient in d-i (read: not at all), so I'm
getting -boot into the loop forwarding your request, for help. Bottom
line for d-i, to test the patch proposed to fix #512740, we need a d-i
sparc image to check for regression on sparc. The patch is attached for
your convenience, see bug log for full context.
I've read through the patch, but must admit I don't understand it. What
seems to be missing is *how* exactly it fixes the problem.
And somehow I doubt this patch stands on its own. I would think that it is
a requirement for patches in other packages (such as partman). If that is
true, then testing the patch in isolation is not much use (as applying it
on its own would not fix the reported issue anyway).
Also, I question the definition of (unused?) tags for usr, var, home etc.
What's the purpose of that? How's that supposed to be used? Does it mean
you're not allowed to put e.g. /srv on a separate partition as there's no
tag for it?
More porter input is definitely needed here.
Below are instructions for building a custom D-I image that includes a
patched libparted and partman. It's not that hard.
Cheers,
FJP
0) preparation
- check out D-I SVN
- install build deps for debian-installer
1) build patched parted
- build parted with the patch (preferably with increased package version)
- install the lib and -dev packages
- suggestion: install 'parted' and test that before continuing
2) build partman against the patched libparted
- cd to packages/partman/partman-base
- increase package version ('dch')
- debuild
3) build a D-I image
- cd to installer/build
- cp ../../partman/*.udeb ./localudebs
- cp all (lib)parted udebs (from the first step) to ./localudebs
- echo partman-base ./pkg-lists/local
# This will include the custom udebs in the D-I initrd and thus ensure
# they'll be used instead of the versions in the archive.
- make reallyclean; make build_netboot
# Instead of build_netboot you can also use build_miniiso.
4) test
- boot the installer from the image you'll find under ./dest
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#512740: Sparc disk labels broken on LDOM and Parallel installs
On Sat, Oct 03, 2009 at 11:53:32AM +0200, Luk Claes wrote:
What's the status of this bug, can the patch be applied to fix it?
Ditto, and getting debian-sparc in the loop.
Executive summary for the sparc porters, there's a patch to fix disk
label b0rkage on sparc with parted. The patch has been applied in Ubuntu
for a while now.
Given that the parted maintainers are unresponsive on this, can please
the porters comment on the patch?
Many thanks in advance,
Cheers.
--
Stefano Zacchiroli -o- PhD in Computer Science \ PostDoc @ Univ. Paris 7
z...@{upsilon.cc,pps.jussieu.fr,debian.org} -- http://upsilon.cc/zack/
Dietro un grande uomo c'è ..| . |. Et ne m'en veux pas si je te tutoie
sempre uno zaino ...| ..: | Je dis tu à tous ceux que j'aime
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#512740: Sparc disk labels broken on LDOM and Parallel installs
On Mon, Feb 08, 2010 at 04:27:14PM +0100, Stefano Zacchiroli wrote: On Sat, Oct 03, 2009 at 11:53:32AM +0200, Luk Claes wrote: What's the status of this bug, can the patch be applied to fix it? Ditto, and getting debian-sparc in the loop. Executive summary for the sparc porters, there's a patch to fix disk label b0rkage on sparc with parted. The patch has been applied in Ubuntu for a while now. Given that the parted maintainers are unresponsive on this, can please the porters comment on the patch? My main concern is whether it affects installer in any adverse way, so if you could arrange for an installer image to be built with the modified parted, we could do some testing. I don't have access to a machine which supports LDOMs, but I could at least confirm that it does not introduce any regressions. Best regards, -- Jurij Smakov ju...@wooyd.org Key: http://www.wooyd.org/pgpkey/ KeyID: C99E03CC -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#512740: Sparc disk labels broken on LDOM and Parallel installs
On Mon, Feb 08, 2010 at 10:04:05PM +, Jurij Smakov wrote:
My main concern is whether it affects installer in any adverse way, so
if you could arrange for an installer image to be built with the
modified parted, we could do some testing. I don't have access to a
machine which supports LDOMs, but I could at least confirm that it
does not introduce any regressions.
Personally, I'm not very proficient in d-i (read: not at all), so I'm
getting -boot into the loop forwarding your request, for help. Bottom
line for d-i, to test the patch proposed to fix #512740, we need a d-i
sparc image to check for regression on sparc. The patch is attached for
your convenience, see bug log for full context.
Thanks for your feedback!
Cheers.
--
Stefano Zacchiroli -o- PhD in Computer Science \ PostDoc @ Univ. Paris 7
z...@{upsilon.cc,pps.jussieu.fr,debian.org} -- http://upsilon.cc/zack/
Dietro un grande uomo c'è ..| . |. Et ne m'en veux pas si je te tutoie
sempre uno zaino ...| ..: | Je dis tu à tous ceux que j'aime
#! /bin/sh /usr/share/dpatch/dpatch-run
## sparc-new-label.dpatch by David S. Miller and
## Fabio M. Di Nitto fabbi...@ubuntu.com
##
## All lines beginning with `## DP:' are a description of the patch.
## DP: Fix sparc disk label generation. This is required for LDOM and
## DP: parallel installations with Solaris 10.
@DPATCH@
diff -urNad parted-1.8.8.git.2008.03.24~/libparted/labels/sun.c
parted-1.8.8.git.2008.03.24/libparted/labels/sun.c
--- parted-1.8.8.git.2008.03.24~/libparted/labels/sun.c 2008-06-24
12:52:11.0 +0100
+++ parted-1.8.8.git.2008.03.24/libparted/labels/sun.c 2008-06-24
12:56:06.0 +0100
@@ -38,12 +38,30 @@
#define SUN_DISK_MAGIC 0xDABE /* Disk magic number */
#define SUN_DISK_MAXPARTITIONS 8
-#define WHOLE_DISK_ID 0x05
+#define SUN_TAG_UNASSIGNED 0x00/* Unassigned partition */
+#define SUN_TAG_BOOT 0x01/* Boot partition */
+#define SUN_TAG_ROOT 0x02/* Root filesystem */
+#define SUN_TAG_SWAP 0x03/* Swap partition */
+#define SUN_TAG_USR0x04/* /usr filesystem */
+#define SUN_TAG_BACKUP 0x05/* Full-disk slice */
+#define SUN_TAG_STAND 0x06/* Stand partition */
+#define SUN_TAG_VAR0x07/* /var filesystem */
+#define SUN_TAG_HOME 0x08/* /home filesystem */
+#define SUN_TAG_ALTSCTR0x09/* Alt sector partition */
+#define SUN_TAG_CACHE 0x0a/* Cachefs partition*/
+#define SUN_TAG_RESERVED 0x0b/* SMI reserved data*/
+#define SUN_TAG_LINUX_SWAP 0x82/* Linux SWAP */
+#define SUN_TAG_LINUX_NATIVE 0x83/* Linux filesystem */
+#define SUN_TAG_LINUX_LVM 0x8e/* Linux LVM*/
+#define SUN_TAG_LINUX_RAID 0xfd/* LInux RAID */
+
+#define SUN_FLAG_UNMNT 0x01/* Unmountable partition*/
+#define SUN_FLAG_RONLY 0x10/* Read only*/
+
#define WHOLE_DISK_PART2 /* as in 0, 1, 2 (3rd
partition) */
-#define LINUX_SWAP_ID 0x82
typedef struct _SunRawPartition SunRawPartition;
-typedef struct _SunPartitionInfoSunPartitionInfo;
+typedef struct _SunPartInfo SunPartInfo;
typedef struct _SunRawLabel SunRawLabel;
typedef struct _SunPartitionDataSunPartitionData;
typedef struct _SunDiskData SunDiskData;
@@ -53,18 +71,31 @@
u_int32_t num_sectors;/* ...and it's length */
};
-struct __attribute__ ((packed)) _SunPartitionInfo {
- u_int8_tspare1;
- u_int8_tid; /* Partition type */
- u_int8_tspare2;
- u_int8_tflags; /* Partition flags */
+struct __attribute__ ((packed)) _SunPartInfo {
+ u_int16_t tag;/* Tagged type */
+ u_int16_t flags; /* Partition flags */
};
+#define SUN_LABEL_ID_SIZE 128
+#define SUN_VOLUME_ID_SIZE 8
+
+#define SUN_LABEL_VERSION 0x0001
+#define SUN_LABEL_SANE 0x600ddeee
+
struct __attribute__ ((packed)) _SunRawLabel {
- charinfo[128]; /* Informative text string */
- u_int8_tspare0[14];
- SunPartitionInfo infos[SUN_DISK_MAXPARTITIONS];
- u_int8_tspare1[246];/* Boot information etc. */
+ charlabel_id[SUN_LABEL_ID_SIZE];/* Informative text
string */
+ u_int32_t version;
+ charvolume_id[SUN_VOLUME_ID_SIZE];
+ u_int16_t num_part;
+ SunPartInfo infos[SUN_DISK_MAXPARTITIONS];
+ u_int16_t pad1;
+ u_int32_t bootinfo[3];
+ u_int32_t sanity;
+ u_int32_t resv[10];
+ u_int32_t part_timestamps[SUN_DISK_MAXPARTITIONS];
+ u_int32_t write_reinstruct;
+ u_int32_t read_reinstruct;
+ u_int8_t
Bug#512740: Sparc disk labels broken on LDOM and Parallel installs
Hi What's the status of this bug, can the patch be applied to fix it? Cheers Luk -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#512740: Sparc disk labels broken on LDOM and Parallel installs
Package: parted Architecture: i386 Version: 1.8.8.git.2008.03.24-11.1 Severity: grave Tags: patch I was exposed to this bug while trying to do a second install of Debian into an LDOM, but its not the only way to trigger it... Parted currently has a bug where if you try to install to more than one disk at once (2 parallel installs), or on LDOM you try to install more than one LDOM which has a physical disk as a backing device, parted will trash the disk label, rendering the disk unusable until you go and relabel it with the 'format' command in solaris. Ubuntu got around this by applying a patch (written by David S. Miller and Fabio M. Di Nitto who happen to do a lot of work on the sparc Linux port). This applies cleanly against the current Lenny source package for parted (which means that the udeb can be rebuilt and installs will be fixed. From the patch description: ## sparc-new-label.dpatch by David S. Miller and ## Fabio M. Di Nitto fabbi...@ubuntu.com ## ## All lines beginning with `## DP:' are a description of the patch. ## DP: Fix sparc disk label generation. This is required for LDOM and ## DP: parallel installations with Solaris 10. I've attached this as it applies cleanly so there's little maintainer work (hopefully). I've set the severity to grave as it causes data loss on the disk you're installing on (if youre already using some partitions on that disk for something else). It also blocks install if its the second disk you've installed Debian on (feel free to correct me!). Thanks, Mike. -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
