From: Dan Williams <dan.j.williams@gmail.com>
To: Tejun Heo <tj@kernel.org>
Cc: Jeff Garzik <jeff@garzik.org>,
linux-ide@vger.kernel.org, jens.axboe@oracle.com,
linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org,
James.Bottomley@hansenpartnership.com, Mauelshagen@redhat.com,
dm-devel@redhat.com
Subject: Re: [PATCHSET] block,scsi,libata: implement alt_size
Date: Mon, 4 May 2009 09:48:37 -0700 [thread overview]
Message-ID: <e9c3a7c20905040948m572e23dj1aa197dbd20ff037@mail.gmail.com> (raw)
In-Reply-To: <49FEA127.1000905@kernel.org>
On Mon, May 4, 2009 at 1:02 AM, Tejun Heo <tj@kernel.org> wrote:
>> Couldn't the "fix" also just be a note to users to disable ignore_hpa
>> if they notice that there raid arrays are not assembling correctly?
>
> I don't know. If it doesn't work automatically, the solution really
> isn't worth much. People generally don't (shouldn't need to) have any
> idea what HPA is. It basically boils down to "Linux doesn't support
> RAID". :-)
>
> Maybe we should just put this issue to the rest and strongly advise
> people against BIOS raids.
That's a bit too far :-). The real issue is that Linux is ignoring a
property of the platform. To me ignoring HPA is like ignoring an e820
reserved memory region. You might be able to get away with it
sometimes, but other times it breaks.
However, in the end I think having 'alt_size' available is a useful
mechanism for discovering and handling the ignore_hpa=1 case. The
policy about when to trust it can be left to userspace.
Thanks,
Dan
next prev parent reply other threads:[~2009-05-04 16:48 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-01 2:55 Tejun Heo
2009-02-01 2:55 ` [PATCH 1/3] block: add alt_size Tejun Heo
2009-02-01 2:55 ` [PATCH 2/3] scsi: add scsi_device->alt_capacity Tejun Heo
2009-02-01 2:55 ` [PATCH 3/3] libata: export HPA size as alt_size Tejun Heo
2009-02-02 17:12 ` [PATCH] libata: change drive ready wait after hard reset to 5s Stuart_Hayes
2009-02-02 17:17 ` Mario Limonciello
2009-02-09 21:48 ` Stuart_Hayes
2009-02-01 2:59 ` [PATCHSET] dmraid: use alt_size Tejun Heo
2009-02-01 3:00 ` [PATCH 1/3] dmraid: set read_info to @offset by default in read_raid_dev() Tejun Heo
2009-02-01 3:00 ` [PATCH 2/3] dmraid: add alt_size to dev_info Tejun Heo
2009-02-01 3:01 ` [PATCH 3/3] dmraid: make nv use alt_size if available Tejun Heo
2009-02-01 3:20 ` [PATCHSET] block,scsi,libata: implement alt_size Jeff Garzik
2009-02-01 4:14 ` Tejun Heo
2009-04-30 1:13 ` Tejun Heo
2009-04-30 1:45 ` Jeff Garzik
2009-04-30 1:50 ` Tejun Heo
2009-04-30 20:00 ` Jeff Garzik
2009-04-30 21:15 ` Dan Williams
2009-05-04 8:02 ` Tejun Heo
2009-05-04 16:48 ` Dan Williams [this message]
2009-05-05 3:28 ` Tejun Heo
2009-05-05 15:34 ` Dan Williams
2009-05-08 9:13 ` Tejun Heo
2009-05-08 16:21 ` Jeff Garzik
2009-05-08 16:23 ` Dan Williams
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=e9c3a7c20905040948m572e23dj1aa197dbd20ff037@mail.gmail.com \
--to=dan.j.williams@gmail.com \
--cc=James.Bottomley@hansenpartnership.com \
--cc=Mauelshagen@redhat.com \
--cc=dm-devel@redhat.com \
--cc=jeff@garzik.org \
--cc=jens.axboe@oracle.com \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=tj@kernel.org \
/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®