All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] stubdoms phy disks sharing
@ 2009-05-18 13:19 Stefano Stabellini
  2009-05-19 10:47 ` Pasi Kärkkäinen
  0 siblings, 1 reply; 3+ messages in thread
From: Stefano Stabellini @ 2009-05-18 13:19 UTC (permalink / raw)
  To: xen-devel

Hi all,
this patch fixes the stubdoms issue with phy disks: add an exception for
stubdoms in the same_vm hotplug script function.

Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> 

---


diff -r 7d552e56d105 tools/hotplug/Linux/block-common.sh
--- a/tools/hotplug/Linux/block-common.sh	Wed May 13 10:39:44 2009 +0100
+++ b/tools/hotplug/Linux/block-common.sh	Mon May 18 11:08:58 2009 +0100
@@ -110,7 +110,8 @@
   # allowed.
   local othervm=$(xenstore_read_default "/local/domain/$otherdom/vm"         \
                   "$FRONTEND_UUID")
-
-  [ "$FRONTEND_UUID" = "$othervm" ]
+  local target=$(xenstore_read_default  "/local/domain/$FRONTEND_ID/target"   \
+                 "-1")
+  [ "$FRONTEND_UUID" = "$othervm" -o "$target" = "$otherdom" ]
 }

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] stubdoms phy disks sharing
  2009-05-18 13:19 [PATCH] stubdoms phy disks sharing Stefano Stabellini
@ 2009-05-19 10:47 ` Pasi Kärkkäinen
  2009-05-19 11:38   ` Stefano Stabellini
  0 siblings, 1 reply; 3+ messages in thread
From: Pasi Kärkkäinen @ 2009-05-19 10:47 UTC (permalink / raw)
  To: Stefano Stabellini; +Cc: xen-devel

On Mon, May 18, 2009 at 02:19:06PM +0100, Stefano Stabellini wrote:
> Hi all,
> this patch fixes the stubdoms issue with phy disks: add an exception for
> stubdoms in the same_vm hotplug script function.
> 

Hi,

Is this needed for Xen 3.4.0 ?

-- Pasi

> Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> 
> 
> ---
> 
> 
> diff -r 7d552e56d105 tools/hotplug/Linux/block-common.sh
> --- a/tools/hotplug/Linux/block-common.sh	Wed May 13 10:39:44 2009 +0100
> +++ b/tools/hotplug/Linux/block-common.sh	Mon May 18 11:08:58 2009 +0100
> @@ -110,7 +110,8 @@
>    # allowed.
>    local othervm=$(xenstore_read_default "/local/domain/$otherdom/vm"         \
>                    "$FRONTEND_UUID")
> -
> -  [ "$FRONTEND_UUID" = "$othervm" ]
> +  local target=$(xenstore_read_default  "/local/domain/$FRONTEND_ID/target"   \
> +                 "-1")
> +  [ "$FRONTEND_UUID" = "$othervm" -o "$target" = "$otherdom" ]
>  }
>  
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] stubdoms phy disks sharing
  2009-05-19 10:47 ` Pasi Kärkkäinen
@ 2009-05-19 11:38   ` Stefano Stabellini
  0 siblings, 0 replies; 3+ messages in thread
From: Stefano Stabellini @ 2009-05-19 11:38 UTC (permalink / raw)
  To: Pasi Kärkkäinen; +Cc: xen-devel

Pasi Kärkkäinen wrote:

> On Mon, May 18, 2009 at 02:19:06PM +0100, Stefano Stabellini wrote:
>> Hi all,
>> this patch fixes the stubdoms issue with phy disks: add an exception for
>> stubdoms in the same_vm hotplug script function.
>>
> 
> Hi,
> 
> Is this needed for Xen 3.4.0 ?
> 

Yes, I think it would be a good idea to backport this patch to the 3.4
branch.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-05-19 11:38 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-05-18 13:19 [PATCH] stubdoms phy disks sharing Stefano Stabellini
2009-05-19 10:47 ` Pasi Kärkkäinen
2009-05-19 11:38   ` Stefano Stabellini

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.