All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl>
To: Andries Brouwer <Andries.Brouwer@cwi.nl>
Cc: Andries Brouwer <aebr@win.tue.nl>,
	Szakacsits Szabolcs <szaka@sienet.hu>,
	Andries Brouwer <Andries.Brouwer@cwi.nl>,
	"Patrick J. LoPresti" <patl@users.sourceforge.net>,
	bug-parted@gnu.org, Steffen Winterfeldt <snwint@suse.de>,
	Thomas Fehr <fehr@suse.de>,
	linux-kernel@vger.kernel.org, Andrew Clausen <clausen@gnu.org>,
	buytenh@gnu.org, msw@redhat.com
Subject: Re: Restoring HDIO_GETGEO semantics for 2.6 (was: Re: [RFC] Restoring HDIO_GETGEO semantics)
Date: Mon, 5 Jul 2004 23:52:51 +0200	[thread overview]
Message-ID: <200407052352.51135.bzolnier@elka.pw.edu.pl> (raw)
In-Reply-To: <20040705210813.GE29504@apps.cwi.nl>

On Monday 05 of July 2004 23:08, Andries Brouwer wrote:
> On Mon, Jul 05, 2004 at 09:05:05PM +0200, Bartlomiej Zolnierkiewicz wrote:
> > Andries, the question was "What should we do with HDIO_GETGEO breakage?"
> > not "Why does somebody need the BIOS geometry?". :-)
> >
> > We can fix HDIO_GETGEO to behave like in 2.4 or remove it (preferable),
> > current situation is bad.
>
> I don't know precisely why.
>
> Neither of your two proposed actions appeals to me.
>
> Here is an ioctl, and it is used for legitimate purposes
> (finding the starting offset of a partition).

It is also _abused_ for less legitimate purposes.

> You cannot remove it. We can think again in 2.7.
> For now, leave the kernel interface constant.

We can at least consider adding BLKPARTSTART and deprecating
HDIO_GETGEO so people at least will know that they should upgrade.

> Is there any advantage in going back? I don't think so.
> The old situation was broken. People lost all data.

According to szaka the "the new situation" is similar. :(

> Also "the old situation" is badly defined. The returned value differs
> for 2.0, 2.2, 2.4, 2.6.

This is just sick: same ioctl - different returned values.
Please never ever do it again - we can avoid such problems in future.

> No. We must go forward.

Yep, 2.7.1 should remove HDIO_GETGEO.
We should have done this in 2.5, really.

> Now distributions can take care of themselves. They can patch the kernel
> as they like, or patch parted as they like, or do any number of other
> things. RedHat and SuSE can take their own decisions. With some luck there
> is a new parted next week or so that they could offer.

What about people running old parted and new vanilla kernels?

> So we can go slowly and quietly, investigate precisely what happens, and
> why and how such things can be fixed. I think I understand rather well what
> is (was) wrong with parted. Maybe Szaka can teach me about other tools that
> are broken. I am confident that we can fix them, maybe in hours rather than
> days.

What about people using old versions of these tools (if any).

What I'm only worried is that there might be cases when 2.4 worked
and 2.6 doesn't which with combination with bugs in the tools cause
"where is my data?" issue.

> As a side result we will have something valuable, namely standard software
> that tries to handle BIOS data. Several orders of magnitude more reliable
> than our old guesses.

Yep.

Bartlomiej


  reply	other threads:[~2004-07-05 21:47 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <s5gwu1mwpus.fsf@patl=users.sf.net>
2004-07-02 16:17 ` [RFC] Restoring HDIO_GETGEO semantics (was: Re: workaround for BIOS / CHS stuff) Szakacsits Szabolcs
2004-07-02 16:50   ` Andries Brouwer
2004-07-02 18:28     ` dwm
2004-07-02 21:12       ` parted maintainership Andries Brouwer
2004-07-02 17:04   ` [RFC] Restoring HDIO_GETGEO semantics (was: Re: workaround for BIOS / CHS stuff) Andries Brouwer
2004-07-02 18:12     ` Szakacsits Szabolcs
2004-07-02 18:45       ` Patrick J. LoPresti
     [not found]         ` <Pine.LNX.4.60.0407022025200.28638@hermes-1.csi.cam.ac.uk>
2004-07-02 19:57           ` Patrick J. LoPresti
2004-07-03  0:17             ` Szakacsits Szabolcs
2004-07-03  0:42               ` Bartlomiej Zolnierkiewicz
2004-07-03  0:56                 ` Andries Brouwer
2004-07-03  1:57                   ` Szakacsits Szabolcs
2004-07-03 13:59                   ` Patrick J. LoPresti
2004-07-05 12:14                   ` Restoring HDIO_GETGEO semantics for 2.6 (was: Re: [RFC] Restoring HDIO_GETGEO semantics) Szakacsits Szabolcs
2004-07-05 13:10                     ` Steffen Winterfeldt
2004-07-05 13:12                     ` Andries Brouwer
2004-07-05 13:13                     ` Bartlomiej Zolnierkiewicz
2004-07-05 14:00                       ` Andries Brouwer
2004-07-05 19:05                         ` Bartlomiej Zolnierkiewicz
2004-07-05 21:08                           ` Andries Brouwer
2004-07-05 21:52                             ` Bartlomiej Zolnierkiewicz [this message]
2004-07-06  0:17                               ` Szakacsits Szabolcs
2004-07-06  1:56                                 ` Andries Brouwer
2004-07-06 18:56                                   ` Szakacsits Szabolcs
2004-07-07  1:28                                     ` Andries Brouwer
2004-07-07 11:14                                       ` Roman Zippel
2004-07-07 11:51                                         ` Szakacsits Szabolcs
2004-07-06  8:33                                 ` Steffen Winterfeldt
2004-07-05 18:09                       ` Szakacsits Szabolcs
2004-07-05 18:58                         ` Bartlomiej Zolnierkiewicz
2004-07-03  3:00             ` [RFC] Restoring HDIO_GETGEO semantics (was: Re: workaround for BIOS / CHS stuff) Andrew Clausen
2004-07-02 23:55         ` Szakacsits Szabolcs
2004-07-03 13:56           ` Patrick J. LoPresti
2004-07-03  2:54         ` Andrew Clausen
     [not found]           ` <Pine.LNX.4.60.0407030843400.2415@hermes-1.csi.cam.ac.uk>
2004-07-03 12:44             ` Andrew Clausen
     [not found]               ` <Pine.LNX.4.60.0407031535230.6149@hermes-1.csi.cam.ac.uk>
2004-07-03 15:02                 ` Andrew Clausen
2004-07-03 14:42           ` Patrick J. LoPresti
2004-07-03  1:35   ` Andrew Clausen
2004-07-03 12:33     ` Andries Brouwer
2004-07-03 14:15     ` Patrick J. LoPresti
2004-07-03 14:45       ` Andrew Clausen
2004-07-03 15:00         ` Patrick J. LoPresti
2004-07-03 20:12         ` Andries Brouwer

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=200407052352.51135.bzolnier@elka.pw.edu.pl \
    --to=b.zolnierkiewicz@elka.pw.edu.pl \
    --cc=Andries.Brouwer@cwi.nl \
    --cc=aebr@win.tue.nl \
    --cc=bug-parted@gnu.org \
    --cc=buytenh@gnu.org \
    --cc=clausen@gnu.org \
    --cc=fehr@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=msw@redhat.com \
    --cc=patl@users.sourceforge.net \
    --cc=snwint@suse.de \
    --cc=szaka@sienet.hu \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.