From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1752256AbdGGRJy (ORCPT <rfc822;w@1wt.eu>);
        Fri, 7 Jul 2017 13:09:54 -0400
Received: from foss.arm.com ([217.140.101.70]:49744 "EHLO foss.arm.com"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1750848AbdGGRJx (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Fri, 7 Jul 2017 13:09:53 -0400
Subject: Re: Problem with commit bf22ff45bed664aefb5c4e43029057a199b7070c
To: Juergen Gross <jgross@suse.com>, jeffy.chen@rock-chips.com,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
References: <dc3e7811-e3f8-1cf9-d556-1ac57288daa5@suse.com>
Cc: xen-devel <xen-devel@lists.xenproject.org>,
        Thomas Gleixner <tglx@linutronix.de>,
        Boris Ostrovsky <boris.ostrovsky@oracle.com>,
        Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>,
        Peter Zijlstra <peterz@infradead.org>
From: Marc Zyngier <marc.zyngier@arm.com>
Organization: ARM Ltd
Message-ID: <9bcc2acd-1349-a834-c141-921d703c77cb@arm.com>
Date: Fri, 7 Jul 2017 18:09:50 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Thunderbird/45.8.0
MIME-Version: 1.0
In-Reply-To: <dc3e7811-e3f8-1cf9-d556-1ac57288daa5@suse.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 07/07/17 15:51, Juergen Gross wrote:
> Commit bf22ff45bed664aefb5c4e43029057a199b7070c ("genirq: Avoid
> unnecessary low level irq function calls") breaks Xen guest
> save/restore handling.
> 
> The main problem are the PV devices using Xen event channels as
> interrupt sources which are represented as an "irq chip" in the kernel.
> When saving the guest the event channels are masked internally. At
> restore time event channels are re-established and unmasked via
> irq_startup(). Unfortunately above commit will let the unmask operation
> be a nop as the irq handling doesn't know about the masking done before.
> 
> I have a patch repairing the issue, but I'm not sure if this way to do
> it would be accepted. I have exported mask_irq() and I'm doing the
> masking now through this function. Would the attached patch be
> acceptable? Or is there a better way to solve the problem?

The correct API to prevent an interrupt from firing would be
disable_irq(), which is much more straightforward and is readily exported.

I'm unsure of what the expected flow is though, so there might be some
more fixes to be made in your code.

Thanks,

	M.
-- 
Jazz is not dead. It just smells funny...