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] [PATCH] Talk_to_backend on xenbus_probe only if drv->pro

To: Murillo Bernardes <mfb@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Talk_to_backend on xenbus_probe only if drv->probe was sucessful
From: Ewan Mellor <ewan@xxxxxxxxxxxxx>
Date: Thu, 15 Dec 2005 13:38:28 +0000
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 15 Dec 2005 13:40:29 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <200512121635.45358.mfb@xxxxxxxxxx>
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>
References: <200512121635.45358.mfb@xxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.9i
On Mon, Dec 12, 2005 at 04:35:45PM -0200, Murillo Bernardes wrote:

> Hi,
> 
> This patch makes xenbus_probe only call talk_to_backend if drv->probe() was 
> sucessful.
> 
> There is no sense in add a watch before drv->probe, because if it fails the 
> otherend_changed callback will be called and receive a struct with invalid 
> pointers and we get a segfault (ooops).

Applied, thanks Murillo.

Ewan.

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