login
Header Space

 
 

Quote: Not An Optimistic Enough Dad

July 22, 2008 - 5:30pm
Submitted by Jeremy on July 22, 2008 - 5:30pm.

"Looking at the code it's apparently because I'm not an optimistic enough dad. But hey, if you had three pre-teenage girls, you might not be all that optimistic either."

— Linus Torvalds, in a July 21st, 2008 message on the Linux Kernel mailing list.

Pre-teenage girls

July 23, 2008 - 8:47am
Anonymous (not verified)

Oh, you think having three pre-teenage girls is difficult?

Wait until they are teenage girls...

Maybe that's has point about

July 23, 2008 - 9:39am
Anonymous (not verified)

Maybe that's has point about optimism; "In two years... Hell WILL break loose" ;)

Teenage Daughters...

July 29, 2008 - 12:06pm
yokem55 (not verified)

There's an old saying in my family that teenage daughters are God's way of punishing men for having been teenage boys.

*chuckle*

July 23, 2008 - 1:36pm

David Miller's reply is a little opaque. :-)

What is that, a base64-encoded commit or something? Indeed, it is, as running it through "base64 -d" yields:

From: Linus Torvalds 
Date: Sun, 20 Jul 2008 18:07:28 -0700 (PDT)

> This warning also seems to be new:
> 
>       net/ipv6/addrconf.c: In function ¡addrconf_add_linklocal¢:
>       net/ipv6/addrconf.c:2318: warning: unused variable ¡net¢
> 
> and looking at the code it's apparently because I'm not an optimistic 
> enough dad.
> 
> But hey, if you had three pre-teenage girls, you might not be all that 
> optimistic either. So I think that's reasonable.
> 
> Problem seems to have been introduced by 53b7997f ("ipv6 netns: Make 
> several "global" sysctl variables namespace aware")

Let's just expand the thing in the one spot it gets
referenced in.

Please apply, thanks.

ipv6: Fix warning in addrconf code.

Reported by Linus.

Signed-off-by: David S. Miller 

diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c
index 580ae50..9f4fcce 100644
--- a/net/ipv6/addrconf.c
+++ b/net/ipv6/addrconf.c
@@ -2315,12 +2315,11 @@ static void init_loopback(struct net_device *dev)
 static void addrconf_add_linklocal(struct inet6_dev *idev, struct in6_addr *addr)
 {
        struct inet6_ifaddr * ifp;
-       struct net *net = dev_net(idev->dev);
        u32 addr_flags = IFA_F_PERMANENT;
 
 #ifdef CONFIG_IPV6_OPTIMISTIC_DAD
        if (idev->cnf.optimistic_dad &&
-

--
Program Intellivision and play Space Patrol!

Dads

July 25, 2008 - 12:22am
Mark1 (not verified)

I hear you . . . as I have two teenage girls and one 22 year old son who is back living at home. It is not necessarily the kids that are always the challenge, but being in unity with the wife takes more energy than working with the kids.

I think I need a rest . . . maybe in 5 or 6 years.

Cheers,

Mark

Comment viewing options

Select your preferred way to display the comments and click "Save settings" to activate your changes.
speck-geostationary