Re: [PATCH] dnet: Dave DNET ethernet controller driver

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Neil Horman <nhorman@...>
Cc: David Miller <davem@...>, <shemminger@...>, <s.hauer@...>, <yanok@...>, <linux-arm-kernel@...>, <netdev@...>, <wd@...>, <dzu@...>
Date: Thursday, March 12, 2009 - 12:25 pm

Neil Horman wrote:

In theory, nothing.  In practice, well, where do I find an example 
initrd that I can build on that has a DHCP client that hands over the 
NFS root information from the DHCP response to a tool that mounts the 
NFS root file system?  So far I haven't seen anything like that and the 
infrastructure to build initrds (such as klibc) is still hard to find 
and use.  Every time I've tried to build an initrd that can do DCHP+NFS 
root it has turned out to be too much trouble and I've had to drop it as 
deadlines come closer.


No, I'm quite aware that there are other ways of doing this, it's just 
that all the other ways are _much_ more work for me.  The harsh reality 
is that if I can get a board working using a hack in the driver in half 
an hour, I'll do that rather than fussing around with an initrd. 
Patching the driver is no big deal, it's quick and fairly painless for 
me, it would just be nice if the kernel had a generic way of doing this 
so that I don't have to.  The kernel requires no userspace, no C library 
and all I have to be able to do is to compile the kernel itself.

It's exactly the same reason why people are still using the in kernel 
DHCP client and NFS root code instead of writing and initrd for it, it's 
  much easier to use and why spend time on something that is much more 
complicated for no gain since the in kernel code is sufficient?

I much prefer to spend time on working on the device drivers that I'm 
paid to work on (and spend more time to try try to get those patches 
into the kernel) than to spend time on rewriting everything for an 
initrd.  And the final product usually won't even have ethernet, so from 
a product perspective it would be wasted time.  Anyway, getting the 
system to boot from NFS is usually the absolutely first thing I have to 
do.  Making a product out of it comes months later, and only if we're 
able to do a proof-of-concept demo on a couple of reference platforms at 
an early stage.

Regarding the flash space, if I have a 1.6MByte kernel image (the size 
of a  compressed Linux 2.6.24 kernel for a Samsung S3C24A0 platform I'm 
working on), those 400kBytes can make the difference between things 
fitting into the 2MByte flash partition the bootloader has reserved for 
the kernel or not.

So it mostly boils down to me being lazy and not wanting to spend a lot 
of time on building new infrastructure.  I'd like something which is a 
bit better than what we have now and can spend some time on it, but I 
don't want to spend a lot of time on it since patching the driver works 
just fine for my purposes.  Perfect is the enemy of good.

   /Christer

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] dnet: Dave DNET ethernet controller driver, Ilya Yanok, (Tue Mar 10, 10:29 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Stephen Hemminger, (Wed Mar 11, 12:18 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Michael Cashwell, (Thu Mar 12, 9:41 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, David Miller, (Thu Mar 12, 10:05 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Robert Schwebel, (Fri Mar 13, 3:49 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, David Miller, (Fri Mar 13, 4:12 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Jamie Lokier, (Thu Mar 12, 2:02 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Neil Horman, (Thu Mar 12, 10:26 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Sascha Hauer, (Wed Mar 11, 4:44 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, David Miller, (Wed Mar 11, 4:49 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Christer Weinigel, (Wed Mar 11, 4:56 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Stephen Hemminger, (Wed Mar 11, 8:13 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Christer Weinigel, (Thu Mar 12, 6:41 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, David Miller, (Thu Mar 12, 8:55 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Christer Weinigel, (Thu Mar 12, 10:21 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Neil Horman, (Thu Mar 12, 11:12 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Mike (mwester), (Thu Mar 12, 4:24 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Neil Horman, (Thu Mar 12, 8:29 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Christer Weinigel, (Thu Mar 12, 12:25 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Neil Horman, (Thu Mar 12, 1:49 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Christer Weinigel, (Thu Mar 12, 2:25 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Jamie Lokier, (Thu Mar 12, 3:25 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Christer Weinigel, (Thu Mar 12, 3:43 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Neil Horman, (Thu Mar 12, 3:05 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Jamie Lokier, (Thu Mar 12, 5:59 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Neil Horman, (Thu Mar 12, 8:16 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Christer Weinigel, (Thu Mar 12, 7:42 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Neil Horman, (Thu Mar 12, 8:21 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Neil Horman, (Fri Mar 13, 6:30 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Christer Weinigel, (Thu Mar 12, 3:29 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Sascha Hauer, (Thu Mar 12, 3:14 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, David Miller, (Thu Mar 12, 1:43 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Christer Weinigel, (Thu Mar 12, 1:52 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Jamie Lokier, (Thu Mar 12, 2:06 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Christer Weinigel, (Thu Mar 12, 2:48 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Sascha Hauer, (Thu Mar 12, 3:04 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Wolfgang Denk, (Thu Mar 12, 4:14 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Russell King - ARM Linux, (Thu Mar 12, 4:35 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Sascha Hauer, (Thu Mar 12, 4:27 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Christer Weinigel, (Thu Mar 12, 3:21 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Neil Horman, (Thu Mar 12, 3:16 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Jamie Lokier, (Thu Mar 12, 3:15 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Dan Williams, (Thu Mar 12, 2:39 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, David Miller, (Thu Mar 12, 11:01 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Sascha Hauer, (Thu Mar 12, 10:34 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Russell King - ARM Linux, (Wed Mar 11, 5:33 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Sascha Hauer, (Wed Mar 11, 4:57 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Wolfgang Denk, (Wed Mar 11, 5:09 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, David Miller, (Wed Mar 11, 9:23 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Riku Voipio, (Wed Mar 11, 3:15 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, David Miller, (Wed Mar 11, 8:44 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Robert Schwebel, (Wed Mar 11, 2:23 pm)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Dan Williams, (Wed Mar 11, 10:00 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Sascha Hauer, (Wed Mar 11, 6:35 am)
Re: [PATCH] dnet: Dave DNET ethernet controller driver, Eric Dumazet, (Wed Mar 11, 12:00 am)