linux-kernel - Re: [2.6 patch] ARM: always select HAVE

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <200804272032.44273.bzolnier@gmail.com>
Date:	Sun, 27 Apr 2008 20:32:44 +0200
From:	Bartlomiej Zolnierkiewicz <bzolnier@...il.com>
To:	Adrian Bunk <bunk@...nel.org>
Cc:	Sam Ravnborg <sam@...nborg.org>, linux-ide@...r.kernel.org,
	linux-kernel@...r.kernel.org, linux-pcmcia@...ts.infradead.org
Subject: Re: [2.6 patch] ARM: always select HAVE_IDE

On Thursday 17 April 2008, Adrian Bunk wrote:
> On Thu, Apr 17, 2008 at 12:00:05PM +0100, Russell King wrote:
> > On Thu, Apr 17, 2008 at 01:48:36PM +0300, Adrian Bunk wrote:
> > > On Thu, Apr 17, 2008 at 10:59:16AM +0100, Russell King wrote:
> > > > On Thu, Apr 17, 2008 at 12:37:53PM +0300, Adrian Bunk wrote:
> > > > > Any objections against the patch below?
> > > > 
> > > > Let's look at the rest of the situation surrounding HAVE_IDE first.
> > > > It's something of a mess:
> > > > 
> > > > avr32, m68knommu, ppc, s390 and v850 do not have asm/ide.h
> > > > 
> > > > avr32, m68knommu, ppc, v850 all set HAVE_IDE, arch/s390/Kconfig contains
> > > > no reference.
> > > > 
> > > > avr32 supports PATA (which is IDE).
> > > > 
> > > > Everything else provides an asm/ide.h and sets HAVE_IDE.
> > > > 
> > > > So:
> > > > 
> > > > Q1. Do avr32, m68knommu, ppc, v850 have IDE support or do they not?
> > > 
> > > avr32 is fixed in 2.6.25 (no more HAVE_IDE)
> > 
> > avr32 has ATA, so the only reason it doesn't actually use IDE is because
> > they're using libata entirely.  This is NO different from the situation
> > on ARM - some machine classes use entirely libata, others use IDE, and
> > some others are trying to give up IDE in favour of libata.
> 
> You have a point that one might argue that avr32 should also select 
> HAVE_IDE and get an asm/ide.h .
> 
> The main difference between avr32 and arm is that on arm there are a 
> bunch of platforms that actually want to use drivers/ide/ at the moment, 
> and the fine-grained select's we have at the moment don't bring any real 
> gain.
> 
> I do actually not care much how this gets resolved (we could even 
> ditch HAVE_IDE and provide asm/ide.h on all architectures) if there's 
> general agreement that this is the way to go.

Seems like a way to go now that all crazy IDE core code vs arch code
inter-dependiences have been fixed (the ideal solution would be to ditch
asm/ide.h completely but this requires some more time/work).

Thanks,
Bart
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/