From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartlomiej Zolnierkiewicz Subject: Re: Multiple functionality breakages in 2.6.12rc3 IDE layer Date: Fri, 29 Apr 2005 01:00:06 +0200 Message-ID: <58cb370e05042816003c2ca4be@mail.gmail.com> References: <1114703284.18809.208.camel@localhost.localdomain> <58cb370e05042813414af5bc1e@mail.gmail.com> <1114727522.18355.242.camel@localhost.localdomain> Reply-To: Bartlomiej Zolnierkiewicz Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7BIT Return-path: Received: from wproxy.gmail.com ([64.233.184.193]:62680 "EHLO wproxy.gmail.com") by vger.kernel.org with ESMTP id S262326AbVD1XAH convert rfc822-to-8bit (ORCPT ); Thu, 28 Apr 2005 19:00:07 -0400 Received: by wproxy.gmail.com with SMTP id 68so915391wra for ; Thu, 28 Apr 2005 16:00:06 -0700 (PDT) In-Reply-To: <1114727522.18355.242.camel@localhost.localdomain> Content-Disposition: inline Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Alan Cox Cc: linux-ide@vger.kernel.org, Linux Kernel Mailing List On 4/29/05, Alan Cox wrote: > On Iau, 2005-04-28 at 21:41, Bartlomiej Zolnierkiewicz wrote: > > On 4/28/05, Alan Cox wrote: > > > Ages ago we added an ide_default driver to clean up all the corner cases > > > > s/clean up/hide/ > > Matter of opinion. No, it hided holes in the locking which is getting fixed finally. > > > for most users. You can no longer > > > - Control the bus state of an interface > > > - Reset an interface > > > - Add an interface if none exist > > > - Issue raw commands > > > - Get an objects bios geometry > > > - Read the identify data by ioctl (its still in proc but may be stale) > > > > Details please. > > If you need details you shouldn't be maintaining that code. Its obvious Give details or quit whining. > 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. > > No functionality was removed AFAIK, see the patches. I spend quite > > a bit of time making sure that nothing breaks up (I missed one special > > case but somebody already posted patch to LKML fixing it). > > Build a kernel without ide-cd. Now try and issue ioctls on it. Doesn't > work any more does it. > > > > > These patches were posted at least two times to both linux-ide and > > linux-kernel, they were in -mm for ages - were you hiding under the > > rock? > > No, just doing an MBA thesis, a job, learning a second language and > trying to beat sense into our politicians. Now I come back to look at It seems that they influenced you heavily... > the ide layer ready for a 2.6.12 merge and its all a bit messy. The open > code was clean and is now duplicated. Copies of subtly different per You must be joking. > driver gendisk/disk layer open routines have appeared that should be Each change was given rationale and detailed changelog, maybe you should get familiar with them. Also look for patch converting device drivers to sysfs (posted few times). > shared. The default driver handling has been removed and half the > options for obscure systems have been marked obsolete in some Gnome like > purge of functionality that might scare small children. > > > > The ability to specify the IDE ports on the command line as needed for > > > some Sony laptop installs have also become "obsolete" over time. They > > > still appear to work but spew a warning that the user will soon be > > > screwed. > > > > This was discussed few times already. > > And the discussion lead to no fixes So fix the real bugs and leave debugging stuff alone. > > Alan, seriously, what is your problem? > > 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. Bartlomiej