Re: Structural changes needed for switch to 2.6.25 kernels

[Ian Campbell]

On Wed, 2008-05-28 at 02:14 +0200, Frans Pop wrote:
> Today I have done the initial work to prepare the switch to 2.6.25 for D-I.

Now seems like a good time to remind you that I would like to add
-686-bigmem kernel udebs to the mix to support running D-I under PAE
Xen.

http://lists.debian.org/debian-boot/2008/05/msg00271.html

Cheers,
Ian.
-- 
Ian Campbell

Most people's favorite way to end a game is by winning.


signature.asc
Description: This is a digitally signed message part

Re: Structural changes needed for switch to 2.6.25 kernels

[Otavio Salvador]

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Frans Pop <[EMAIL PROTECTED]> writes:

> Please comment based on my git log entries below. If anyone wants to review 
> the actual patches before they are committed I can provide them.
> If there are no comments I'll commit in about 2 days.

First I'd like to thank you by doing it. It is indeed a big change.

> Overview of changes
> ===

All changes looks ok and well designed. No need for patch review I
think.

Please go ahead and commit them if noone objects (specially porters).

- -- 
O T A V I OS A L V A D O R
- -
 E-mail: [EMAIL PROTECTED]  UIN: 5906116
 GNU/Linux User: 239058 GPG ID: 49A5F855
 Home Page: http://otavio.ossystems.com.br
- -
"Microsoft sells you Windows ... Linux gives
 you the whole house."
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Processed by Mailcrypt 3.5.8+ 

iD8DBQFIPLIDLqiZQEml+FURAgRoAJ0UziyPKtDrgmlQGoce8t9saXkdRQCfYrRL
F0tQ74+w47fLe6Eft5x29Dk=
=Jd5E
-END PGP SIGNATURE-


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

Re: Structural changes needed for switch to 2.6.25 kernels

[Frans Pop]

On Wednesday 28 May 2008, you wrote:
> The first two changes below are minor: modules made optional because they
> are no longer available.

Damn. Knew I was missing something. There was another dropped module.

> Overview of changes
> ===
  0.  irda-modules: irport module is no longer available
  ---
> 1.  scsi-modules: module atp870u is no longer available
> ---
> 2.  ide-modules: generic module no longer available (powerpc only)
[...]


signature.asc
Description: This is a digitally signed message part.

Structural changes needed for switch to 2.6.25 kernels

[Frans Pop]

Today I have done the initial work to prepare the switch to 2.6.25 for D-I.

Note that I have done no checks at all for new/removed/changed modules, not 
even for i386/amd64! I'll leave that to others.

What I have done is only the structural changes required to build using 
basically the current definitions, which was somewhat complex this time. 
All architectures build now (with updated version of kernel-wedge).

The work is based on 2.6.25-3, but -4 was uploaded today. Also note that the 
first upload of l-m-e seems to be having troubles on some arches.

The first two changes below are minor: modules made optional because they 
are no longer available. The third change is a simple module rename.

Changes 4 and 5 are the most invasive. I've had to create several new module 
udebs to cope with new interdependencies and to avoid unwanted 
dependencies. The first of these solves the problem earlier reported by 
Martin Michlmayr and affects partman-crypto.

Changes 6 and 7 are minor arch specific fixups needed after change 5.

Change 8 is in the installer build system and also needed after change 5.


Please comment based on my git log entries below. If anyone wants to review 
the actual patches before they are committed I can provide them.
If there are no comments I'll commit in about 2 days.

Cheers,
FJP

Overview of changes
===
1.  scsi-modules: module atp870u is no longer available
---
2.  ide-modules: generic module no longer available (powerpc only)
---
3.  crypto-core-modules: blkcipher has been renamed to crypto_blkcipher
---
4.  Move dm-crypt to a separate udeb

The dm-crypt module is moved from md-modules to crypto-dm-modules to
avoid having md-modules depend on crypto-core-modules. With this change
all dependencies on crypto-core-modules are defined in kernel-wedge.

Exception is m68k which did not have md-modules to begin with.

 !! Consequence of this change is that partman-crypto will have to start
 !! loading crypto-dm-modules when needed.
---
5.  Changes because of separate nls_base modules

Most architectures (except m68k and some arm, mips and mipsel subarchs)
now have a separate nls_base module on which all nls_* modules and some
file system modules (isofs, fat, ntfs, jfs, hfs) depend.

* move the isofs module from ide-modules to a separate udeb
* let cdrom-core-modules depend on the new isofs-modules
* this leaves ide-modules empty for m68k, so remove it
* add an nls-core-modules udeb

 !! Consequence of this change is that hd-media targets will need to include
 !! the isofs-modules udeb in their pkg-lists.
---
6.  powerpc fixup: move nls_utf8 from core-modules to hfs-modules
---
7.  m68k fixup: mac does not have the nls_base module

Create a separate modules definition directory for m68k-mac as that
subarch does not need nls-core-modules while amiga and atari do.
---
8.  Include isofs-modules in hd-media images

Now that the isofs module has been split out of the ide-modules udeb
(kernels >=2.6.25), it needs to be included explicitly.


signature.asc
Description: This is a digitally signed message part.

win32-loader 0.6.6 MIGRATED to testing

[Debian testing watch]

FYI: The status of the win32-loader source package
in Debian's testing distribution has changed.

  Previous version: 0.6.3
  Current version:  0.6.6

-- 
This email is automatically generated; [EMAIL PROTECTED] is responsible.
See http://people.debian.org/~henning/trille/ for more information.


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

partman-base 120 MIGRATED to testing

[Debian testing watch]

FYI: The status of the partman-base source package
in Debian's testing distribution has changed.

  Previous version: 119
  Current version:  120

-- 
This email is automatically generated; [EMAIL PROTECTED] is responsible.
See http://people.debian.org/~henning/trille/ for more information.


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

Re: Memtest

[Frans Pop]

On Tuesday 27 May 2008, Daniel Dickinson wrote:
> Jérémy Bobbio <[EMAIL PROTECTED]> wrote:
> >   This tester runs independently of any OS - it is run at computer
> >   boot-up, so that it can test *all* of you memory.  You may want to
> >   look at `memtest' (in package `sysutils'), which allows to test
> >   your memory within Linux, but this one won't be able to test your
> >   whole RAM.
> >
> > I think I do prefer to encourage people to use the proper tools than
> > to provide a less than accurate test in the installer main menu.
>
> I think what he is thinking of is having memtest in the syslinux menu
> on the CD.  That would be just like running a memtest bootdisk except
> that it'd be an option on the install cd.  Which would be useful
> (unless it's already there and I missed it).

That's not what Giacomo says in his last message. However...

See #230860.
Patches to implement that in debian-cd would be welcome.
Note that debian-cd would have to edit the syslinux menu files (which are 
provided by D-I) to add an appropriate menu option.

Adding memtest86+ to debian-cd would have a much higher chance of being 
accepted than anything in D-I, but OTOH we have recently accepted an option 
to stress test a new system (try booting a daily built image with
'modules=cpuburn-udeb' added to the boot parameters).
If it is light enough and does not have any weird dependencies we could also 
consider a memory test option as an _optional_ component (which means that 
it would never be offered very early in the installation).

Please submit a detailed proposal for implementation and we can look at it. 
Please provide facts about why it would be better than having memtest.
This current discussion does not seem to be going anywhere anymore.

Cheers,
FJP


signature.asc
Description: This is a digitally signed message part.

Re: Memtest

[Daniel Dickinson]

On Tue, 27 May 2008 12:17:22 +0200
Jérémy Bobbio <[EMAIL PROTECTED]> wrote:

>   This tester runs independently of any OS - it is run at computer
>   boot-up, so that it can test *all* of you memory.  You may want to
>   look at `memtest' (in package `sysutils'), which allows to test
>   your memory within Linux, but this one won't be able to test your
>   whole RAM.
> 
> I think I do prefer to encourage people to use the proper tools than
> to provide a less than accurate test in the installer main menu.

I think what he is thinking of is having memtest in the syslinux menu
on the CD.  That would be just like running a memtest bootdisk except
that it'd be an option on the install cd.  Which would be useful
(unless it's already there and I missed it).

Regards,

Daniel

-- 
And that's my crabbing done for the day.  Got it out of the way early, 
now I have the rest of the afternoon to sniff fragrant tea-roses or 
strangle cute bunnies or something.   -- Michael Devore
GnuPG Key Fingerprint 86 F5 81 A5 D4 2E 1F 1C  http://gnupg.org
No more sea shells:  Daniel's Webloghttp://cshore.wordpress.com


signature.asc
Description: PGP signature

Bug#483182: setting package to libdebian-installer-extra4 libdebian-installer4-dev libdebian-installer4-udeb libdebian-installer4 libdebian-installer-extra4-udeb libdebian-installer ...

[Otavio Salvador]

# Automatically generated email from bts, devscripts version 2.10.28
#
# libdebian-installer (0.59) UNRELEASED; urgency=low
#
#  * Apply patch to include 'Apple Inc.' as a possible vendor string for a
#Intel's based Mac. Thanks to Julien BLACHE <[EMAIL PROTECTED]> for
#the patch (closes: #483182).
#

package libdebian-installer-extra4 libdebian-installer4-dev 
libdebian-installer4-udeb libdebian-installer4 libdebian-installer-extra4-udeb 
libdebian-installer
tags 483182 + pending




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

Re: Skip kbd-chooser when running under Xen

[Ian Campbell]

On Mon, 2008-05-26 at 12:38 +0200, Ferenc Wagner wrote:
> Hi,
> 
> So far I've received no comments on this, may I ask for some again?

Will this patch do the right thing if Xen is being installed using the
paravirutal framebuffer device present in 2.6.26-rc? It looks to me as
if it will not.

Ian.

> 
> Thanks,
> Feri.
> 
> Ferenc Wagner <[EMAIL PROTECTED]> writes:
> 
> > Frans Pop <[EMAIL PROTECTED]> writes:
> >
> >> On Thursday 08 May 2008, Ferenc Wagner wrote:
> >>
> >>> vmlinuz-2.6.18-6-xen-686 suite=etch DEBIAN_FRONTEND=text
> >> [...]
> >>> kbd-chooser[881]: ERROR **: Couldn't get a file descriptor referring to
> >>> the console
> >>> main-menu[670]: WARNING **: Configuring 'kbd-chooser' failed with error
> >>> code 1
> >>
> >> That seems to make fairly clear why kbd-chooser is failing...
> >
> > Still not fully clear to me, but name it an already fixed Xen bug.
> >
> >>> vmlinuz-2.6.25-1-xen-686 console=hvc0 suite=etch DEBIAN_FRONTEND=text:
> >>
> >> And apparently here kbd-chooser _can_ find a file descriptor for the 
> >> console.
> >
> > Yes, it looks like hvc0 is a much better console than the earlier
> > hacks.  Which is not particularly surprising.
> >
> >  * Should we skip running it from a virtualized D-I?  (I guess yes,
> >especially that it's so simple to achieve.)
> >>
> >> If I understand you correctly: no, we do not want to add a question for 
> >> this.
> >
> > Maybe not.  What I'm talking about it the following prompt:
> >
> > Select a keyboard layout
> > 
> >
> > Keymap to use:
> >   1. American English [*]  22. Italian
> >   2. Belarusian23. Japanese
> > [...]
> >
> > which doesn't appear during serial installation.
> >
> >>> Yes, testing for functionality would be much better.  Like skipping
> >>> kbd-chooser if there's no keyboard attached.  Is serial installation
> >>> handled by checking the kernel command line?
> >>
> >> I suggest someone starts looking at the kbd-chooser code...
> >> For serial console the relevant parts are kbd-chooser.c from line 534 
> >> down. 
> >
> > Maybe you want to commit the following fix for a missing hyphen:
> >
> > Index: packages/kbd-chooser/kbd-chooser.c
> > ===
> > --- packages/kbd-chooser/kbd-chooser.c  (revision 53339)
> > +++ packages/kbd-chooser/kbd-chooser.c  (working copy)
> > @@ -552,7 +552,7 @@
> >  }
> >  
> >  /**
> > - * @brief set debian-installer/serial console as to whether we are using a 
> > serial console
> > + * @brief set debian-installer/serial-console as to whether we are using a 
> > serial console
> >   * @return 1 if present, 0 if absent, 2 if unknown.
> >   */
> >  sercon_state
> >
> >> Both UML and serial console are already being detected there. Seems to me 
> >> that it should be possible to handle Xen in the same or a similar way.
> >
> > Yes.  Line 669 of kbd-chooser.c forces console-tools/archs to
> > no-keyboard, just like during UML installation.  The latter is
> > especially similar to the Xen case, and bases the decision on
> > /proc/cpuinfo.  This suggests that introducing an analogous function
> > which checks for Xen (in the line of [ -d /sys/bus/xen ]) and oring
> > that to the above condition would be enough.
> >
> >> When testing based on hvc0 you should be aware that is also used by 
> >> PowerPC 
> >> systems.
> >
> > This way we wouldn't test for hvc0 at all.  Unfortunately, Xen's hvc
> > isn't a serial device, so that case can't catch it.
> >
> >> Bottom line: please implement something that fits _within_ the existing 
> >> structure of kbd-chooser. Don't add hacks.
> >
> > What about somethink like this?  (Apply before the above!)  Totally
> > untested, POC only. :)
> >
> > Index: packages/kbd-chooser/kbd-chooser.c
> > ===
> > --- packages/kbd-chooser/kbd-chooser.c  (revision 53339)
> > +++ packages/kbd-chooser/kbd-chooser.c  (working copy)
> > @@ -552,6 +552,26 @@
> >  }
> >  
> >  /**
> > + * @brief set debian-installer/xen-console as to whether we are using a 
> > Xen console
> > + * @return 1 if present, 0 if absent, 2 if unknown.
> > + */
> > +sercon_state
> > +check_if_xen_console (void)
> > +{
> > +   sercon_state present = SERIAL_UNKNOWN;
> > +   struct debconfclient *client = mydebconf_client ();
> > +
> > +   if (!access("/sys/bus/xen", F_OK))
> > +   present = SERIAL_PRESENT;
> > +   else
> > +   present = SERIAL_ABSENT;
> > +
> > +   debconf_set (client, "debian-installer/xen-console", present ? "true" : 
> > "false");
> > +   di_info ("Setting debian-installer/xen-console to %s", present ? "true" 
> > : "false");
> > +   return present;
> > +}
> > +
> > +/**
> >   * @brief set debian-installer/serial console as to whether we are using a 
> > serial console
> >   * @return 1 if present, 0 if absent, 2 if unknown.
> >   */
> > @@ -633,6 +653,7 @@
> > int c

Processed: setting package to libdebian-installer-extra4 libdebian-installer4-dev libdebian-installer4-udeb libdebian-installer4 libdebian-installer-extra4-udeb libdebian-installer ...

[Debian Bug Tracking System]

Processing commands for [EMAIL PROTECTED]:

> # Automatically generated email from bts, devscripts version 2.10.28
> #
> # libdebian-installer (0.59) UNRELEASED; urgency=low
> #
> #  * Apply patch to include 'Apple Inc.' as a possible vendor string for a
> #Intel's based Mac. Thanks to Julien BLACHE <[EMAIL PROTECTED]> for
> #the patch (closes: #483182).
> #
> package libdebian-installer-extra4 libdebian-installer4-dev 
> libdebian-installer4-udeb libdebian-installer4 
> libdebian-installer-extra4-udeb libdebian-installer
Ignoring bugs not assigned to: libdebian-installer4-dev 
libdebian-installer-extra4 libdebian-installer4 libdebian-installer4-udeb 
libdebian-installer libdebian-installer-extra4-udeb

> tags 483182 + pending
Bug#483182: [PATCH] libdebian-installer: add new vendor string for Apple
Tags were: patch
Tags added: pending

>
End of message, stopping processing here.

Please contact me if you need assistance.

Debian bug tracking system administrator
(administrator, Debian Bugs database)


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

Bug#483210: tasksel error

[Predrag Gavrilovic]

Package: tasksel
Version 2.74

Using tasksel on already installed system (Debian testing AMD64,
installed 24.5.2008. using daily built buissines card iso, installed
desktop system, installation went OK) gives following error:

"my" variable @deps masks earlier declaration in same scope at
/usr/bin/tasksel line 535.

then tasksel apparently starts, but leaving it or selecting manual
package selection, tasksel exits with:

Use of uninitialized value in string eq at /usr/bin/tasksel line 529.
Use of uninitialized value in string eq at /usr/bin/tasksel line 529.

... and so on 114 times.

To make it clear there were no problems during installation. Both
tasksel and tasksel-data were upgraded on 26.5.2008. from 2.73 to
2.74, so this version was not used during installation.


Best regards,
Predrag Gavrilovic



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

Bug#483182: [PATCH] libdebian-installer: add new vendor string for Apple

[Julien BLACHE]

Package: libdebian-installer4
Version: 0.58
Tags: patch

Hi,

Apple changed its company name in Q1 2007 (from "Apple Computer, Inc."
to "Apple Inc."), and the vendor string embedded on the machines
changed as a consequence on machines produced after this date.

The attached patch adds the new vendor string to the x86 subarch
detection code.

Please apply,

JB.

-- 
 Julien BLACHE <[EMAIL PROTECTED]>  |  Debian, because code matters more 
 Debian & GNU/Linux Developer|   
 Public key available on  - KeyID: F5D6 5169 
 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 

diff -ru orig/libdebian-installer-0.58/src/system/subarch-x86-linux.c libdebian-installer-0.58/src/system/subarch-x86-linux.c
--- orig/libdebian-installer-0.58/src/system/subarch-x86-linux.c	2007-12-31 23:34:31.0 +0100
+++ libdebian-installer-0.58/src/system/subarch-x86-linux.c	2008-05-27 13:55:06.855293130 +0200
@@ -215,6 +215,7 @@
 
 static struct map map_manufacturer[] = {
 	{ "Apple Computer, Inc.", "mac" },
+	{ "Apple Inc.", "mac" },
 	{ NULL, NULL }
 };