Bug#416063: tagging 416063

[Mike Hommey <mh@glandium.org>]

On Mon, Mar 30, 2009 at 01:11:52AM +0200, maximilian attems wrote:
> On Sat, 28 Mar 2009, Mike Hommey wrote:
> 
> > On Wed, Mar 28, 2007 at 12:19:49AM +0200, maximilian attems wrote:
> > > # Automatically generated email from bts, devscripts version 2.9.26
> > > tags 416063 - patch
> > 
> > Would you mind explaining?
> > 
> > The patch solves two issues:
> > - inconsistency between documentation and behaviour:
> >      You can read the following in /etc/initramfs-tools/initramfs.conf:
> >      # most - Add all framebuffer, acpi, filesystem, and harddrive drivers.
> >      As it happens, most does *not* add all framebuffer drivers
> 
> yeah none at all, fixed in latest git.
> the patch was not ok as we don't want to have another hardcoded list
> to maintain, just added all the fb modules on one go, see
> 
> f40e826287d2776094f48302c196652f2ba8326b
>   hook-functions: MODULES=most add all fb modules per default
> http://git.debian.org/?p=kernel/initramfs-tools.git;a=summary
> 
> > - workaround the lack of dependency on intel_agp for intelfb.
> >      Without the workaround, you can't have intel_agp loaded before
> >      intelfb without *not* using video=intelfb syntax, thus losing
> >      /dev/fb device creation.
> 
> that looks more like a kernel bug,
> also intelfb should just work fine if not on agp.
> or are you saying there are missing symbols there?

This is what happens in intelfb_pci_register, unconditionally:

        /* Use agpgart to manage the GATT */
        if (!(bridge = agp_backend_acquire(pdev))) {
                ERR_MSG("cannot acquire agp\n");
                cleanup(dinfo);
                return -ENODEV;
        }

Mike