All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: linux-ide@vger.kernel.org,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: Multiple functionality breakages in 2.6.12rc3 IDE layer
Date: Fri, 29 Apr 2005 02:23:39 +0200	[thread overview]
Message-ID: <58cb370e05042817232461fb09@mail.gmail.com> (raw)
In-Reply-To: <1114731804.24687.259.camel@localhost.localdomain>

On 4/29/05, Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:
> On Gwe, 2005-04-29 at 00:00, Bartlomiej Zolnierkiewicz wrote:
> > > why. You've disabled open() of a device with no bound driver.
> >
> > Guess what open() for ide-default was doing in 2.6?
> >
> > return -ENXIO;
> >
> > and no it wasn't my change - it was the effect of fixing
> > locking of the higher layers.
> 
> Yes so it needed fixing and without all the kref, kmalloc, unique object
> structure per ide driver code spew too.

IDE is similar to SCSI now in this respect.
Are you claiming that SCSI needs fixing too?

> > > The fact that the IDE layer appears to be getting worse not better,
> > > which given the starting point is a remarkable achievement.
> >
> > Personal insults are easy, get technical facts.
> 
> I consider that a technical fact. The last IDE code I maintained fully
> in 2.4 had mostly working locking, drive hotplug, open for unbound
> drivers, didnt oops on spurious irqs and wasn't losing all sorts of
> useful boot options. I had hoped that I wouldnt have to totally fork the
> 2.6 IDE code in order to get back to where 2.4-ac was and get the
> locking working so you can't oops it via /proc

Somehow you seem to forget that I took maintaince over
2.5.5x (or 2.5.6x) and there was a lot new stuff added when
you were away (i.e. driver-model and IDE code needs to conform
with it to get sane power management and sysfs support)
and that a lot of other things have changed (influencing IDE).

Feel free to fork so you'll be wasting yours time only and not mine.

      reply	other threads:[~2005-04-29  0:23 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-04-28 15:48 Multiple functionality breakages in 2.6.12rc3 IDE layer Alan Cox
2005-04-28 18:13 ` Bill Davidsen
2005-04-28 22:26   ` Alan Cox
2005-04-29 15:34     ` Bill Davidsen
2005-04-28 20:41 ` Bartlomiej Zolnierkiewicz
2005-04-28 22:32   ` Alan Cox
2005-04-28 23:00     ` Bartlomiej Zolnierkiewicz
2005-04-28 23:43       ` Alan Cox
2005-04-29  0:23         ` 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=58cb370e05042817232461fb09@mail.gmail.com \
    --to=bzolnier@gmail.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --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 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.