From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: James Bottomley <jejbbe@hansenpartnership.com>,
Dan Williams <dan.j.williams@intel.com>,
David Miller <davem@davemloft.net>,
mroos@linux.ee, linux-kernel@vger.kernel.org,
linux-scsi@vger.kernel.org,
Arjan van de Ven <arjan@infradead.org>
Subject: Re: 3.4.0-02580-g72c04af regression on sparc64 - partitions not recognized
Date: Fri, 25 May 2012 21:27:21 +0200 [thread overview]
Message-ID: <201205252127.21436.rjw@sisk.pl> (raw)
In-Reply-To: <Pine.LNX.4.44L0.1205251034190.1907-100000@iolanthe.rowland.org>
On Friday, May 25, 2012, Alan Stern wrote:
> 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.
That's correct.
> Whether or not the boot kernel can have modular drivers isn't clear to me;
It can, but then the code in question is irrelevant and the resume has to
be triggered in a different way. For example, by writing to
/sys/power/resume from the initrd boot scripts.
> 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.
This particular one is carried out before the initrd is loaded.
Thanks,
Rafael
next prev parent reply other threads:[~2012-05-25 19:22 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-22 9:14 Meelis Roos
2012-05-22 19:12 ` David Miller
2012-05-22 19:17 ` Meelis Roos
2012-05-22 19:22 ` David Miller
2012-05-23 16:46 ` Meelis Roos
2012-05-23 18:04 ` David Miller
2012-05-23 21:00 ` Alan Stern
2012-05-23 22:56 ` James Bottomley
2012-05-24 2:22 ` Dan Williams
2012-05-24 7:42 ` James Bottomley
2012-05-24 7:54 ` Dan Williams
2012-05-24 8:05 ` James Bottomley
2012-05-24 19:52 ` Rafael J. Wysocki
2012-05-25 7:45 ` James Bottomley
2012-05-25 14:38 ` Alan Stern
2012-05-25 19:27 ` Rafael J. Wysocki [this message]
2012-05-28 16:46 ` Eldad Zack
2012-05-28 17:56 ` Dan Williams
2012-05-28 20:25 ` Eldad Zack
2012-05-25 7:55 ` mroos
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201205252127.21436.rjw@sisk.pl \
--to=rjw@sisk.pl \
--cc=arjan@infradead.org \
--cc=dan.j.williams@intel.com \
--cc=davem@davemloft.net \
--cc=jejbbe@hansenpartnership.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=mroos@linux.ee \
--cc=stern@rowland.harvard.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®