Bug#988776: Bug#983357: Netinst crashes xen domU when loading kernel

2021-08-25 Thread Phillip Susi


Chuck Zmudzinski  writes:
> If it doesn't work, I am also willing to try approach a by patching
> the Linux kernel xen-kbdfront driver by removing the for loops that
> advertise those 654 keys. I tend to agree with Philip that this is
> totally unnecessary, but I suppose I could be wrong about that.
> I read the discussion Philip had with the Xen developers and they
> seemed to want to keep the Xen keyboard driver as it is.

That was the first thing I tried and the libinput maintainer pointed out
that if you don't advertise the keys, you can't use the keys.  In other
words, somebody presses that key on their keyboard and the domU won't
recognize it.



Bug#988776: Bug#983357: Netinst crashes xen domU when loading kernel

2021-08-24 Thread Phillip Susi


Ben Hutchings  writes:

> I think a proper fix would be one of:
>
> a. If the Xen virtual keyboard driver is advertising capabilities it
>doesn't have, stop it doing that.
> b. Change the implementation of modalias attributes to allow longer
>values.
>
> It's not clear to me whether the Xen driver is advertising correctly or
> not.  If it is, then the solution should be b, but that may be too
> disruptive a change to the kernel.  So a reasonable workaround might
> be:
>
> c. Change the input subsystem to limit the length of the
>capabilities part of the modalias.

The problem with a) is that the Xen keyboard is not a physical keyboard
and so it has no way of knowing what keys it actually has.  It is a fake
input device designed to pass through whatever input the Xen hypervisor
sends down.  As such, any key could come in.  If it doesn't advertise
that it has all of these keys, then they would not be accepted by
libinput when the hypervisor sends them down.

This seems to be the heart of the problem: libinput was designed
assuming that all keyboards can and must report what keys are actually
present, and then libinput tries to cram that information into the
modalias rather than some other sysfs attribute as it should ( or not at
all... I still don't see how this information is actually supposed to be
useful to userspace ).

As for b), the problem isn't with the modalias attribute itself, but
when the kernel tries to copy it into the environment block for the udev
callout.  The environment block is only a single page, and so limited to
4 KB.  And that's for everything else that goes into the environment,
not just the modalias.



Bug#988776: Bug#983357: Netinst crashes xen domU when loading kernel

2021-05-25 Thread Phillip Susi


Michael Biebl writes:

> So this is a change in behaviour in the kernel?

Yes, this commit fixed the kernel to report the error instead of
silently failing:

commit df44b479654f62b478c18ee4d8bc4e9f897a9844
Author: Peter Rajnoha 
Date:   Wed Dec 5 12:27:44 2018 +0100

kobject: return error code if writing /sys/.../uevent fails

Propagate error code back to userspace if writing the /sys/.../uevent
file fails. Before, the write operation always returned with success,
even if we failed to recognize the input string or if we failed to
generate the uevent itself.

With the error codes properly propagated back to userspace, we are
able to react in userspace accordingly by not assuming and awaiting
a uevent that is not delivered.

Signed-off-by: Peter Rajnoha 
Signed-off-by: Greg Kroah-Hartman 

> What happens if you boot the installed system? Does udevadm trigger fail 
> there as well?

Yes, it does; that is how I was able to track down the problem.

> I feel a bit uneasy changing the udev start script this late in the 
> release cycle (especially when it appears like covering up an issue 
> someplace else).
>
> I'll let Marco make the judgement on this though, as he has the most 
> experience with those udev udeb start scripts as the original author.

So far I have been removing the -e from the shbang line in the
start-udev script and remastering the iso so I can get it to boot.  It
would probably be a better idea to just add a || true to the udevadm
trigger call.  I feel fairly certain that no matter what the cause of
the coldplug failure, the user is going to be better off ignoring it and
trying to proceed than a kernel panic.



Bug#987688: Please add more filesystem drivers to hd-media

2021-04-27 Thread Phillip Susi
Package: debian-installer
Version: 20210415

I just tried using the hd-media build to boot from an existing hard disk
and install without using removable media and was not able to do so
because I used btrfs on the hard disk, and most filesystems are packaged
as separate udebs rather than being built into the initrd.  I think it
was only iso9660, fat, and ext4 that were in there.  Please consider
moving btrfs and probably a few other filesystem modules into the
hd-media initrd so that it can be used outside of ext4 and fat.



Bug#983357: #983357: Netinst crashes xen domU when loading kernel

2021-04-22 Thread Phillip Susi
reassign 983357 linux
severity grave
thanks

I rebuilt the iso using the version of isolinux from stable and it still
crashed the domU.  When I rebuilt it using the vmlinux and initrd.gz
from the stable iso, it successfully boots, so it appears to be caused
by the kernel.

Interestingly, there appears to be a different kernel build just for use
under xen in install.amd/xen and using that one also works.  Maybe we
need a menu option in isolinux to load that kernel instead?



Bug#983357: Netinst crashes xen domU when loading kernel

2021-02-22 Thread Phillip Susi
Package: debian-installer
Version: 20201202

Every bullseye netinst image I have tried to boot in a xen domU has
crashed and rebooted the domU after choosing any entry from the boot
menu.  I thought there might have been something wrong with my xen
server, which was running Ubuntu 18.04, but I rebuilt it the other day
using bullseye as the dom0, and I still can't install bullseye in a
domU.



Bug#822367: partman should check and warn if missing bios_grub partition

2016-04-23 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA512

Package: partman-partitioning
Version: 111

When installing to a GPT partitioned disk in bios mode, a bios_grub
partition is required to install grub.  Partman should warn the user
of this if they do not have one.
-BEGIN PGP SIGNATURE-
Version: GnuPG v2

iQEcBAEBCgAGBQJXG+AXAAoJEBB5UWFcu6UW2v8H/i/cK7EDTjQzijzf20V+wvLJ
Z2bnCejkzvLfd/58lZ6ECHj/Vlp4Ab3ooyzrIsOr+fmLXeaPdD/hiE8/7NrHnoF8
K0jiXvfNC5GE6CLbhGb5LteArXPLIabAZboMklkPwlTobI55Xp/A3b4wzIQvu8JW
Ar4LMUOhPC6AF9W9YuBYB8AcfI02O0cQf5zd75qNtNJg9NZ3PuZ4wVwqGhYHH3zl
PZJOxCWWId/0gjhSf++3xOoJVLjYJVUUAc9Iry/pm0Usj57sofKx8ZhWM5TdCaGP
eD+9oWshHn+MfIWMRRNHstWEriQAvJoBgeXr92DlYcc1856F9TqiqMmaYNIulF4=
=wrti
-END PGP SIGNATURE-



Bug#778617: grub-install often breaks in UEFI mode

2015-02-18 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 2/17/2015 5:51 PM, Steve McIntyre wrote:
 On Tue, Feb 17, 2015 at 09:32:56AM -0500, Phillip Susi wrote:
 Package: grub-installer Version: 1.110
 
 I'm forwarding this patch from Ubuntu:
 
 https://launchpad.net/bugs/1303790
 
 On EFI, the boot disk argument given to grub-install is a dummy
 and is often the wrong disk or a non existent disk.  Grub is
 still installed to the correct place since it goes through the
 filesystem to /boot/EFI, but the function make_active_partition
 tries to look at the partition table on the dummy disk and throws
 an error.  Hence, I wrote the attached patch to disable the
 function on EFI.
 
 Hi Phillip,
 
 The logic looks ok, I guess, but I'd check more explicitly for 
 grub-efi-* instead of !grub-pc. There's several other
 architectures that may use grub, and I'm not sure if/how they may
 need special handling for active partitions...

Sounds good to me if you want to switch it when you apply the patch.


-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)

iQEcBAEBAgAGBQJU5KcwAAoJENRVrw2cjl5R48YH/isp8MR/2Ps4F0v7fzpxpgkb
Oe2e/s+ziB14mhKQ1dNHb8KcIeFfg60cCTSNw8kmb05Zk+v+GsaAHpKX0Dk62i5T
gy+9HXPlyUqZTzCe24mQnSAgSbrgm8B9OD0VRgr6fEzAbW2/+co9wI4l1NQHKifK
fPls70SKQBZyqZDpr4F0uBEBDfB16kOTJr8P+a9MEhdzkwxA0CyuQObwaIC5V0xd
DpQqmiE+g5v9zpJgBZDVV/c1t+mBAzSTnORg+/1J8qXf48yHOtA2Or0nJb/VxvBM
U6B32fsy7g+4hnZVq1U3Q+JlrNIB4BdyyEoatg68i2TnjS5b/aDXCVPBR0TpsoY=
=igUq
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/54e4a730.2060...@ubuntu.com



Bug#778617: grub-install often breaks in UEFI mode

2015-02-17 Thread Phillip Susi
Package: grub-installer
Version: 1.110

I'm forwarding this patch from Ubuntu:

https://launchpad.net/bugs/1303790

On EFI, the boot disk argument given to grub-install is a dummy and is
often the wrong disk or a non existent disk.  Grub is still installed to
the correct place since it goes through the filesystem to /boot/EFI, but
the function make_active_partition tries to look at the partition table
on the dummy disk and throws an error.  Hence, I wrote the attached
patch to disable the function on EFI.


revno: 121
committer: Phillip Susi ps...@ubuntu.com
branch nick: grub-installer
timestamp: Tue 2015-02-17 09:15:44 -0500
message:
  Don't try to mark a partition as active, except on grub-pc.
  This was causing failures for grub-efi (LP: #1303790).
diff:
=== modified file 'debian/changelog'
--- debian/changelog2014-10-06 18:29:15 +
+++ debian/changelog2015-02-17 14:15:44 +
@@ -1,3 +1,10 @@
+grub-installer (1.78ubuntu24) vivid; urgency=medium
+
+  * Don't try to mark a partition as active, except on grub-pc.
+This was causing failures for grub-efi (LP: #1303790).
+
+ -- Phillip Susi ps...@ubuntu.com  Tue, 27 Jan 2015 15:51:09 -0500
+
 grub-installer (1.78ubuntu23) utopic; urgency=medium
 
   * On ppc64el, wipe the PReP partition before installing grub-ieee1275,

=== modified file 'grub-installer'
--- grub-installer  2014-10-06 18:29:15 +
+++ grub-installer  2015-02-17 14:15:44 +
@@ -1006,6 +1006,12 @@
 }
 
 make_active_partition () {
+   if [ $grub_package != grub-pc ]; then
+   # only do this for grub-pc since on EFI $bootdev is a dummy argument
+   # and looking for a partition table on the wrong or non existing disk
+   # crashes the installer LP:#1303790
+   return
+   fi
bootdisk=
bootpart=
case $bootdev in


signature.asc
Description: OpenPGP digital signature


Bug#777647: partman-efi always complains when installing from usb

2015-02-17 Thread Phillip Susi
On 2/17/2015 1:37 AM, Steve McIntyre wrote:
 Any futher clues on this at all? I have next to no knowledge about how
 the Ubuntu installer code uses the d-i packages, which makes it
 difficult for me to comment much more.

I emailed a log with my attempt at analyzing it to Colin Watson the
other day and am still waiting to hear back from him.  I think you can
close this bug now as I'm fairly certain it is an Ubiquity issue.  I
looked at some logs from older releases and indeed they all seem to run
all of the init.d scripts first, and only later process the visual.d and
other scripts, so somehow ubiquity must be running things wrongly and
getting the visual.d scripts to run before the init.d ones and that is
the cause of the problem.

I do still wonder though, why the script doesn't trip up when installing
from a USB stick.  Is there some logic somewhere I don't see that causes
all of partman to ignore/forget about the disk you are installing from,
so that the efi script doesn't see that and count it as a non ESP?


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/54e380b7@gmail.com



Bug#777647: partman-efi always complains when installing from usb

2015-02-11 Thread Phillip Susi
On 2/11/2015 4:38 AM, Steve McIntyre wrote:
 Hmmm, and just testing with the latest Debian amd64 daily netinst all
 works flawlessly here still, using the same empty disk in a KVM setup
 as your bug reporter.

Strange.. I'll have to poke around with that.

 This suggests the problem is Ubuntu-specific, or there's an issue with
 other software. You allude to potential changes in libparted; how do
 the versions in Debian and Ubuntu compare here?

They are in sync.  The change in particular that I think may be related
is that in parted3, it now keeps the partition table cached between
commands rather than re-reading it every time.  This has the affect that
when you create a partition and tell parted it will hold an ext2
filesystem, and then print the table, it now reports that it contains an
ext2 filesystem since it is no longer re-reading the disk and finding
the partition to be empty.

Now that I think about it though, this change was in parted proper and
so should not affect other libparted clients like parted_server, so that
seems to have been a red herring.

The thing I saw that struck me as similar to that parted change was that
when I put a set +x in the partman-efi script, it appeared to have
identified an ext2 filesystem on the disk that partman was formatting
with the usual default ext4 root and swap partitions so I kind of
assumed it simply had not gotten around to running mke2fs yet but had
asked libparted to make an ext2 partition and then partman-efi
identified it as a non efi system partition.

This script really should be running before the disk has been modified
in any way though right?  So it should still just see a blank disk or a
disk with no partitions and not count it as a non ESP.  Hrm... I'll do
some more debugging tonight.


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/54db85a4.9050...@gmail.com



Bug#777647: partman-efi always complains when installing from usb

2015-02-11 Thread Phillip Susi
On 02/11/2015 11:47 AM, Steve McIntyre wrote:
 Quite, that's exactly how it's meant to work and it's what I've seen
 in my development and testing. Silly question - is ubiquity trying to
 run some of the d-i bits in parallel, or something?

That's what I was wondering.  I'm looking at /var/log/partman now and it
appears that visual.d/35name runs and then I see /bin/perform_recipie
issue a NEW_PARTITION command to make the ext2 partition ( which will
later be formatted as fat32 for the esp ), and then the ext4 partition
for the root and then swap.  Later init.d/50efi runs and sees the
partition that the earlier script created even though it has not
actually been committed to disk yet ( i.e. blkid still sees a blank disk ).

Did we change the ordering in ubuntu or something so that the problem is
that visual.d has priority 35 but init.d/efi has priority 50 when they
should run the other order?  I would think that all of the init.d
scripts would be run before any visual.d scripts though, and the
priorities just order them within their group.  If that's not the case
then I guess they simply have the wrong priority.


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/54db9afa.1060...@gmail.com



Bug#777647: partman-efi always complains when installing from usb

2015-02-10 Thread Phillip Susi
Package: partman-efi
Version: 59

partman-efi uses flawed logic that trips up when installing from usb.
init.d/efi scans the system and counts the number of efi system
partitions, and the number of non efi system partitions.  If it does not
find an EFI system partition, but does find at least one non ESP, then
it throws the non_efi_system warning/question.  The problem is that when
installing to a disk that does not already have an efi system partition,
the script *always* detects a non EFI system partition, and throws the
warning.  This may be connected to a recent change in libparted.
Looking at the partman logs and the output after adding a set -x to the
init.d/efi script, it appears to me that what is happening is that the
system asks libparted to create a new partition on the hard disk that
will become an EFI system partition.  Either init.d/efi runs before the
partition has been formatted with a fat filesystem, or parted_server is
still running and is keeping the ext2 fs type cached from before it
was formatted.  Either way, the script decides it sees an ext2
filesystem on the drive, and that counts as a non EFI system partition,
and so it throws the message.

Forwarding this from launchpad bug #1418706 in Ubuntu.


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/54daccd8.1030...@gmail.com



Bug#777647: partman-efi always complains when installing from usb

2015-02-10 Thread Phillip Susi
title 777647 partman-efi complains about boot problem whenever the
target disk does not have an efi system partition
thanks

Sorry, I mentioned that it involved installing from a usb stick, but it
also happens when installing from cdrom; using a usb stick has nothing
to do with it.


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/54dacead.4070...@gmail.com



Bug#777166: Please consider not hard coding the name Debian into error messages

2015-02-05 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Package: partman-efi
Version: 59

Recently a new message was added to d-i:

 This machine's firmware has started the installer in UEFI mode but
 it looks like there may be existing operating systems already
 installed using BIOS compatibility mode. If you
 continue to install Debian in UEFI mode, it might be difficult to
 reboot the machine into any BIOS-mode operating systems later.

Please consider replacing Debian with `lsb_release -si` or similar
so that people installing other distros based on debian don't get a
message about a different operating system than the one they are
installing.

-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)

iQEcBAEBAgAGBQJU09MKAAoJENRVrw2cjl5RvvMH/1WYchn0xt77ZRySIodqz9sx
VgDNmuXNbafhE6NATzOSnQxYwwprMHcS5HD8rfNoF6G9Zg//a6pGQa6BJ73C0Zn/
1oeb5fiOFTlcaJbpelDsx7cxH9aWWrSRN4HEiJIeR9u3FssJcJEU9r5Iuw7Kj+f/
uT8qNdibKnyKZziYeCgw06DGdtA6l64PgH4apjlCUcKr1q9x+zlP0HE1lASJxKas
gSfRjeDLtus4SFsAq3EhEZd/W9QfttFTG7aknsJgqmAoVZn2pq+ceogD9CfTA121
8E3iACB7icTGOE7z+j13VRCkIh3Zmi2Xp+b92pj9BkXGnrXpFEOBfx1RZXr5tXI=
=zk48
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/54d3d30a.1030...@ubuntu.com



Bug#738922: [PATCH 2/3] Use dosfstools to format/check fat instead of libparted

2014-03-13 Thread Phillip Susi
index be32c03..8a8fc5b 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -3,6 +3,8 @@ partman-basicfilesystems (91) UNRELEASED; urgency=low
   * Use only mkswap instead of parted to format swap, since this is no
 longer supported in parted3.
   * Remove check_swap, as there is no such thing as fscking swap
+  * Use dosfstools to format and check fat filesystems rather than
+libparted.
 
  -- Phillip Susi ps...@ubuntu.com  Wed, 12 Feb 2014 15:56:24 -0500
 
diff --git a/debian/control b/debian/control
index f326b28..82f7288 100644
--- a/debian/control
+++ b/debian/control
@@ -12,6 +12,6 @@ Vcs-Git: 
git://anonscm.debian.org/d-i/partman-basicfilesystems.git
 Package: partman-basicfilesystems
 Package-Type: udeb
 Architecture: all
-Depends: ${misc:Depends}, e2fsprogs-udeb, ext2-modules, fat-modules
+Depends: ${misc:Depends}, e2fsprogs-udeb, ext2-modules, fat-modules, 
dosfstools-udeb
 Provides: partman-filesystem
 Description: Add to partman support for ext2, linux-swap, fat16 and fat32
-- 
1.8.3.2


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/1394733567-3479-2-git-send-email-ps...@ubuntu.com



Bug#738922: [PATCH 1/3] Remove use of parted for swap create/check

2014-03-13 Thread Phillip Susi
Use mkswap to create and there is no such thing as checking
swap, so remove it.
---
 check.d/check_swap   | 22 --
 commit.d/format_swap | 27 ++-
 debian/changelog |  8 
 3 files changed, 18 insertions(+), 39 deletions(-)

diff --git a/check.d/check_swap b/check.d/check_swap
index 12f9a42..c9e3e2b 100755
--- a/check.d/check_swap
+++ b/check.d/check_swap
@@ -23,28 +23,6 @@ for dev in $DEVICES/*; do
if [ $method = swap ]; then
swap=:
fi
-   [ ! -f $id/format ] || continue
-   if [ $method = swap ]; then
-   log Check the swap space in $dev/$id
-   template=partman-basicfilesystems/progress_swap_checking
-   db_subst $template PARTITION $num
-   db_subst $template DEVICE $(humandev $(cat device))
-   name_progress_bar $template
-   open_dialog CHECK_FILE_SYSTEM $id
-   read_line status
-   close_dialog
-   if [ $status != good ]; then
-   db_subst 
partman-basicfilesystems/swap_check_failed PARTITION $num
-   db_subst 
partman-basicfilesystems/swap_check_failed DEVICE $(humandev $(cat device))
-   db_set 
partman-basicfilesystems/swap_check_failed true
-   db_input critical 
partman-basicfilesystems/swap_check_failed || true
-   db_go || true
-   db_get 
partman-basicfilesystems/swap_check_failed
-   if [ $RET = true ]; then
-   exit 1
-   fi
-   fi
-   fi
done
 done
 
diff --git a/commit.d/format_swap b/commit.d/format_swap
index cdf0589..e2dc4b0 100755
--- a/commit.d/format_swap
+++ b/commit.d/format_swap
@@ -30,24 +30,17 @@ for dev in $DEVICES/*; do
close_dialog
db_subst $template PARTITION $num
db_subst $template DEVICE $(humandev $(cat device))
-   name_progress_bar $template
-   open_dialog CREATE_FILE_SYSTEM $id linux-swap
-   read_line status
-   close_dialog
-   sync
-   if [ $status != OK ]; then
-   db_progress START 0 3 partman/text/formatting
-   db_progress INFO $template
-   db_progress SET 1
-   if log-output -t partman --pass-stdout \
-  mkswap $device /dev/null; then
-   sync
-   status=OK
-   else
-   status=failed
-   fi
-   db_progress STOP
+   db_progress START 0 3 partman/text/formatting
+   db_progress INFO $template
+   db_progress SET 1
+   if log-output -t partman --pass-stdout \
+  mkswap $device /dev/null; then
+   sync
+   status=OK
+   else
+   status=failed
fi
+   db_progress STOP
 
if [ $status != OK ]; then
db_subst 
partman-basicfilesystems/create_swap_failed TYPE linux-swap
diff --git a/debian/changelog b/debian/changelog
index 7d67b7c..be32c03 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,11 @@
+partman-basicfilesystems (91) UNRELEASED; urgency=low
+
+  * Use only mkswap instead of parted to format swap, since this is no
+longer supported in parted3.
+  * Remove check_swap, as there is no such thing as fscking swap
+
+ -- Phillip Susi ps...@ubuntu.com  Wed, 12 Feb 2014 15:56:24 -0500
+
 partman-basicfilesystems (90) unstable; urgency=low
 
   [ Updated translations ]
-- 
1.8.3.2


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/1394733567-3479-1-git-send-email-ps...@ubuntu.com



Bug#738922: [PATCH 3/3] Use mke2fs instead of libparted

2014-03-13 Thread Phillip Susi
+   if [ -f $id/reserved_for_root 
]; then
+   log-output -t partman 
--pass-stdout \
+   tune2fs -m 
$(cat $id/reserved_for_root) $device /dev/null
+   fi
+   fi
+   if [ $status != OK ]; then
+   RET=''
+   db_metaget 
partman/filesystem_short/$filesystem description || RET=''
+   [ $RET ] || RET=$filesystem
+   
T=partman-basicfilesystems/create_failed
+   db_subst $T TYPE $RET
+   db_subst $T PARTITION $num
+   db_subst $T DEVICE $(humandev 
$(cat device))
+   db_input critical $T || true
+   db_go || true
+   exit 1
+   fi
+   $id/formatted
+   ;;
+
 
fat16|fat32)
db_progress START 0 3 partman/text/formatting
diff --git a/debian/changelog b/debian/changelog
index 8a8fc5b..16790bf 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -5,6 +5,7 @@ partman-basicfilesystems (91) UNRELEASED; urgency=low
   * Remove check_swap, as there is no such thing as fscking swap
   * Use dosfstools to format and check fat filesystems rather than
 libparted.
+  * Use mke2fs instead of libparted
 
  -- Phillip Susi ps...@ubuntu.com  Wed, 12 Feb 2014 15:56:24 -0500
 
-- 
1.8.3.2


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/1394733567-3479-3-git-send-email-ps...@ubuntu.com



Bug#739006: [PATCH] Switch to using mkdosfs instead of libparted

2014-03-13 Thread Phillip Susi
---
 commit.d/format_efi | 15 +++
 debian/changelog|  6 ++
 debian/control  |  2 +-
 3 files changed, 18 insertions(+), 5 deletions(-)

diff --git a/commit.d/format_efi b/commit.d/format_efi
index b93ba4b..abedec4 100755
--- a/commit.d/format_efi
+++ b/commit.d/format_efi
@@ -53,11 +53,18 @@ for dev in $DEVICES/*; do
db_subst $template TYPE $RET
db_subst $template PARTITION $num
db_subst $template DEVICE $(humandev $(cat device))
-   name_progress_bar $template
-   open_dialog CREATE_FILE_SYSTEM $id $new_efi_fs
-   read_line status
-   close_dialog
+   db_progress START 0 3 partman/text/formatting
+   db_progress INFO $template
+   db_progress SET 1
+
+   if log-output -t partman --pass-stdout \
+   mkfs.vfat -F ${new_efi_fs#fat} $device /dev/null; 
then
+   status=OK
+   else
+   status=failed
+   fi
sync
+   db_progress STOP
 
if [ $status != OK ]; then
db_subst partman-basicfilesystems/create_failed 
TYPE efi
diff --git a/debian/changelog b/debian/changelog
index 2005268..edf0382 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+partman-efi (40) UNRELEASED; urgency=low
+
+  * Switch to using mkdosfs instead of libparted
+
+ -- Phillip Susi ps...@ubuntu.com  Fri, 14 Feb 2014 15:44:22 -0500
+
 partman-efi (39) unstable; urgency=low
 
   [ Updated translations ]
diff --git a/debian/control b/debian/control
index 9700ef2..94c7965 100644
--- a/debian/control
+++ b/debian/control
@@ -10,5 +10,5 @@ Vcs-Git: git://anonscm.debian.org/d-i/partman-efi.git
 Package: partman-efi
 Package-Type: udeb
 Architecture: i386 ia64 amd64
-Depends: partman-base (= 114), efi-modules, ${misc:Depends}
+Depends: partman-base (= 114), efi-modules, dosfstools-udeb, ${misc:Depends}
 Description: Add to partman support for EFI boot partitions
-- 
1.8.3.2


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/1394737925-3998-1-git-send-email-ps...@ubuntu.com



Bug#738922: parted3 preparation

2014-03-12 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 3/12/2014 9:34 AM, Colin Watson wrote:
 I think it would be very helpful to split up the logical chunks of
 this.
 
 Moving ext2 support means that we need to make sure that 
 partman-basicfilesystems and partman-ext3 land in unstable and
 testing at the same time, and derived distributions need to do that
 too.  Given the general lack of tool support for this sort of
 lockstep change in udebs, I'm uncomfortable with bundling it into
 this change.  Was there a good reason for that or did you just
 think it was tidier?  If the latter, I think it would in fact be
 better avoided.

The purpose of the patch is to stop depending on libparted.  The
existing ext2 support thus needed removed and transitioned to the code
that is currently in partman-ext3.  I suppose I could have copied and
pasted that code into partman-basicfilesystems, but needlessly
duplicating code seemed silly.

 I think it would be a good idea to keep the sync call in place, at
 least in the event that mkfs succeeds.

What for?  If the system crashes 3 seconds later, who cares if the
disk was properly formatted or not?

 -Template: partman-basicfilesystems/text/noatime -Type: text -#
 :sl2: -# Note to translators: Please keep your translations of
 this string below -# a 65 columns limit (which means 65
 characters in single-byte languages) -_Description: noatime - do
 not update inode access times at each access
 
 Even aside from my comments about moving ext2 support, you must
 not remove all these mount option templates.  select_mountoptions
 always picks up the templates from
 partman-basicfilesystems/text/$op, and these mount options are made
 available for various different file systems.

It looks to me like these templates were duplicated in
basicfilesystems and -ext3 because both ext2 and ext[34] needed them.
 With ext2 removed from basicfilesystems, they seem to be unreferenced
cruft now.  If the system always looks for them in -basicfilesystems,
then why are they duplicated in -ext3?  I'm pretty sure I checked the
menu and they still showed up properly.


-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJTIGjoAAoJEI5FoCIzSKrwercIAJr53yNvtcbBdPgYr8j2tjkS
XRfTtnCe37UNcR0cETq1A4egd/BudtgHCzpRwaJbahehjU7gPDebJA0rLGOmgjB+
DcCumNTyI3air5/dgvZVokopIiJ7HutnxNVpq91y6wvXrX4u5F0FNcrDOy3NfKMe
sc/qfonTE1ILEvJ1CrLbREK7kreKZDbVWN90CWIt0qaLvyaxzSHaD54NrkKtxez2
1gmbj/syx/sr400Rc3LCKifIZfY86R2m61xeLhgR2cWebkP899dAh+lqsTMoxs6t
1C5eaZYHnAkWUjrFdsbZk02chrHMmxaNopMThxNK1oSAKu4Zopf/QPhBxHY04BQ=
=H65F
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/532068e8.2040...@ubuntu.com



Bug#739006: Switch to mkdosfs instead of libparted

2014-02-19 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 2/19/2014 10:14 AM, Steve McIntyre wrote:
 This *looks* sane enough to me, but I've not had a chance to test
 it directly yet. What testing have you done, please?

None yet.  I guess I could figure out how to get qemu to emulate an
efi system.  On the other hand, it is a trivial change that is
identical to the patch I submitted for partman-basicfilesystems, and
that one I have tested.


-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJTBNJDAAoJEI5FoCIzSKrwOTwIAIUHh83eNBMxJx6JfYtlDc7W
7bLF3Rh8CaDkULn32vBYz+6ra/pgWfXEjh5LwiRVS+HlLwldfmXS5erP2hl0/DeQ
jxWFbfHHhmab1rzH4kWw24Jr4kYQUe7QiEABRrweIkhWayeR+6sCzD35UStY6tl4
U4L3SNAWj5h54/78ofrUF+BPJ6ssDmPPDLi3PAPjkaWCJxYfAN/JpUzNTQR4Kqtb
RtRk7SrBpi6CxkDZf5sWZ86p6vUBZGfUR3peWpSQurwKjdl/CVQj441sptqo4rwv
C+Obuj3l05Caj15g8aGVLwppOARC3u8f/t2tkV9mTYn5V/kqZ0nzqP/H5IT8bvQ=
=+RBr
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/5304d243.5070...@ubuntu.com



Bug#738923: parted3 preparations

2014-02-14 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 2/14/2014 12:09 AM, Cyril Brulebois wrote:
 The parted3 transition is about transitioning from libparted0 to 
 libparted2? Anyway, why is the -dev package versioned at all?
 Unless you keep several source packages in the archive for a while,
 there's no use for having a versioned -dev package.

I believe the reason is that for a long time, upstream did not manage
their soname versioning correctly, leaving it at zero despite breaking
ABI, and finally with parted 3.0, jumped to 2 since they removed the
filesystem apis.  So debian packaged the library as -0debian1.  I'm
not sure why they put the 0 in the -dev package as well.

-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJS/iZ5AAoJEI5FoCIzSKrwdCgIAKRiJuOjrCgCGJu/QdoR1b1G
YjVhjkU73NxR7KT74jnCEMGDncYPdf3qQyUSqbSdijUT6tkUB2bP5mQooqic96kH
OAAFPcMzM3pxYdzElusC1IHEOV4Y4kPCL8Kq0xutNu05nDMOgx1zZY+Iu0BRGkqF
WkQ2df9VkOL/r/f68iXU9jC68iHdTv6Cj5xsNJ+D/MK/QksYg15ST+V3praFhsAo
MSWoN2ArGz0db7MqM/QrNHFn2L1c5VgOq8R+EnViB7Q1J9/RW0/y9W1TPsbCBE+5
AJGhdAkcvqpCrlu1Y/w1MrAXcekKlhwJtFe0FGXss4oA7kKeBRCKZrUaaidyik0=
=ayLd
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/52fe2679.3090...@ubuntu.com



Bug#738922: parted3 preparation

2014-02-14 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

I made one small mistake with this patch.  It turns out there aren't
symlinks for mkfs.fat{16,32}, so the mkfs line needs changed from
mkfs.$filesystem to mkfs.vfat -F ${filesystem#fat}.

-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJS/kuwAAoJEI5FoCIzSKrwhGYIAJf88Y1Zmd42mFRxYD2jlcqj
3YRN/CQvkKmcqae3DvBm+pRnepGLkSuKmda5RvfRlmqRgfMh/S9n91TmiH38oJeY
PA7Oc7K0jctIHGTt4R7sqexZxEcZKK398vaAuX65H9Pe9xJ7D7sHx6jfFBY1GiTI
wrNA/CpTYZxGjWZKxSKBJGvS10uAUk3AkPE3d+x3QvGvOVIuDOfKwAC5vx8H7BYO
jJ2X+3PA1tjKL6KymLVLpYub8KZ8Zo4b0Me5lw6y2OuF2a6B5gPtTUxKFM6tr5jq
uktFPjxEa/M4dtA5IrogDGxiawZxSYT7oFwgrao7AAqC6ixKPC2/nEAXEFYSoiU=
=vGcJ
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/52fe4bb0@ubuntu.com



Bug#681261: update fstab even if non-existent (possibly fstab.d too)

2014-02-14 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

For what it's worth, I agree with Rui that this is a bug in
live-build, not partman.  I'm not sure how it gets that way since
Ubuntu's live-build still has that touch command, but the live
squashfs image on Ubuntu's install cds does not have the fstab.

Also there is no such thing as fstab.d.  This directory was mistakenly
added to util-linux in anticipation of the feature being added, but it
never was.  It will be removed soon.

-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJS/nvEAAoJEI5FoCIzSKrwv/wH/AwUXUtZBUQawDgAjmPjMrze
4OrkACCzOj3i3C7kPnQkmcuecpanrmf7BTlN8797rlTZUF77s71OW4ahqSbOGCy+
prTT8Zr4jma0ZMvXFWGQqC17UVWNLddJ09PNwF4v4LliOO2qUyFtt2EdzDOc/Orz
ODZz3TAvM3RvgboRNpdXvaIdDhbotV0bsd5LbHco95ffRvkiFV5qlD803QoCjO9u
g+/6S2WxgoIEMuxLjSPMQzEM5kXfnIh/+nNL6T6Ei2oh9Ac2tLd+7Cjx8+lwnUIz
eQm1ZdxDS2CXTfyuUfA4g+cMzdNvZggx7vejvxbJojXbhHUtJorUrK1A2HC8zik=
=pXmD
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/52fe7bc4.8000...@ubuntu.com



Bug#739006: Switch to mkdosfs instead of libparted

2014-02-14 Thread Phillip Susi
Package: partman-efi
Tags: patch

The attached patch switches partman-efi to use mkdosfs instead of
libparted to format the efi system partition, in preparation for the
parted3 transition.

diff -Nru partman-efi-39/commit.d/format_efi partman-efi-40/commit.d/format_efi
--- partman-efi-39/commit.d/format_efi  2012-09-20 10:45:15.0 -0400
+++ partman-efi-40/commit.d/format_efi  2014-02-14 15:40:50.0 -0500
@@ -54,10 +54,13 @@
db_subst $template PARTITION $num
db_subst $template DEVICE $(humandev $(cat device))
name_progress_bar $template
-   open_dialog CREATE_FILE_SYSTEM $id $new_efi_fs
-   read_line status
-   close_dialog
-   sync
+   if log-output -t partman --pass-stdout \
+   mkfs.vfat -F 32 $device /dev/null; then
+   status=OK
+   else
+   status=failed
+   fi
+   db_progress STOP
 
if [ $status != OK ]; then
db_subst partman-basicfilesystems/create_failed 
TYPE efi
diff -Nru partman-efi-39/debian/changelog partman-efi-40/debian/changelog
--- partman-efi-39/debian/changelog 2013-09-08 10:29:40.0 -0400
+++ partman-efi-40/debian/changelog 2014-02-14 15:45:19.0 -0500
@@ -1,3 +1,9 @@
+partman-efi (40) unstable; urgency=low
+
+  * Switch to using mkdosfs instead of libparted
+
+ -- Phillip Susi ps...@ubuntu.com  Fri, 14 Feb 2014 15:44:22 -0500
+
 partman-efi (39) unstable; urgency=low
 
   [ Updated translations ]
diff -Nru partman-efi-39/debian/control partman-efi-40/debian/control
--- partman-efi-39/debian/control   2013-07-13 04:36:55.0 -0400
+++ partman-efi-40/debian/control   2014-02-14 15:46:30.0 -0500
@@ -10,5 +10,5 @@
 Package: partman-efi
 Package-Type: udeb
 Architecture: i386 ia64 amd64
-Depends: partman-base (= 114), efi-modules, ${misc:Depends}
+Depends: partman-base (= 114), efi-modules, dosfstools-udeb, ${misc:Depends}
 Description: Add to partman support for EFI boot partitions


signature.asc
Description: OpenPGP digital signature


Bug#738864: Remove copy operation for parted3 transition

2014-02-13 Thread Phillip Susi
Package: partman-partitioning
Tags: patch

This patch removes the copy operation from the partman menu in
preparation for the parted3 transition, which no longer supports this.

diff -Nru partman-partitioning-99/active_partition/copy/choices 
partman-partitioning-100/active_partition/copy/choices
--- partman-partitioning-99/active_partition/copy/choices   2011-01-19 
00:05:10.0 -0500
+++ partman-partitioning-100/active_partition/copy/choices  1969-12-31 
19:00:00.0 -0500
@@ -1,7 +0,0 @@
-#!/bin/sh
-
-. /usr/share/debconf/confmodule
-
-db_metaget partman-partitioning/text/copy description
-printf copy\t${RET}\n
-
diff -Nru partman-partitioning-99/active_partition/copy/do_option 
partman-partitioning-100/active_partition/copy/do_option
--- partman-partitioning-99/active_partition/copy/do_option 2011-01-19 
00:05:10.0 -0500
+++ partman-partitioning-100/active_partition/copy/do_option1969-12-31 
19:00:00.0 -0500
@@ -1,59 +0,0 @@
-#!/bin/sh
-
-. /lib/partman/lib/base.sh
-. /lib/partman/lib/commit.sh
-
-dev=$2
-id=$3
-
-cd $dev
-
-db_set partman-partitioning/confirm_copy false
-db_input critical partman-partitioning/confirm_copy || true
-db_go || exit 0
-db_get partman-partitioning/confirm_copy
-
-[ $RET = true ] || exit 0
-
-ask_for_source () {
-   local noninteractive
-   noninteractive=true
-   while true; do
-   source_dev=''
-   source_id=''
-   while [ ! $source_id ]; do
-   choices=$(partition_tree_choices)
-   debconf_select critical 
partman-partitioning/source_partition $choices asdfasdfasdf
-   case $? in
-   1)
-   $noninteractive
-   ;;
-   255)
-   return 1
-   ;;
-   esac
-   noninteractive='return 1'
-   source_dev=${RET%//*}
-   source_id=${RET#*//}
-   done
-   source_dev=${source_dev##*/}
-   if perform_copying; then break; fi
-   done
-   return 0
-}
-
-perform_copying () {
-   commit_changes partman-partitioning/copy_commit_failed || exit 0
-
-   name_progress_bar partman-partitioning/progress_copying
-   open_dialog COPY_PARTITION $id $source_dev $source_id
-   close_dialog
-
-   for s in /lib/partman/init.d/*; do
-   if [ -x $s ]; then
-   $s || exit 0
-   fi
-   done
-}
-
-ask_for_source
diff -Nru partman-partitioning-99/active_partition/_numbers 
partman-partitioning-100/active_partition/_numbers
--- partman-partitioning-99/active_partition/_numbers   2011-01-19 
00:05:10.0 -0500
+++ partman-partitioning-100/active_partition/_numbers  2014-02-13 
10:02:35.0 -0500
@@ -2,5 +2,4 @@
 65 toggle_bootable
 66 change_flags
 80 resize
-83 copy
 87 delete
diff -Nru partman-partitioning-99/debian/changelog 
partman-partitioning-100/debian/changelog
--- partman-partitioning-99/debian/changelog2014-01-05 09:33:40.0 
-0500
+++ partman-partitioning-100/debian/changelog   2014-02-13 10:04:13.0 
-0500
@@ -1,3 +1,9 @@
+partman-partitioning (100) unstable; urgency=low
+
+  * Remove partition copy operation for libparted 3 transition
+
+ -- Phillip Susi ps...@ubuntu.com  Thu, 13 Feb 2014 10:03:14 -0500
+
 partman-partitioning (99) unstable; urgency=low
 
   [ Updated translations ]
diff -Nru partman-partitioning-99/debian/partman-partitioning.templates 
partman-partitioning-100/debian/partman-partitioning.templates
--- partman-partitioning-99/debian/partman-partitioning.templates   
2012-03-28 21:01:36.0 -0400
+++ partman-partitioning-100/debian/partman-partitioning.templates  
2014-02-13 10:00:14.0 -0500
@@ -13,33 +13,6 @@
 # :sl1:
 _Description: Computing the new state of the partition table...
 
-Template: partman-partitioning/confirm_copy
-Type: boolean
-# :sl2:
-_Description: Write previous changes to disk and continue?
- Before you can select a partition to copy, any previous changes have to be
- written to disk.
- .
- You cannot undo this operation.
- .
- Please note that the copy operation may take a long time.
-
-Template: partman-partitioning/source_partition
-Type: select
-Choices-C: ${CHOICES}
-Choices: ${DESCRIPTIONS}
-# :sl2:
-_Description: Source partition:
- Please choose the partition which contains the data you want to copy.
-
-Template: partman-partitioning/copy_commit_failed
-Type: error
-# :sl2:
-_Description: Copy operation failure
- An error occurred while writing the changes to the storage devices.
- .
- The copy operation has been aborted.
-
 Template: partman-partitioning/impossible_resize
 Type: error
 # :sl2:
@@ -251,11 +224,6 @@
 # :sl2:
 _Description: Resize the partition (currently ${SIZE})
 
-Template: partman

Bug#738864: Remove copy operation for parted3 transition

2014-02-13 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 2/13/2014 12:08 PM, Michael Tokarev wrote:
 Out of curiocity, why libparted is needed to copy a partition into
 some other place?  How about
 
 dd if=/dev/source-partition of=/dev/dest-partition
 
 ?
 
 (Well, with proper options, maybe with iflag=direct etc, but
 that's details).

I suppose it is theoretically possible.  I just never thought the
option was useful in the first place and don't really have a desire to
try to get that working, so removing it was the simplest thing to do.


-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJS/RtzAAoJEI5FoCIzSKrwGK0IAIcRM08emylKQ6ySYdiq9I93
embsDXGmOcD0IYBJgbU3nTgcqauhos+jzjCOn9jiCqNcVhdwAWCFmOiVpB+XUV9G
xjxHOdWvlPCMDiuHOYsMJefiGzJrNrf1Brp54FlW58jtJdZUzdpaUXb+dLQMnEer
98p97BkYx3hUujTUZAzYkJ4x0r6NZbsLLMsdY/6H5WmiRbRRQyjATMqUYKQr1y64
Ep1D+Jm28TjFny8DjC4twZXNakBPMh428FFjvOz+jE9eRZPYjIFU4qOl3MwRyEd7
7XP+5OFnSRw5SkPLr+KRplcR1rv47mQ+DXPXXM6O57qvZ8SA3y+Hzl1ZusY3daI=
=OTEa
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/52fd1b74.9090...@ubuntu.com



Bug#738919: parted3 preparations

2014-02-13 Thread Phillip Susi
Package: partman-ext3
Tags: patch

In preparation for the parted3 transition, this patch adds ext2 support
to partman-ext3, since my other patch removes ext2 from
partman-basicfilesystems.

diff -Nru partman-ext3-81/active_partition/ext3/choices 
partman-ext3-82/active_partition/ext3/choices
--- partman-ext3-81/active_partition/ext3/choices   2011-01-19 
00:01:08.0 -0500
+++ partman-ext3-82/active_partition/ext3/choices   2014-02-13 
14:07:41.0 -0500
@@ -15,7 +15,7 @@
 filesystem=$(cat $part/acting_filesystem)
 
 case $filesystem in
-ext3|ext4)
+ext2|ext3|ext4)
:
;;
 *)
@@ -46,7 +46,7 @@
  -o $part/formatted -ot $part/method \
  -o $part/formatted -ot $part/filesystem ] || return 0
case $filesystem in
-   ext3|ext4)
+   ext2|ext3|ext4)
if [ -f $part/label ]; then
label=$(cat $part/label)
else
diff -Nru partman-ext3-81/check.d/alignment_ext3 
partman-ext3-82/check.d/alignment_ext3
--- partman-ext3-81/check.d/alignment_ext3  2011-07-23 21:02:07.0 
-0400
+++ partman-ext3-82/check.d/alignment_ext3  2014-02-13 14:10:12.0 
-0500
@@ -24,7 +24,7 @@
filesystem=$(cat $id/acting_filesystem)
mountpoint=$(cat $id/mountpoint)
case $filesystem in
-   ext3|ext4)
+   ext2|ext3|ext4)
if [ -f $id/formatted ]  \
   [ $id/formatted -nt $id/method ]  \
   ([ ! -f $id/filesystem ] || \
diff -Nru partman-ext3-81/check.d/nomountpoint_ext3 
partman-ext3-82/check.d/nomountpoint_ext3
--- partman-ext3-81/check.d/nomountpoint_ext3   2011-01-19 00:01:08.0 
-0500
+++ partman-ext3-82/check.d/nomountpoint_ext3   2014-02-13 14:13:25.0 
-0500
@@ -20,7 +20,7 @@
[ -f $id/acting_filesystem ] || continue
filesystem=$(cat $id/acting_filesystem)
case $filesystem in
-   ext3|ext4)
+   ext2|ext3|ext4)
[ ! -f $id/mountpoint ] || continue
RET=''
db_metaget partman/filesystem_short/$filesystem 
description || RET=''
diff -Nru partman-ext3-81/commit.d/format_ext3 
partman-ext3-82/commit.d/format_ext3
--- partman-ext3-81/commit.d/format_ext32011-01-19 00:01:08.0 
-0500
+++ partman-ext3-82/commit.d/format_ext32014-02-13 14:14:22.0 
-0500
@@ -22,7 +22,7 @@
  -a -f $id/acting_filesystem ] || continue
filesystem=$(cat $id/acting_filesystem)
case $filesystem in
-   ext3|ext4)
+   ext2|ext3|ext4)
if [ -f $id/formatted ]  \
   [ $id/formatted -nt $id/method ]  \
   ([ ! -f $id/filesystem ] || \
diff -Nru partman-ext3-81/debian/changelog partman-ext3-82/debian/changelog
--- partman-ext3-81/debian/changelog2013-12-18 15:57:36.0 -0500
+++ partman-ext3-82/debian/changelog2014-02-13 14:16:19.0 -0500
@@ -1,3 +1,9 @@
+partman-ext3 (82) unstable; urgency=low
+
+  * Pick up support for ext2
+
+ -- Phillip Susi ps...@ubuntu.com  Thu, 13 Feb 2014 14:15:28 -0500
+
 partman-ext3 (81) unstable; urgency=low
 
   [ Updated translations ]
diff -Nru partman-ext3-81/debian/partman-ext3.templates 
partman-ext3-82/debian/partman-ext3.templates
--- partman-ext3-81/debian/partman-ext3.templates   2011-07-23 
21:02:07.0 -0400
+++ partman-ext3-82/debian/partman-ext3.templates   2014-02-13 
16:02:34.0 -0500
@@ -1,3 +1,14 @@
+Template: partman/filesystem_long/ext2
+Type: text
+# :sl2:
+_Description: Ext2 file system
+
+Template: partman/filesystem_short/ext2
+Type: text
+# :sl1:
+# Short file system name (untranslatable in many languages)
+_Description: ext2
+
 Template: partman-ext3/text/ext3
 Type: text
 # :sl1:
diff -Nru partman-ext3-81/finish.d/aptinstall_ext3 
partman-ext3-82/finish.d/aptinstall_ext3
--- partman-ext3-81/finish.d/aptinstall_ext32011-01-19 00:01:08.0 
-0500
+++ partman-ext3-82/finish.d/aptinstall_ext32014-02-13 14:15:00.0 
-0500
@@ -14,7 +14,7 @@
[ -f $id/method -a -f $id/acting_filesystem ] || continue
filesystem=$(cat $id/acting_filesystem)
case $filesystem in
-   ext3|ext4)
+   ext2|ext3|ext4)
ext3=yes
;;
esac
diff -Nru partman-ext3-81/fstab.d/ext3 partman-ext3-82/fstab.d/ext3
--- partman-ext3-81/fstab.d/ext32011-01-19 00:01:08.0 -0500
+++ partman-ext3-82/fstab.d/ext32014-02-13 14:05:31.0 -0500
@@ -13,7 +13,7 @@
method=$(cat $id/method)
filesystem=$(cat $id/acting_filesystem)
case $filesystem

Bug#738922: parted3 preparation

2014-02-13 Thread Phillip Susi
 \
-  mkswap $device /dev/null; then
-   sync
-   status=OK
-   else
-   status=failed
-   fi
-   db_progress STOP
+   db_progress START 0 3 partman/text/formatting
+   db_progress INFO $template
+   db_progress SET 1
+   if log-output -t partman --pass-stdout \
+  mkswap $device /dev/null; then
+   sync
+   status=OK
+   else
+   status=failed
fi
+   db_progress STOP
 
if [ $status != OK ]; then
db_subst 
partman-basicfilesystems/create_swap_failed TYPE linux-swap
diff -Nru partman-basicfilesystems-90/debian/changelog 
partman-basicfilesystems-91/debian/changelog
--- partman-basicfilesystems-90/debian/changelog2014-01-14 
01:05:06.0 -0500
+++ partman-basicfilesystems-91/debian/changelog2014-02-13 
16:31:04.0 -0500
@@ -1,3 +1,14 @@
+partman-basicfilesystems (91) unstable; urgency=low
+
+  * Use only mkswap instead of parted to format swap, since this is no
+longer supported in parted3.
+  * Remove support for ext2, which will be moved to
+partman-ext3.
+  * Remove check_swap, as there is no such thing as fscking swap
+  * Depend on dosfstools-udeb for mkfs and fsck
+
+ -- Phillip Susi ps...@ubuntu.com  Wed, 12 Feb 2014 15:56:24 -0500
+
 partman-basicfilesystems (90) unstable; urgency=low
 
   [ Updated translations ]
diff -Nru partman-basicfilesystems-90/debian/control 
partman-basicfilesystems-91/debian/control
--- partman-basicfilesystems-90/debian/control  2013-07-13 04:49:02.0 
-0400
+++ partman-basicfilesystems-91/debian/control  2014-02-13 16:27:35.0 
-0500
@@ -12,6 +12,6 @@
 Package: partman-basicfilesystems
 Package-Type: udeb
 Architecture: all
-Depends: ${misc:Depends}, e2fsprogs-udeb, ext2-modules, fat-modules
+Depends: ${misc:Depends}, e2fsprogs-udeb, ext2-modules, fat-modules, 
dosfstools-udeb
 Provides: partman-filesystem
-Description: Add to partman support for ext2, linux-swap, fat16 and fat32
+Description: Add to partman support for linux-swap, fat16 and fat32
diff -Nru partman-basicfilesystems-90/debian/partman-basicfilesystems.templates 
partman-basicfilesystems-91/debian/partman-basicfilesystems.templates
--- partman-basicfilesystems-90/debian/partman-basicfilesystems.templates   
2013-11-30 02:23:20.0 -0500
+++ partman-basicfilesystems-91/debian/partman-basicfilesystems.templates   
2014-02-13 15:29:19.0 -0500
@@ -156,39 +156,12 @@
 # :sl2:
 _Description: Reserved blocks:
 
-Template: partman-basicfilesystems/specify_reserved
-Type: string
-# :sl2:
-_Description: Percentage of the file system blocks reserved for the super-user:
-
 Template: partman-basicfilesystems/text/usage
 Type: text
 # :sl2:
 # Up to 25 character positions
 _Description: Typical usage:
 
-Template: partman-basicfilesystems/text/typical_usage
-Type: text
-# :sl2:
-# In the following context: Typical usage: standard
-_Description: standard
-
-Template: partman-basicfilesystems/specify_usage
-Type: select
-# :sl2:
-# Translate standard the same way as in the
-# partman-basicfilesystems/text/typical_usage template.  Do not
-# translate news, largefile and largefile4.
-Choices: ${CHOICES}
-_Description: Typical usage of this partition:
- Please specify how the file system is going to be used, so that
- optimal file system parameters can be chosen for that use.
- .
- standard = standard parameters,
- news = one inode per 4KB block,
- largefile = one inode per megabyte,
- largefile4 = one inode per 4 megabytes.
-
 Template: partman-basicfilesystems/text/specify_mountpoint
 Type: text
 # This is an item in the menu Partition settings
@@ -201,17 +174,6 @@
 # In the following context: Mount point: none
 _Description: none[ Do not translate what's inside the brackets and just put 
the translation for the word none in your language without any brackets. This 
none relates to Mount point: ]
 
-Template: partman/filesystem_long/ext2
-Type: text
-# :sl2:
-_Description: Ext2 file system
-
-Template: partman/filesystem_short/ext2
-Type: text
-# :sl1:
-# Short file system name (untranslatable in many languages)
-_Description: ext2
-
 Template: partman/filesystem_long/fat16
 Type: text
 # :sl2:
@@ -269,48 +231,6 @@
 _Description: Mount options:
  Mount options can tune the behavior of the file system.
 
-Template: partman-basicfilesystems/text/noatime
-Type: text
-# :sl2:
-# Note to translators: Please keep your translations of this string below
-# a 65 columns limit (which means 65 characters in single-byte languages)
-_Description

Bug#738923: parted3 preparations

2014-02-13 Thread Phillip Susi
Package: partman-base
Tags: patch

In preparation for the parted3 transition, this patch removes support
for filesystem creation and checking from parted_server.  The dependency
and linker flag changes can be left out for now and it should still
build against parted2.  They can be added after parted3 has been
uploaded to actually make the switch.

diff -Nru partman-base-172/debian/changelog partman-base-173/debian/changelog
--- partman-base-172/debian/changelog   2014-01-05 09:20:41.0 -0500
+++ partman-base-173/debian/changelog   2014-02-13 10:44:21.0 -0500
@@ -1,3 +1,10 @@
+partman-base (173) unstable; urgency=low
+
+  * parted_server: remove support for filesystem create/check/copy
+and link to libparted2 instead of libparted0.
+
+ -- Phillip Susi ps...@ubuntu.com  Wed, 12 Feb 2014 11:57:20 -0500
+
 partman-base (172) unstable; urgency=low
 
   [ Updated translations ]
diff -Nru partman-base-172/debian/control partman-base-173/debian/control
--- partman-base-172/debian/control 2013-07-13 03:07:32.0 -0400
+++ partman-base-173/debian/control 2014-02-12 11:57:06.0 -0500
@@ -3,7 +3,7 @@
 Priority: standard
 Maintainer: Debian Install System Team debian-boot@lists.debian.org
 Uploaders: Anton Zinoviev zinov...@debian.org, Colin Watson 
cjwat...@debian.org, Christian Perrier bubu...@debian.org, Max Vozeler 
x...@debian.org
-Build-Depends: debhelper (= 9), dh-di (= 2), po-debconf (= 0.5.0), 
libparted0-dev (= 2.2)
+Build-Depends: debhelper (= 9), dh-di (= 2), po-debconf (= 0.5.0), 
libparted2-dev
 Vcs-Browser: http://anonscm.debian.org/gitweb/?p=d-i/partman-base.git
 Vcs-Git: git://anonscm.debian.org/d-i/partman-base.git
 
diff -Nru partman-base-172/Makefile partman-base-173/Makefile
--- partman-base-172/Makefile   2011-01-18 23:56:34.0 -0500
+++ partman-base-173/Makefile   2014-02-12 14:07:57.0 -0500
@@ -1,5 +1,5 @@
 CFLAGS=-Wall -Os -D_GNU_SOURCE
-LIBS=-lparted
+LIBS=-lparted -lparted-fs-resize -luuid
 
 all: parted_server parted_devices partmap
 
diff -Nru partman-base-172/parted_server.c partman-base-173/parted_server.c
--- partman-base-172/parted_server.c2012-07-02 21:01:55.0 -0400
+++ partman-base-173/parted_server.c2014-02-13 10:48:20.0 -0500
@@ -258,48 +258,6 @@
 oprintf(%.0f %s\n, 1000 * timer-frac, timer-state_name);
 }
 
-/* Like ped_file_system_create but automaticaly creates PedTimer */
-PedFileSystem *
-timered_file_system_create(PedGeometry *geom, PedFileSystemType *type)
-{
-PedFileSystem *result;
-PedTimer *timer;
-start_timer();
-timer = ped_timer_new(timer_handler, NULL);
-result = ped_file_system_create(geom, type, timer);
-stop_timer();
-ped_timer_destroy(timer);
-return result;
-}
-
-/* Like ped_file_system_check but automaticaly creates PedTimer */
-int
-timered_file_system_check(PedFileSystem *fs)
-{
-int result;
-PedTimer *timer;
-start_timer();
-timer = ped_timer_new(timer_handler, NULL);
-result = ped_file_system_check(fs, timer);
-stop_timer();
-ped_timer_destroy(timer);
-return result;
-}
-
-/* Like ped_file_system_copy but automaticaly creates PedTimer */
-PedFileSystem *
-timered_file_system_copy(PedFileSystem *fs, PedGeometry *geom)
-{
-PedFileSystem *result;
-PedTimer *timer;
-start_timer();
-timer = ped_timer_new(timer_handler, NULL);
-result = ped_file_system_copy(fs, geom, timer);
-stop_timer();
-ped_timer_destroy(timer);
-return result;
-}
-
 /* Like ped_file_system_resize but automaticaly creates PedTimer */
 int
 timered_file_system_resize(PedFileSystem *fs, PedGeometry *geom)
@@ -940,14 +898,6 @@
 ped_file_system_close(fs);
 return false;
 }
-log(try to check the file system for errors);
-if (NULL != fs  !timered_file_system_check(fs)) {
-/* TODO: inform the user. */
-log(uncorrected errors);
-ped_file_system_close(fs);
-return false;
-}
-log(successfully checked);
 if (part-type  PED_PARTITION_LOGICAL)
 maximize_extended_partition(disk);
 if (!ped_disk_set_partition_geom(disk, part, constraint, start, end))
@@ -1867,83 +1817,6 @@
 }
 
 void
-command_check_file_system()
-{
-char *id;
-PedPartition *part;
-PedFileSystem *fs;
-char *status;
-scan_device_name();
-if (dev == NULL)
-critical_error(The device %s is not opened., device_name);
-open_out();
-if (1 != iscanf(%as, id))
-critical_error(Expected partition id);
-log(command_check_file_system(%s), id);
-part = partition_with_id(disk, id);
-free(id);
-fs = ped_file_system_open((part-geom));
-if (NULL == fs)
-status = n

Bug#717239: partman-ufs: depends on obsolete package

2014-02-12 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

It also depends on ufsutils-udeb, which is only built on kfreebsd.  It
seems to me that this package needs changed from all to kfreebsd only.
 This prevents building d-i monolithic images.

-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJS+5SYAAoJEI5FoCIzSKrw13QIAI3M83yQ0JpLmOV5aEPwaVNI
/aMG5EBLtLQaZDoLjtEXnIvddazS7dFZhaW5pboPdArEFD3l2OtpSTD5FSJbt5US
dUKosgdsyErQ/TWgns89rXkxcpFB1qp823QeGnA3io6bGHeGkxzpAjOywqDXbPq1
rS8Kp1g1bGxg9o9A1XRPX85Z6EZOQgqx48DItU0UDOfOQDMZbTbGdZzp4YjP2d4h
g9acd1hk/vUD0YyobPY/nmOkeaxEa2nduwOTVEzXVekU6d2nlGLULRU+Nt0YpUZg
CLPBMKfyK/77JIkBZ0d1Rc1J1FtndhzV04GSUAZrN0xhqqZGlq/q2o5S7zJ6bCA=
=/0GG
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/52fb9498.9080...@ubuntu.com



Bug#696123: partman-base: need progress wrapper for non-libparted-based filesystem operations

2014-02-11 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 2/10/2014 4:36 PM, Colin Watson wrote:
 Phillip, you've asked me about this a number of times.  I think the
 work involved is fairly clear, although I don't seem to have
 managed to make time for it personally.  Perhaps, rather than
 continuing to ask me about this, you could have a go at the
 necessary d-i integration?  I don't think it should be that hard
 really.

Could you point me in the direction of how to test with my locally
built parted3 udebs?  I tried using debian-installer and putting it in
the local directory but it is ignored I guess because it doesn't go in
the initrd, but is loaded later from the pool.  In the case of the
netinst image, it just downloads the official 2.3 udeb from the mirror
since there is no pool, and when I tried make build_cdrom_isolinux, it
just prepared the kernel and initrd in a directory rather than making
an iso.


-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJS+ot+AAoJEI5FoCIzSKrwXb0H/16Zfn8tA4aZW59RphthGzsM
7r+WRJ2hW8fsdU7Xq8xzxy1W2caECYnqTm/+mKHRdLOcsEEdx+WwsRquGYSLEhyP
5eNQiBAHnjGajaPyEXioG/BUX9mP6x7DrbgtsswtQg5TD3DQCuOMWpx4LwRPLlvk
ZuQPwib3Qb+5l6c8ZVKPV4wFxV5N9xxCuXqt5uA6AUCLTYDhK22S8bustnRxiTt5
XK2TM0flD9kcc3w1pjAW1rhQVbRwgEP+GwOq77GvJhMJUUV7qLwQMclOefKIk0kQ
YYQnjnq0nqoOQ3P3AbLA2XwsresutRay/AEpAEL32fFcRKy2BX6gcKjO97LXpE0=
=yPlS
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/52fa8b7f.5080...@ubuntu.com



Bug#696123: partman-base: need progress wrapper for non-libparted-based filesystem operations

2014-02-10 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Say Colin, it has been over a year so I thought I would ping on this.
 Is this really the only issue that is still preventing the parted3
transition?  If so I think it may be time go go ahead and make the
move rather than continue to block on this, which really is a very
minor issue given that ext4 only takes seconds to format these days
thanks to lazy_itable_init.

-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJS+TFTAAoJEI5FoCIzSKrwc8IIAIjkcTN5Bc42LyL3JdLqihNL
7k6BZL5XPbSHaf+qiBKSs1T5fDa16ZEqYZUzP7nfDHDCcnXAHt1+R2bZ1sAIfyRK
NAzLxD+DhD+oPsyT61XHMfMh/HiNLLwZk1sieNQhq/tQ4hzJVZ45YQJ/JE+z2ht1
v21Nr+fIA+p1PEipQvjQSMECGBXLlE5+opxJI7vMvSQ1brmOUvJxi9qSlIQQNov5
eBBTaPSBEa25cBB13LtNWNwEsK2PIN7k3fBJa/IfF3lMe172WCVZEb2fvRKoKSz+
iWP0j+6DX5Wlaxj1IAQhTOc9UvdHRvNMuQN3lLQSH8C7G+KKx4Yw0bSCgP2FWAU=
=9Yk2
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/52f93153.7080...@ubuntu.com



Bug#610359: d-i rc1 fails on software raid system

2013-07-16 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

 Do you mean that removing logical volume on ISW and creating an MD 
 mirror (i.e., software mirror via normal kernel modules) I would
 get a software mirror accelerated with ISW hardware?

There is no hardware acceleration as this is a fake raid controller.
The only reason to use it instead of mdadm software raid is if you
must dual boot with Windows.  If you don't do that, then mdadm is much
better supported and feature complete.

For what it's worth, you can choose to install grub to the /dev/mapper
device and it works in Ubuntu.  I'm not sure if we're carrying patches
that are needed off the top of my head.


-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJR5W+LAAoJEJrBOlT6nu75CXgH/j4K8j1iKUPgo026Nv3wV0UQ
hxrrlN8C2KbTKuRENM1QwNwhefQlTOQlkxEC8259orK8aHOPrWoRzedvKnqpf7IJ
XoNrA77LIpxBL3cqOKZbfs7YRY64L2PHjQzRWrs+bq7qpxJdCTV5EvkZ4qjdIhC9
u3HboxG5YxmbzYpKxfbXaeuUMDuPeKrbiDuhL9ArsZbnYH3aDif7yViYJY+E/eAw
06xCBz39dEQYpdY+j6YWdT3P/c4FRFEYWN8x6a0cxjYpew5d3QMuNVR3z9escrd6
hptqp9lUPoGwNnCNSHBj8HaJtH0lOn5wYXkdO7OqxJ6BiblGW1R5G8OP9lnlgog=
=QzCy
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/51e56f8b.3020...@ubuntu.com



Bug#700633: Debootstrap is very slow. Please use eatmydata to fix this.

2013-04-01 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 3/31/2013 4:17 PM, Marc Haber wrote:
 The following patch introduces an --include-early option which
 allows to introduce eatmydata early enough:

Is this intended to be applied instead of, or on top of my initial
patch?  If the former ( which I'm guessing it is ), I don't see where
in-target is modified to *use* it?


-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJRWZY4AAoJEJrBOlT6nu75s1oH/isdxtsbIsI4jcp/sVNSoIXt
XKS+2e/4qoQH3pnAeUvRWqYidgoCfZZXMK08FBcD2vWt+dgCBvroXTK+h3MU7bav
+1QiyEr9P0ba/w5qbCNf84YjOLd3AXSTBmi1vlQDC4tXIzGi2KyBM5gWKDLKL7i1
sPgJFdiDb8AYOI0WB5fCRr/ba2fQgGW81TdIK7aTE2/2IOrlisQ6NyNhahglNJ2G
qHODM5vjo0SkBvHqlhM+npOsKdva1l6IMRHlfoonGtx+o01YzNlC4SnonIzRvx8S
5+vhXIzRYCH/WeY7b45ygdY/i1Q8uJFxecQn2Kcw2GkbEZeaPHlql9as1kO4HP0=
=vyZQ
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/51599638.8080...@ubuntu.com



Bug#700633: Debootstrap is very slow. Please use eatmydata to fix this.

2013-02-16 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 02/16/2013 06:54 PM, Bob Proulx wrote:
 # time debootstrap wheezy testinstall
 http://localmirror/ftp.us.debian.org/debian ... real2m58.577s 
 user0m49.639s sys 0m8.749s

Wow, that's pretty fast.  I was testing on a server with a raid5 array
and was seeing nearly 10m original time, down to 1-2m patched.  And
that's with a pre built debootstrap tarball so nothing needed downloaded.



-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with undefined - http://www.enigmail.net/

iQEcBAEBAgAGBQJRICQUAAoJEJrBOlT6nu75g+UIANj9vklDF3cYmxrsxTs5ImIB
ev0IsbrjLz7BePRG/OCusG/V1i2Q4R8XHEfHxJ3vkP/aAh/q7uqsDXJ6ozjSG7P/
32BmRjJx9VBzmKVQIEFfjbFbqIEr+3SBRMAWF5eCvnGXd0rXhBjcslSJ4xbVVq/M
IuwXw8clwgo2POSWKxNmK5/Rk8EJZddRil6ioGeOKq3hRr5jDAx7W/x30dnYDtQE
ImjTCP+n0rDd97VncblRA3Opsurzeqjqhs5YRHS7CFumph5yUH5GuVNQrm71Ztc7
T34g8UnkDUnPQ/YWrM1UExpokwkauEcxxtwn8RkS35Ie+AKBINotykDBspoXbHA=
=/i4U
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/51202414.6020...@ubuntu.com



Bug#700633: Debootstrap is very slow. Please use eatmydata to fix this.

2013-02-15 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Package: debootstrap
Version: 1.0.46

Debootstrap is very, very slow. I watched a server install crawl by
for something like an hour at a rate of about 1 package per second. A
simple debootstrap chroot takes 10 minutes to setup, with a
predownloaded tarball. This slowness is due to dpkg making excessive
use of fsync(). Requests to fix dpkg in the past have been rejected
and I was told to use eatmydata. After patching debootstrap to add
eatmydata to the required list, and activate it during the second
stage install, the time to construct the chroot dropped from 10m to
2m. This should also make installing new systems MUCH faster.
-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJRHlSFAAoJEJrBOlT6nu75+dcIAK/GwMGqqZDQJRLS7//MoRYH
DXFJa4PExLKnvyDvqONMV/gRNY7fc6E8kuEERZeySflgYzdxU28CwNCgB2+UFydh
DiyARnEv+fXmjnSS0YtvOt8ZTr2z0UuMXpuMGz0+Yca88gGEnlPiB0tqcbQx0mrG
GpGBnYLKpdk/mz5tsRo+LFgAKC8fsp6ievf+6w7vIsjrkWkuL5i+3lPBjmGlBtZ6
Bfaxk31Px9OBCpI1sX/NFNilQl++MAK2Ejm6FMblRSW7aWJXROHIGJzz74wQKqjP
Kweqf6bqrcKQnBlKwKDooniIEqbYs0ewgF9jqkx+dz60taBPs+2swKtv2d82pRQ=
=MoT3
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/511e5485.90...@ubuntu.com



Bug#700633: patch

2013-02-15 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

This simple two line patch does the trick:

- --- sid 2013-02-15 11:03:15.384977238 -0500
+++ sid.orig2013-02-15 10:50:23.381293976 -0500
@@ -16,7 +16,7 @@
 esac

 work_out_debs () {
- -   required=$(get_debs Priority: required) eatmydata
+   required=$(get_debs Priority: required)

if doing_variant - || doing_variant fakechroot; then
#required=$required $(get_debs Priority: important)
@@ -68,7 +68,7 @@

 second_stage_install () {
setup_devices
- -   export LD_PRELOAD=/usr/lib/libeatmydata/libeatmydata.so
+
x_core_install () {
smallyes '' | in_target dpkg --force-depends --install
$(debfor $@)
}
-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJRHl4QAAoJEJrBOlT6nu75XKMIAM1neTpblkJAoZbbOTQ9EqwH
JqUPT4guGGkzZyRmrj7yk7arwhOazEWY9A5YVTG7O6M6EcUaMdAR5Wb+E8LJ8/8a
OZ7oJu5AcDARepYNuRe4Fi2jOidIU3lYL2DbCNuJEtGhT+KU0Qkoz3RMs1zdaLaI
UYN17bwlFczAXVd9iYOP7GHu/T1nYg/5Es7xf9xR+bieyWUuo8ANIArrOdcXwoj5
Cep5IN3D3lZarKqrHXiV7rSSFsl7YLSTZlqS8el13hxxZU9UP/qFzAdeqqL2hR/y
pWK35R9BU7qiH3FP8+B7OmTh5qIRyV49mn30JYNYlWyrQmsXN+XKgVIdYq5vjW0=
=6v3+
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/511e5e10.9070...@ubuntu.com



Bug#700633: patch

2013-02-15 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 2/15/2013 12:18 PM, Didier 'OdyX' Raboud wrote:
 Reversed patch?

Woops, yes...



-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJRHoNYAAoJEJrBOlT6nu75DZUH/3WjYpr4mELvRFs/9s3RHH87
+UhL/HJDesUHsRRy33j0mVieJqUm055zxQ3v/lHI04v4+Tb2ymhOVxcLrYEJw+E+
lrZA4KicxXbVfwa+IwgQX2aNG6qcW3SPdXMnRdlBxaFhZzQRI0hbm6S/MUbCTkp1
/MKWUd+MmJSSp1wh97g7fVttWfbKjeDLiuT09owVHBIr82qItwItvbJr4Z9PCH/T
EEWwMbm0bRxzRVGxqPQOF+IqjEeFsxM1GaRVhzjOpNM3zGrN+5qtptCeH9Ze98P7
i6eZ5osRkvdyJ4GBpSz3qSt9lQgMqvPwM+1OwX68+rIv4rnq2Np1UExYATDtcX0=
=jmqW
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/511e8358.7000...@ubuntu.com



Speeding up install and other questions

2013-02-15 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

I've been working on making debootstrap and installing go faster, as
they currently take about 5x longer than they should ( on ext4, it's
even worse on btrfs ).  I have patched debootstrap to add the
eatmydata package to the required list and enable it for phase 2, and
this makes the bootstrap 5x faster.  I read that the actual installer
uses cdebootstrap instead, so I have been looking at making the same
changes to it and have some questions.

debootstrap manually extracts the required packages, and fakes a
status file showing them to be unpacked before executing the extracted
dpkg in a chroot to install the important packages.  cdebootstrap does
not bother faking the status file, so it ends up running dpkg to
reinstall all of the manually extracted packages.  This seems wasteful.

In the debootstrap case, it is meant to be able to be run from a
foreign system, so the manual extraction makes sense, but for the
normal installer, it already has a functioning dpkg on the installer
system, and it can be passed --root= to point it to the target rather
than being run from the target in a chroot, so why not skip the
extract phase and do this instead?

-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJRHqIVAAoJEJrBOlT6nu75aawH/3yBF1tqWCC0K1N2yIb9hl9i
/QuD99qSTVVQRPe1dvgroRYkc/snfdI+NuZYbTQAfrEScnNkX6ig48N71YcP3jJl
JcnbsUI6sw1G6a9oEHYTYBawl8MFS3FY8/sBkGyXhQ8Jl44cibltSWxXj381bgRX
5vC6wwwoeM9XnqCPHXng67eSlx3hxTY5F80FHot8VMMV1DtNC0ffqxX5y1DGq1kS
Vfg81fCQI7ssggqaBqpGLVxMqdCXepEVFI/MN676p2TcVifL1dXkPEDoUWScQi74
IV5TK/co9UWNdl4S9gE80Fdr8asW2SSc+6eJM2xZ04GTOMD/zidvTEkxaLjwII8=
=japY
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/511ea215.1090...@ubuntu.com



Bug#700633: Debootstrap is very slow. Please use eatmydata to fix this.

2013-02-15 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 02/15/2013 09:09 PM, Cyril Brulebois wrote:


- --force-unsafe-io only eliminates *some* of the syncs, and seems to
only be applied in the latter part of the installation that takes
place after debootstrap.

I started working on a patch to dpkg to add another --force option to
eliminate the rest of the syncs a year or two ago and was told on the
dpkg mailing list to just use libeatmydata, as it would also make sure
any syncs done by the [pre|post][inst|rm] scripts are also disabled.


-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with undefined - http://www.enigmail.net/

iQEcBAEBAgAGBQJRHv+pAAoJEJrBOlT6nu75t28H/0tfko7zwRQ0Y1hWN8hE1TXj
ZaDxoIZ1aQ6fVAm4uow+7vXiP4TBPnR61sHvs7mO791sNlVutG7Lz6D4XG7dWmID
N/hKRbX/wYz02rEeuEw2z3MrynfWq53fX9I+PBn0aFyBNtLM0oHMGoaSDw9TRnbB
/+9+v+nLd//dA8gvXGLsx97ildV6uS4ZtI814l3uC3DnhrSBvqP4TVOtaGmiGe7m
NZdb5FyfSt3vGBeMrFsuAHDl8IyQyWX6gDcZvUuz7qcSXHS2NCrAAs0IsoM41bpN
3oZtVJn6uYB/rvkR4DfyYh8FerhiQ9x/Qx96Dmxz6cE3+CrGoYdY3FLGzsB0ydQ=
=mn+R
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/511effa9.3050...@ubuntu.com



Bug#696123: (no subject)

2012-12-16 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

I'm confused.  My understanding is that partman was already using
external utilities for everything besides fat, hfs, and ext2 resize,
and that there were already bug reports covering the lack of progress
indication with those.  Parted3 appears to have moved the resize
functions for fat and hfs to a separate library, so partman should be
able to just link to the new library and be no worse off than before,
with the exception of ext2, and well, who really cares about that?

What you describe here seems to be targeted at fixing the lack of
progress indication already covered by other bugs.  While providing
progress indication is nice, I'm not sure why it is still blocking the
migration to parted3.

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with undefined - http://www.enigmail.net/

iQEcBAEBAgAGBQJQzoRfAAoJEJrBOlT6nu75mVgH/A6LQ34N3rAl2nCy1TH9dvB4
FWZiIhNcDriBkN2AFjfbTMs37P59N5Pf4xm+r3O9X4uwV7dr5nF7oyDx1DYIgOWB
i08jyfm6HIsBnoOxCvDr/zVumFN3AEfJ3Od0AyDzufZJ+WIEsrFUOKHv1iUHgXem
co1cGKfGgJiRd0ObbgSsm91VzXK3cimR7yL4AeXzqls7TwLJEx0YVpUw1bGv8Aaz
iA0Az5xce+v0IjWBP60WNUSkL2F4GQXJCZvHeCVI7T/G+gtUk9OlXxyzFtQAVl8Q
EpM2/JTXJZ/SzF7eMWVe0khVJURgDtn9p2sICzwSIXFMXgu45k/DUiudpkv5CHg=
=yS46
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/50ce845f.7090...@ubuntu.com



Bug#696123: partman-base: need progress wrapper for non-libparted-based filesystem operations

2012-12-16 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

 I have trouble believing that this is going to be easier than
 patching the most common utilities to add an option for a
 machine-readable progress indicator.

Not too long ago I was looking at patching gparted to support this and
while it did not seem difficult to process the output of the e2fsprogs
utilities for the progress information, I was wondering why the
various fs utilities don't already have a flag to generate machine
readable output... and I think some of the utilities supported this,
and some did not, even within e2fsprogs.

IIRC, I found that at least some of them already have a flag to write
output to a specified auxiliary fd to indicate progress, it just
seemed that a standard needed decided on and implemented in all of them.


-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with undefined - http://www.enigmail.net/

iQEcBAEBAgAGBQJQzpAJAAoJEJrBOlT6nu75XrkH/RdWFitPR0tsOXWDv5rFtYuu
mWSLXzcVT/meq8o/pfr0QthOjggeFNG5Bl1cKSQYEVs6kwLd37HP1CW93jSLqjHp
9t99OgIaE1xH320yp2fpX4/ScMl8eEcV8tOUzPlyth+RDMU0tI11lMOIKZb/eeha
I74zQ3CkBY6tT026ZSodsaq1bua6dmY6lzgYvxIImXTYDY58OvjL5LbmM/l4urJS
ZlVMTIxfE7FYX+YPKwYcqn1j5MirQlbniaFgY1Pi6eVv26KqghKuuZ5RH16Fuxdi
qcMA52Xq5J3fZl5O0kMsZLBBvM3B3g1ORDypHc+gEwrrGPVPWjfOHj7BCSsnx/E=
=VINZ
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/50ce9009.4030...@ubuntu.com



Re: [Utnubu-maintainers] Bug#350250: dmraid udeb needed

2006-03-01 Thread Phillip Susi
I posted this reply several weeks ago and have not seen any further 
messages on the subject, but I am not subscribed to debian-boot, so was 
there follow up discussion there that I missed?


Phillip Susi wrote:
I posted to utnubu-discuss a few weeks ago that I was working on this 
myself.  I never did quite figure out the proper way to udebify the 
package and integrate it into debian-installer, but I did get it 
integrated into mkinitramfs.  You can find my attempt at 
http://revu.tauware.de/details.py?upid=1566.


Is debian using initramfs in a compatible way with ubuntu?  Specifically 
do kernel packages' postinst build the initramfs with mkinitramfs?  If 
so then you might want to integrate my changes in that area as well.


Filippo Giunchedi wrote:

On Sat, Jan 28, 2006 at 10:22:23AM +0100, Marco Amadori wrote:

Package: dmraid
Severity: wishlist

As noticed in dbug:338719, a dmraid udeb is needed in 
debian-installer to proper configure software raid arrays.


I've uploaded a new version of dmraid right now with dmraid-udeb (I'm 
shipping

dmraid-udeb binary linked with libselinux, could that cause problems?)

I'm also CC'ing debian-boot since some people might be interested







--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]