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

[Xen-devel] [PATCH] Remove uneeded 'else if' when closing event channels

To: Xen-Devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: [Xen-devel] [PATCH] Remove uneeded 'else if' when closing event channels.
From: Tony Breeds <tony@xxxxxxxxxxxxxxxxxx>
Date: Fri, 2 Dec 2005 16:05:19 +1100
Cc: Rusty Russell <rusty@xxxxxxxxxxxxxxx>
Delivery-date: Fri, 02 Dec 2005 05:06:11 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Mail-followup-to: Xen-Devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Rusty Russell <rusty@xxxxxxxxxxxxxxx>, Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.9i
Hi all,
        As far as I can see the remote domain cannot change from
underneath us.  Which means that the test for that situation is
pointless

This patch removes it.

Signed-off-by: Tony Breeds <tony@xxxxxxxxxxxxxxxxxx>

----------------------

diff -r f62f9b1732b9 xen/common/event_channel.c
--- a/xen/common/event_channel.c        Thu Dec  1 19:43:04 2005
+++ b/xen/common/event_channel.c        Fri Dec  2 15:54:25 2005
@@ -342,11 +342,6 @@
                 goto again;
             }
         }
-        else if ( d2 != chn1->u.interdomain.remote_dom )
-        {
-            rc = -EINVAL;
-            goto out;
-        }
     
         port2 = chn1->u.interdomain.remote_port;
         BUG_ON(!port_is_valid(d2, port2));

Yours Tony

   linux.conf.au       http://linux.conf.au/ || http://lca2006.linux.org.au/
   Jan 23-28 2006      The Australian Linux Technical Conference!


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel