From mboxrd@z Thu Jan  1 00:00:00 1970
From: Igor Mammedov <imammedo@redhat.com>
Subject: Re: [PATCH 1/2] xen: Clear IRQ_GUEST bit from irq_desc
	status if its action is NULL
Date: Tue, 13 Sep 2011 15:56:37 +0200
Message-ID: <4E6F6115.4060204@redhat.com>
References: <1315921459-17059-1-git-send-email-imammedo@redhat.com>
	<1315921459-17059-3-git-send-email-imammedo@redhat.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xensource.com>
In-Reply-To: <1315921459-17059-3-git-send-email-imammedo@redhat.com>
List-Unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xensource.com>
List-Help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-Subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
Sender: xen-devel-bounces@lists.xensource.com
Errors-To: xen-devel-bounces@lists.xensource.com
To: xen-devel@lists.xensource.com
Cc: keir.fraser@citrix.com, JBeulich@suse.com
List-Id: xen-devel@lists.xenproject.org

This one has wrong Subject. Just forget about it, please.
I'll send it with correct subj.

On 09/13/2011 03:44 PM, Igor Mammedov wrote:
> Signed-off-by: Igor Mammedov<imammedo@redhat.com>
> Reviewed-by: Jan Beulich<JBeulich@suse.com>
>
> diff -r 2049f7ca3177 -r 884814bfc22e xen/arch/x86/irq.c
> --- a/xen/arch/x86/irq.c	Tue Sep 13 14:44:59 2011 +0200
> +++ b/xen/arch/x86/irq.c	Tue Sep 13 14:47:46 2011 +0200
> @@ -1599,12 +1599,7 @@ static int pirq_guest_force_unbind(struc
>
>       action = (irq_guest_action_t *)desc->action;
>       if ( unlikely(action == NULL) )
> -    {
> -        desc->status&= ~IRQ_GUEST;
> -        dprintk(XENLOG_G_WARNING, "dom%d: pirq %d: desc->action is NULL!\n",
> -            d->domain_id, pirq->pirq);
> -        goto out;
> -    }
> +        goto unbind;
>
>       for ( i = 0; (i<  action->nr_guests)&&  (action->guest[i] != d); i++ )
>           continue;
> @@ -1612,6 +1607,7 @@ static int pirq_guest_force_unbind(struc
>           goto out;
>
>       bound = 1;
> + unbind:
>       oldaction = __pirq_guest_unbind(d, pirq, desc);
>
>    out:
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel

-- 
Thanks,
  Igor