mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
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

  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®