Re: [patch 0/4] [resend] Add configuration options to disable features

2008-08-01 Thread David Woodhouse
On Fri, 2008-08-01 at 12:15 -0700, Linus Torvalds wrote: > > On Thu, 31 Jul 2008, Josh Boyer wrote: > > On Thu, 2008-07-31 at 20:50 +0200, Ulrich Teichert wrote: > > > > > > I do not think of NTP as desktop or server application, but that's > > > probably just me, > > > > No, it's not just you.

Re: [patch 0/4] [resend] Add configuration options to disable features

2008-08-01 Thread Linus Torvalds
On Thu, 31 Jul 2008, Josh Boyer wrote: > On Thu, 2008-07-31 at 20:50 +0200, Ulrich Teichert wrote: > > > > I do not think of NTP as desktop or server application, but that's > > probably just me, > > No, it's not just you. NTP is useful in cases where things do care > about time but hardware d

Re: [patch 0/4] [resend] Add configuration options to disable features

2008-08-01 Thread Robert Schwebel
On Thu, Jul 31, 2008 at 03:46:28PM -0400, Josh Boyer wrote: > > I do not think of NTP as desktop or server application, but that's > > probably just me, > > No, it's not just you. NTP is useful in cases where things do care > about time but hardware designers were too cheap to put an RTC on the > b

Re: [patch 0/4] [resend] Add configuration options to disable features

2008-07-31 Thread David Woodhouse
On Thu, 2008-07-31 at 15:46 -0400, Josh Boyer wrote: > On Thu, 2008-07-31 at 20:50 +0200, Ulrich Teichert wrote: > > Hi, > > > > >I don't know of any embedded products that ship with NTP turned > > >on. > > > > Well, I do. To be exact, I've developed parts of it. But it's numbers > > are only int

Re: [patch 0/4] [resend] Add configuration options to disable features

2008-07-31 Thread Josh Boyer
On Thu, 2008-07-31 at 20:50 +0200, Ulrich Teichert wrote: > Hi, > > >I don't know of any embedded products that ship with NTP turned > >on. > > Well, I do. To be exact, I've developed parts of it. But it's numbers > are only into the thousands, so that makes it insignificant ;-) > > >It's best t

Re: [patch 0/4] [resend] Add configuration options to disable features

2008-07-31 Thread Ulrich Teichert
Hi, >I don't know of any embedded products that ship with NTP turned >on. Well, I do. To be exact, I've developed parts of it. But it's numbers are only into the thousands, so that makes it insignificant ;-) >It's best to assume, with embedded, that we're not shipping >ANY of the desktop or serv

Re: [patch 0/4] [resend] Add configuration options to disable features not needed on embedded devices

2008-07-31 Thread Tim Bird
David Miller wrote: > Some folks might find it instructive to do a google code search > or similar on the multicast socket options this things dikes out > of the tree. > > Even simple things like NTP will spew failures with this CONFIG_IGMP > thing turned off. And that's just the tip of the icebe

Re: [patch 0/4] [resend] Add configuration options to disable features not needed on embedded devices

2008-07-31 Thread Tim Bird
Tim Bird wrote: > It sounds like you don't want to talk about it any more, > but could you please give me the 30-second overview > of why ETHTOOL is required for proper ipv4 operation? ... > Sorry if I missed an earlier explanation. Never mind. I hadn't read the whole thread yet. ===

Re: [patch 0/4] [resend] Add configuration options to disable features not needed on embedded devices

2008-07-31 Thread Tim Bird
David Miller wrote: > From: David Woodhouse <[EMAIL PROTECTED]> > Date: Thu, 31 Jul 2008 10:51:52 +0100 > >> But there are a lot of people who really don't need these features >> and really want the option of leaving them out. > > I'll say it one last time. > > If you have ipv4 enabled, you need

Re: [patch 0/4] [resend] Add configuration options to disable features not needed on embedded devices

2008-07-31 Thread David Miller
From: David Woodhouse <[EMAIL PROTECTED]> Date: Thu, 31 Jul 2008 11:15:16 +0100 > While I agree with Andrew's observation, I'd also respectfully submit > that your argument is more fundamentally bogus than that. TCP and UDP > are _not_ universally available. They go away if you set CONFIG_INET=n.

Re: [patch 0/4] [resend] Add configuration options to disable features not needed on embedded devices

2008-07-31 Thread David Woodhouse
On Thu, 2008-07-31 at 03:02 -0700, David Miller wrote: > I explained why I didn't want to apply the IGMP one too. > > Andrew didn't like my objections, but that doesn't mean I > need to defend my position further. You said that it was part of the core BSD socket API and "Like TCP and UDP, multicas

Re: [patch 0/4] [resend] Add configuration options to disable features not needed on embedded devices

2008-07-31 Thread David Miller
From: David Woodhouse <[EMAIL PROTECTED]> Date: Thu, 31 Jul 2008 10:59:15 +0100 > I have drivers which don't even _have_ ethtool support, and they seem to > work fine. But leaving aside the debate on that point, your statement > also seemed to be covering the other patches, such as the IGMP one an

Re: [patch 0/4] [resend] Add configuration options to disable features not needed on embedded devices

2008-07-31 Thread David Woodhouse
On Thu, 2008-07-31 at 02:55 -0700, David Miller wrote: > From: David Woodhouse <[EMAIL PROTECTED]> > Date: Thu, 31 Jul 2008 10:51:52 +0100 > > > But there are a lot of people who really don't need these features > > and really want the option of leaving them out. > > I'll say it one last time. >

Re: [patch 0/4] [resend] Add configuration options to disable features not needed on embedded devices

2008-07-31 Thread David Miller
From: David Woodhouse <[EMAIL PROTECTED]> Date: Thu, 31 Jul 2008 10:51:52 +0100 > But there are a lot of people who really don't need these features > and really want the option of leaving them out. I'll say it one last time. If you have ipv4 enabled, you need ETHTOOL. -- To unsubscribe from thi

Re: [patch 0/4] [resend] Add configuration options to disable features not needed on embedded devices

2008-07-31 Thread David Woodhouse
On Thu, 2008-07-31 at 02:40 -0700, David Miller wrote: > From: Thomas Petazzoni <[EMAIL PROTECTED]> > Date: Thu, 31 Jul 2008 11:27:03 +0200 > > > Changes since previous post: > > > > * Add Matt Mackall's Signed-off-by on all patches > > * Make bonding and bridging select ethtool in the ethtool-

Re: [patch 0/4] [resend] Add configuration options to disable features not needed on embedded devices

2008-07-31 Thread David Miller
From: Thomas Petazzoni <[EMAIL PROTECTED]> Date: Thu, 31 Jul 2008 11:27:03 +0200 > Changes since previous post: > > * Add Matt Mackall's Signed-off-by on all patches > * Make bonding and bridging select ethtool in the ethtool-related >patch. The ethtool config option needs to be selected b

[patch 0/4] [resend] Add configuration options to disable features not needed on embedded devices

2008-07-31 Thread Thomas Petazzoni
Hi, This is a resend of the small patch list I sent on July, 29th. I'm resending the patches because they didn't make it to vger lists for some setup problem on my side, and because I've been asked by Adrian Bunk to resend them in order to get proper review. However, please note that the patches