linux-kernel - Re: [PATCH] [MTD] driver/Makefile: Initialize "mtd" and "spi" before "net"

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>] [day] [month] [year] [list]

Message-Id: <20090818024908.90f528e1.akpm@linux-foundation.org>
Date:	Tue, 18 Aug 2009 02:49:08 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	"Sudhakar Rajashekhara" <sudhakar.raj@...com>
Cc:	<linux-mtd@...ts.infradead.org>,
	<davinci-linux-open-source@...ux.davincidsp.com>,
	<david-b@...bell.net>, <dwmw2@...radead.org>,
	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] [MTD] driver/Makefile: Initialize "mtd" and "spi"
 before "net"

On Tue, 18 Aug 2009 15:06:35 +0530 "Sudhakar Rajashekhara" <sudhakar.raj@...com> wrote:

> On Tue, Aug 18, 2009 at 13:49:30, Andrew Morton wrote:
> > On Tue, 18 Aug 2009 12:34:04 -0400 Sudhakar Rajashekhara <sudhakar.raj@...com> wrote:
> > 
> > > On TI's da850/omap-l138 EVM, MAC address is stored in SPI flash.
> > > 
> > > This patch changes the initialization sequence of the drivers
> > > by moving mtd and spi ahead of net in drivers/Makefile thereby
> > > enabling da850/omap-l138 ethernet driver to read the MAC address
> > > while booting.
> > > 
> > > Signed-off-by: Sudhakar Rajashekhara <sudhakar.raj@...com>
> > > ---
> > >  drivers/Makefile |    4 ++--
> > >  1 files changed, 2 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/drivers/Makefile b/drivers/Makefile
> > > index bc4205d..2a1d41f 100644
> > > --- a/drivers/Makefile
> > > +++ b/drivers/Makefile
> > > @@ -42,6 +42,8 @@ obj-y				+= macintosh/
> > >  obj-$(CONFIG_IDE)		+= ide/
> > >  obj-$(CONFIG_SCSI)		+= scsi/
> > >  obj-$(CONFIG_ATA)		+= ata/
> > > +obj-$(CONFIG_MTD)		+= mtd/
> > > +obj-$(CONFIG_SPI)		+= spi/
> > >  obj-y				+= net/
> > >  obj-$(CONFIG_ATM)		+= atm/
> > >  obj-$(CONFIG_FUSION)		+= message/
> > > @@ -50,8 +52,6 @@ obj-y				+= ieee1394/
> > >  obj-$(CONFIG_UIO)		+= uio/
> > >  obj-y				+= cdrom/
> > >  obj-y				+= auxdisplay/
> > > -obj-$(CONFIG_MTD)		+= mtd/
> > > -obj-$(CONFIG_SPI)		+= spi/
> > >  obj-$(CONFIG_PCCARD)		+= pcmcia/
> > >  obj-$(CONFIG_DIO)		+= dio/
> > >  obj-$(CONFIG_SBUS)		+= sbus/
> > 
> > That isn't a particularly maintainable way of fixing this, especially
> > as there are no comments in that Makefile to prevent someone from
> > rebreaking it in the future.
> > 
> > A better fix would be to use suitably prioritised initcalls - see
> > include/linux/init.h around line 187.
> > 
> 
> Currently mtd, spi and net subsystems are initialized with module_init.
> If I change the way in which these sub-systems are initialized, then
> I'll be breaking the support for these sub-systems to work as modules.

In what way?

> There are some comments in drivers/Makefile which say why particular
> sub-systems are ahead of others like:
> 
> 	...
> 	...
> 	# PnP must come after ACPI since it will eventually need to check if acpi
> 	# was used and do nothing if so
> 	...
> 	...
> 	# char/ comes before serial/ etc so that the VT console is the boot-time
> 	# default.
> 	...
> 

It would be nice to fix those as well ;)
--
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/