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: Bjorn Helgaas <bjorn.helgaas@hp.com>,
	B.Zolnierkiewicz@elka.pw.edu.pl, linux-kernel@vger.kernel.org,
	linux-ide@vger.kernel.org, linux-ia64@vger.kernel.org
Subject: Re: [PATCH] IDE: don't offer IDE_GENERIC on ia64
Date: Tue, 16 Aug 2005 20:42:07 +0000	[thread overview]
Message-ID: <58cb370e050816134270b445ea@mail.gmail.com> (raw)
In-Reply-To: <1124223946.22924.4.camel@localhost.localdomain>

On 8/16/05, Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:
> > >   * non-functional HDIO_REGISTER_HWIF ioctl (ain't really working either)
> >
> > HDIO_SCAN_HWIF - I don't know about this one.  How are we supposed to
> > follow the "new ports shouldn't define IDE_ARCH_OBSOLETE_INIT" injunction
> > if we lose all this functionality without it?  ia64 is about as close to
> > a new port as you get :-)
> 
> It allows you to register interfaces from user space.
> 
> As to IDE_ARCH_OBOSOLETE_INIT, its a red herring. It's not worth the
> trouble to avoid it given the expected short remaining life time of the
> old IDE layer. Far better would be to help get the new SATA layer doing
> PATA so drivers/ide can be deleted.

My opinion on this matter differ, I want to:
* finish rewriting IDE code
* deprecate crap
* (later) drop crap and merge rest with libata

I will write precised TODO (w/ status) soon.

IMO this is much better solution as:
* you go from working code into small steps (evolution)
* you don't have to maintain both libata and IDE driver
   for X years (deleting drivers/ide anytime soon is a pipedream)
* as IDE becomes less popular testing gets harder
  (especially when developing new code - you don't
   have previous state)
* it shouldn't be that hard - I have many parts of the stuff
  done (they need some polishing)

Bjorn, thanks for working on this, IDE_ARCH_OBSOLETE_INIT
( -> ide_init_hwif_ports() ) needs to be fixed in order to make REAL
improvements.

Bartlomiej

WARNING: multiple messages have this Message-ID (diff)
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Bjorn Helgaas <bjorn.helgaas@hp.com>,
	B.Zolnierkiewicz@elka.pw.edu.pl, linux-kernel@vger.kernel.org,
	linux-ide@vger.kernel.org, linux-ia64@vger.kernel.org
Subject: Re: [PATCH] IDE: don't offer IDE_GENERIC on ia64
Date: Tue, 16 Aug 2005 22:42:07 +0200	[thread overview]
Message-ID: <58cb370e050816134270b445ea@mail.gmail.com> (raw)
In-Reply-To: <1124223946.22924.4.camel@localhost.localdomain>

On 8/16/05, Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:
> > >   * non-functional HDIO_REGISTER_HWIF ioctl (ain't really working either)
> >
> > HDIO_SCAN_HWIF - I don't know about this one.  How are we supposed to
> > follow the "new ports shouldn't define IDE_ARCH_OBSOLETE_INIT" injunction
> > if we lose all this functionality without it?  ia64 is about as close to
> > a new port as you get :-)
> 
> It allows you to register interfaces from user space.
> 
> As to IDE_ARCH_OBOSOLETE_INIT, its a red herring. It's not worth the
> trouble to avoid it given the expected short remaining life time of the
> old IDE layer. Far better would be to help get the new SATA layer doing
> PATA so drivers/ide can be deleted.

My opinion on this matter differ, I want to:
* finish rewriting IDE code
* deprecate crap
* (later) drop crap and merge rest with libata

I will write precised TODO (w/ status) soon.

IMO this is much better solution as:
* you go from working code into small steps (evolution)
* you don't have to maintain both libata and IDE driver
   for X years (deleting drivers/ide anytime soon is a pipedream)
* as IDE becomes less popular testing gets harder
  (especially when developing new code - you don't
   have previous state)
* it shouldn't be that hard - I have many parts of the stuff
  done (they need some polishing)

Bjorn, thanks for working on this, IDE_ARCH_OBSOLETE_INIT
( -> ide_init_hwif_ports() ) needs to be fixed in order to make REAL
improvements.

Bartlomiej

  reply	other threads:[~2005-08-16 20:42 UTC|newest]

Thread overview: 67+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-08-11 20:24 [PATCH] IDE: don't offer IDE_GENERIC on ia64 Bjorn Helgaas
2005-08-11 20:24 ` Bjorn Helgaas
2005-08-11 20:34 ` Christoph Hellwig
2005-08-11 20:34   ` Christoph Hellwig
2005-08-11 20:37   ` Bjorn Helgaas
2005-08-11 20:37     ` Bjorn Helgaas
2005-08-12 19:41   ` Jan Engelhardt
2005-08-12 19:41     ` Jan Engelhardt
2005-08-16 10:02   ` Bartlomiej Zolnierkiewicz
2005-08-16 10:02     ` Bartlomiej Zolnierkiewicz
2005-08-16 13:42     ` Alan Cox
2005-08-16 14:05       ` Alan Cox
2005-08-11 20:36 ` Jeff Garzik
2005-08-11 20:36   ` Jeff Garzik
2005-08-11 20:45   ` Bjorn Helgaas
2005-08-11 20:45     ` Bjorn Helgaas
2005-08-11 20:56     ` Jeff Garzik
2005-08-11 20:56       ` Jeff Garzik
2005-08-11 21:42       ` Bjorn Helgaas
2005-08-11 21:42         ` Bjorn Helgaas
2005-08-11 21:48         ` Jeff Garzik
2005-08-11 21:48           ` Jeff Garzik
2005-08-11 21:56           ` Jeff Garzik
2005-08-11 21:56             ` Jeff Garzik
2005-08-11 23:07             ` Bjorn Helgaas
2005-08-11 23:07               ` Bjorn Helgaas
2005-08-11 23:49               ` Jeff Garzik
2005-08-11 23:49                 ` Jeff Garzik
2005-08-12  8:35               ` Alan Cox
2005-08-12  8:35                 ` Alan Cox
2005-08-12 10:33               ` Maciej W. Rozycki
2005-08-12 10:33                 ` Maciej W. Rozycki
2005-08-11 23:01         ` Jack Steiner
2005-08-11 23:01           ` Jack Steiner
2005-08-11 21:58 ` Luck, Tony
2005-08-11 21:58   ` Luck, Tony
2005-08-11 21:58   ` Luck, Tony
2005-08-11 22:03   ` Jeff Garzik
2005-08-11 22:03     ` Jeff Garzik
2005-08-11 23:08 ` Jack Steiner
2005-08-11 23:08   ` Jack Steiner
2005-08-12  8:40 ` Alan Cox
2005-08-12  8:40   ` Alan Cox
2005-08-15 21:07   ` Bjorn Helgaas
2005-08-15 21:07     ` Bjorn Helgaas
2005-08-16  9:38     ` Bartlomiej Zolnierkiewicz
2005-08-16  9:38       ` Bartlomiej Zolnierkiewicz
2005-08-16  9:54       ` Bartlomiej Zolnierkiewicz
2005-08-16  9:54         ` Bartlomiej Zolnierkiewicz
2005-08-16 12:55       ` Alan Cox
2005-08-16 12:55         ` Alan Cox
2005-08-16 12:59         ` Bartlomiej Zolnierkiewicz
2005-08-16 12:59           ` Bartlomiej Zolnierkiewicz
2005-08-16 13:40         ` Matthew Wilcox
2005-08-16 13:40           ` Matthew Wilcox
2005-08-16 13:49           ` Bartlomiej Zolnierkiewicz
2005-08-16 13:49             ` Bartlomiej Zolnierkiewicz
2005-08-16 19:16       ` Bjorn Helgaas
2005-08-16 19:16         ` Bjorn Helgaas
2005-08-16 19:59         ` Alan Cox
2005-08-16 20:25           ` Alan Cox
2005-08-16 20:42           ` Bartlomiej Zolnierkiewicz [this message]
2005-08-16 20:42             ` Bartlomiej Zolnierkiewicz
2005-08-16 21:48             ` Alan Cox
2005-08-16 21:48               ` Alan Cox
2005-08-16 20:16         ` Bartlomiej Zolnierkiewicz
2005-08-16 20:16           ` Bartlomiej Zolnierkiewicz

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=58cb370e050816134270b445ea@mail.gmail.com \
    --to=bzolnier@gmail.com \
    --cc=B.Zolnierkiewicz@elka.pw.edu.pl \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=bjorn.helgaas@hp.com \
    --cc=linux-ia64@vger.kernel.org \
    --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.