Re: [2.6.19 PATCH 4/7] ehea: ethtool interface

On Fri, 2006-08-18 at 17:41 +0200, Thomas Klein wrote:
> Hi Alexey,
> 
> first of all thanks a lot for the extensive review.
> 
> 
> Alexey Dobriyan wrote:
> >> +	u64 hret = H_HARDWARE;
> > 
> > Useless assignment here and everywhere.
> > 
> 
> Initializing returncodes to errorstate is a cheap way to prevent
> accidentally returning (uninitalized) success returncodes which
> can lead to catastrophic misbehaviour.

If you try to return an uninitialized value the compiler will warn you,
you'll then look at the code and realise you missed a case, you might
save yourself a bug. By unconditionally initialising you are lying to
the compiler, and it can no longer help you.

cheers

-- 
Michael Ellerman
IBM OzLabs

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person

Attachment: signature.asc
Description: This is a digitally signed message part

---

• Follow-Ups:
  • Re: [2.6.19 PATCH 4/7] ehea: ethtool interface
    • From: Andy Gay <[email protected]>

• References:
  • [2.6.19 PATCH 4/7] ehea: ethtool interface
    • From: Jan-Bernd Themann <[email protected]>
  • Re: [2.6.19 PATCH 4/7] ehea: ethtool interface
    • From: Alexey Dobriyan <[email protected]>
  • Re: [2.6.19 PATCH 4/7] ehea: ethtool interface
    • From: Thomas Klein <[email protected]>

• Prev by Date: Re: [RFC/PATCH] Fixes for ULi5261 (tulip driver)
• Next by Date: Get list of open TCP/UDP ports from inside kernel
• Previous by thread: Re: [2.6.19 PATCH 4/7] ehea: ethtool interface
• Next by thread: Re: [2.6.19 PATCH 4/7] ehea: ethtool interface
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]