fails to handle C/H/S sanity check when opening Sun disklabel

[Colin Watson]

This patch was applied to parted in Dapper, but has now been reverted for Edgy in the cause of bringing parted back into sync:

parted (1.6.25.1-1ubuntu1) dapper; urgency=low

  [ Fabio M. Di Nitto ]
  * Disable CHS check in udeb:
    - add patch sparc-no-chs-in-udeb.dpatch.
    - build libparted udeb with -DNOCHSEXCEPTION=1

  libparted did grow a C/H/S sanity check in disk_sun.c. While this is a sane
  check to perform, parted_server doesn't understand the exception and fails
  because it gets a wrong size of the disk when writing a new label.
  Since we are in deep feature freeze, there is no time to make a proper fix
  into parted_server (that needs to be propagated all the way and back to
  the UI to handle it properly).
  For now let's workaround this issue disabling the feature ONLY for the udeb.

 -- Fabio M. Di Nitto <email address hidden> Tue, 28 Feb 2006 10:23:03 +0100

parted_server should handle this exception properly (at present it deactivates the exception handler while opening disklabels; simply deleting this code is probably not the right fix though). Fabio says he'll get hold of a partman log demonstrating this problem.

[Fabio Massimo Di Nitto (fabbione)]

Hi Colin,

i was not able to reproduce the problem using partman anymore. As we discussed i did build the libparted without the workaround, installed it as soon as possible in d-i, made sure with parted to be able to reproduce the problem and attempted an install.

You might want to notice that /proc/partitions does not have sdb3 that should be there are part of SUN layout. It shows up instead at the reboot.

DEBCONF_DEBUG was set at 5 as you requested.

If nothing show up here, we will have to try to look again with a clean edgy installer.

Fabio

[Fabio Massimo Di Nitto (fabbione)]

I was not able to reproduce the original problem with new version of parted. Closing it as released