[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.44L0.1205251034190.1907-100000@iolanthe.rowland.org>
Date: Fri, 25 May 2012 10:38:29 -0400 (EDT)
From: Alan Stern <stern@...land.harvard.edu>
To: James Bottomley <jejbbe@...senpartnership.com>
cc: "Rafael J. Wysocki" <rjw@...k.pl>,
Dan Williams <dan.j.williams@...el.com>,
David Miller <davem@...emloft.net>, <mroos@...ux.ee>,
<linux-kernel@...r.kernel.org>, <linux-scsi@...r.kernel.org>,
Arjan van de Ven <arjan@...radead.org>
Subject: Re: 3.4.0-02580-g72c04af regression on sparc64 - partitions not
recognized
On Fri, 25 May 2012, James Bottomley wrote:
> > > This is actually looks wrong: it works if SCSI is built in, but it's a
> > > nop if SCSI is a module (the nop function is gated by the else clause of
> > > #ifdef CONFIG_SCSI)
> > >
> > > Rafael, you added this not via the SCSI tree,
> >
> > That's correct, it was committed directly by Linus.
> >
> > > is that the intention?
> >
> > Pretty much it is.
> >
> > The code snippet is slightly out of context and it is a part of the
> > software_resume() routine, which is only called when the kernel's built-in
> > image reading code checks whether or not the image is present. It won't
> > work anyway if SCSI is not built in.
>
> I don't understand this.
>
> Why would it make a difference whether SCSI is modular at hybernation
> resume time? The reason it makes a difference at boot time is because
> there's no initrd to wait for the scans and mount the root if we're not
> modular, so the init path has to do it. However, when resuming an
> image, the module is already loaded into that image, so there should be
> no difference at all between steps taken in the modular and non-modular
> cases.
I think Rafael is referring to the boot kernel -- the one that reads in
the hibernation image initially. Whether or not the boot kernel can
have modular drivers isn't clear to me; I don't know when the check for
a valid image in the swap area is carried out in relation to starting
up the initramfs task.
Alan Stern
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists