WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] xen 2.6.8.1 sparse diff

To: Adam Heath <doogie@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] xen 2.6.8.1 sparse diff
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Tue, 05 Oct 2004 20:27:57 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 05 Oct 2004 20:36:04 +0100
Envelope-to: steven.hand@xxxxxxxxxxxx
In-reply-to: Your message of "Tue, 05 Oct 2004 14:04:04 CDT." <Pine.LNX.4.58.0410051359200.1194@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
List-archive: <http://sourceforge.net/mailarchive/forum.php?forum=xen-devel>
List-help: <mailto:xen-devel-request@lists.sourceforge.net?subject=help>
List-id: List for Xen developers <xen-devel.lists.sourceforge.net>
List-post: <mailto:xen-devel@lists.sourceforge.net>
List-subscribe: <https://lists.sourceforge.net/lists/listinfo/xen-devel>, <mailto:xen-devel-request@lists.sourceforge.net?subject=subscribe>
List-unsubscribe: <https://lists.sourceforge.net/lists/listinfo/xen-devel>, <mailto:xen-devel-request@lists.sourceforge.net?subject=unsubscribe>
Sender: xen-devel-admin@xxxxxxxxxxxxxxxxxxxxx
> So, I'm begining to look more closely at xen.  Trying to get it all to
> compile, and work with our hardware.
> 
> After overlaying the 2.6.8.1 sparse tree, I looked at the diff of the source.
> In net/core/skbuff.c, I see a function alloc_skb_from_cache.  This function
> uses goto, when it's not needed.

Firstly, that function is a minimal modification of
alloc_skb(). Secondly, that style of exit/error/unusual path using
'goto' is used throughout Linux, for good reasons:
 1. Having a common exit path per function reduces bugs as only need
    to write epilogue code once. Hence the 'out' label.
 2. Older gcc versions need help in outlining rarely-executed basic
    blocks. Hence the 'nodata' label.

 -- Keir


-------------------------------------------------------
This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
Use IT products in your business? Tell us what you think of them. Give us
Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
http://productguide.itmanagersjournal.com/guidepromo.tmpl
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/xen-devel

<Prev in Thread] Current Thread [Next in Thread>