From: ebiederm@xmission.com (Eric W. Biederman)
To: "Miller, Mike (OS Dev)" <Mike.Miller@hp.com>
Cc: vgoyal@in.ibm.com, Maneesh Soni <maneesh@in.ibm.com>,
Andrew Morton <akpm@osdl.org>,
Neela.Kolli@engenio.com, linux-scsi@vger.kernel.org,
fastboot@lists.osdl.org, linux-kernel@vger.kernel.org
Subject: Re: [Fastboot] [RFC] [PATCH 2/2] kdump: cciss driver initialization issue fix
Date: Mon, 26 Jun 2006 11:52:28 -0600 [thread overview]
Message-ID: <m1veqnst2b.fsf@ebiederm.dsl.xmission.com> (raw)
In-Reply-To: <E717642AF17E744CA95C070CA815AE550E1555@cceexc23.americas.cpqcorp.net> (Mike Miller's message of "Mon, 26 Jun 2006 11:51:52 -0500")
"Miller, Mike (OS Dev)" <Mike.Miller@hp.com> writes:
> Thanks Eric, that helps me understand. Section 8.2.2 of the open cciss
> spec supports a reset message. Target 0x00 is the controller. We could
> add this to the init routine to ensure the board is made sane again but
> this would drastically increase init time under normal circumstances.
Where does the init time penalty come from? How large is the
init penalty? I suspect it is from waiting for the scsi disks to spin up.
But I am just guessing in the dark.
> And I suspect this is a hard reset, also. Not sure if that would
> negatively impact kdump. If there were some condition we could test
> against and perform the reset when that condition is met it would not
> impact 99.9% of users.
I am wondering if it is possible to look at the controller and
see if it is in a bad state, (i.e. in some state besides just coming
out of reset) and if so issue a reset. If this really is a long operation
that would be the ideal way to handle it.
If the amount of time is really user noticeable and testing for it
is impossible then it is probably time to talk kernel command line
options.
Although it might simply be appropriate to handle commands completing
you didn't start. I am not at all familiar with that particular piece
of hardware so I can't make a good guess on what needs to happen there.
> Thoughts, comments, flames?
Good question.
It is a bit of a pain but not too hard to setup a test environment
so you can reproduce this if you are interested. Vivek should
be the authority there.
Eric
next prev parent reply other threads:[~2006-06-26 17:52 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-06-23 21:01 [RFC] [PATCH 1/2] introduce crashboot kernel command line parameter Vivek Goyal
2006-06-23 21:04 ` [RFC] [PATCH 2/2] kdump: cciss driver initialization issue fix Vivek Goyal
2006-06-24 6:55 ` Andrew Morton
2006-06-24 11:19 ` Vivek Goyal
2006-06-24 11:30 ` Andrew Morton
2006-06-24 12:08 ` Vivek Goyal
2006-06-24 17:13 ` Eric W. Biederman
2006-06-26 2:11 ` [Fastboot] " Maneesh Soni
2006-06-26 13:35 ` Vivek Goyal
2006-06-26 14:17 ` Eric W. Biederman
2006-06-26 15:32 ` [Fastboot] " Vivek Goyal
2006-06-26 16:00 ` Eric W. Biederman
2006-06-26 16:13 ` Miller, Mike (OS Dev)
2006-06-26 16:35 ` Vivek Goyal
2006-06-26 16:38 ` [Fastboot] " Eric W. Biederman
2006-06-26 16:51 ` Miller, Mike (OS Dev)
2006-06-26 17:04 ` Vivek Goyal
2006-06-26 17:24 ` Andrew Morton
2006-06-26 17:22 ` [Fastboot] " Vivek Goyal
2006-06-26 17:52 ` Eric W. Biederman [this message]
2006-06-26 18:18 ` Vivek Goyal
2006-06-26 18:51 ` [Fastboot] " Miller, Mike (OS Dev)
2006-06-26 19:21 ` Eric W. Biederman
2006-06-26 19:43 ` Vivek Goyal
2006-06-26 21:24 ` [Fastboot] " Miller, Mike (OS Dev)
2006-06-26 19:36 ` Vivek Goyal
2006-06-26 17:16 ` Vivek Goyal
2006-06-26 17:31 ` Andrew Morton
2006-06-26 17:39 ` Eric W. Biederman
2006-06-26 17:56 ` James Bottomley
2006-06-26 18:23 ` Eric W. Biederman
2006-06-27 2:42 ` [RFC] [PATCH 2/2] kdump: cciss driver initialization?issue fix Horms
2006-06-26 9:09 ` Horms
2006-06-26 13:45 ` [Fastboot] " Vivek Goyal
2006-06-27 2:30 ` Horms
[not found] ` <E1FttEj-0002JH-00@calista.eckenfels.net>
2006-06-23 22:39 ` [RFC] [PATCH 1/2] introduce crashboot kernel command line parameter Vivek Goyal
2006-06-24 6:55 ` Andrew Morton
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=m1veqnst2b.fsf@ebiederm.dsl.xmission.com \
--to=ebiederm@xmission.com \
--cc=Mike.Miller@hp.com \
--cc=Neela.Kolli@engenio.com \
--cc=akpm@osdl.org \
--cc=fastboot@lists.osdl.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=maneesh@in.ibm.com \
--cc=vgoyal@in.ibm.com \
/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