From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Frans Pop <elendil@planet.nl>,
linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/3] ide: add "ignore_hpa" module parameter
Date: Wed, 27 May 2009 13:41:52 +0200 [thread overview]
Message-ID: <200905271341.52268.bzolnier@gmail.com> (raw)
In-Reply-To: <20090527113857.2a30e9d7@lxorguk.ukuu.org.uk>
On Wednesday 27 May 2009 12:38:57 Alan Cox wrote:
> > All information was there (kernel logs).
> >
> > Moreover the default in libata is to not remove HPA settings.
> >
> > I don't recommend parsing kernels logs in search of such information but
> > you're are stretching the reality too far to match with your arguments.
>
> So you disagree that sysfs is needed and propose an alternative that you
> say you don't recommend (and which doesn't solve the block problem).
This is not what I said (if this is a straw man attempt, please don't do it).
> I used to work for a distro, and telling Bill Nottingham that the
> reliable long term way for Fedora to obtain some interface data was by
> grepping dmesg wouldn't have gone down well.
Knowing the severity of accidentally removing data from HPA (which you
described yourself in the other mail) and also not being able to access
valid data from HPA (bug #13365) I would simply use in the installer
the alternative [*] (that I *don't* recommend as the long-term solution)
and then start working on adding proper sysyfs kernel interfaces (that
I *don't* consider not needed) and proper installer support.
[*] it would of course require passing "libata.ignore_hpa=1" to kernel
during installation time
This would result in much less hassle for users, much less support costs
for distribution and having proper in-kernel / installer support faster
than doing it the non-pragmatic way (since we would save a lot of time on
handling bug-reports and discussions like this one).
prev parent reply other threads:[~2009-05-27 11:41 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-25 21:44 [PATCH 1/3] ide: add "ignore_hpa" module parameter Bartlomiej Zolnierkiewicz
2009-05-25 22:27 ` Frans Pop
2009-05-25 22:36 ` Alan Cox
2009-05-25 23:06 ` Frans Pop
2009-05-26 12:23 ` Bartlomiej Zolnierkiewicz
2009-05-26 12:53 ` Frans Pop
2009-05-26 13:01 ` Alan Cox
2009-05-26 18:08 ` Andries E. Brouwer
2009-05-27 9:49 ` Alan Cox
2009-05-26 17:32 ` Bartlomiej Zolnierkiewicz
2009-05-26 23:49 ` Robert Hancock
2009-05-26 12:53 ` Alan Cox
2009-05-26 17:15 ` Bartlomiej Zolnierkiewicz
2009-05-27 10:38 ` Alan Cox
2009-05-27 11:41 ` Bartlomiej Zolnierkiewicz [this message]
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=200905271341.52268.bzolnier@gmail.com \
--to=bzolnier@gmail.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=elendil@planet.nl \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.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;
as well as URLs for NNTP newsgroup(s).