From: Marc Zyngier <marc.zyngier@arm.com>
To: zijun_hu <zijun_hu@zoho.com>
Cc: <tglx@linutronix.de>, <jason@lakedaemon.net>, <james.morse@arm.com>,
        <sudeep.holla@arm.com>, <linux-kernel@vger.kernel.org>,
        <zijun_hu@htc.com>
Subject: Re: [PATCH 1/1] irqchip/gicv3: iterate over possible CPUs by for_each_possible_cpu()
In-Reply-To: <59BA1062.1010406@zoho.com> (zijun hu's message of "Thu, 14 Sep
        2017 13:15:14 +0800")
Organization: ARM Ltd
References: <59BA1062.1010406@zoho.com>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)
Date: Thu, 14 Sep 2017 20:20:45 +0100
Message-ID: <86o9qdqe7m.fsf@arm.com>
MIME-Version: 1.0
Content-Type: text/plain
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 469
Lines: 15

On Thu, Sep 14 2017 at  1:15:14 pm BST, zijun_hu <zijun_hu@zoho.com> wrote:
> From: zijun_hu <zijun_hu@htc.com>
>
> get_cpu_number() doesn't use existing helper to iterate over possible
> CPUs, so error happens in case of discontinuous @cpu_possible_mask
> such as 0b11110001.

Do you have an example of such a situation? Your patch is definitely an
improvement, but I'd like to understand how you get there...

Thanks,

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