From mboxrd@z Thu Jan  1 00:00:00 1970
From: Julien Grall <julien.grall@linaro.org>
Subject: Re: [PATCH v5 12/21] xen/arm: move is_vcpu_running
	function to sched.h
Date: Thu, 19 Jun 2014 12:03:21 +0100
Message-ID: <53A2C379.6060109@linaro.org>
References: <1402580192-13937-1-git-send-email-vijay.kilari@gmail.com>	<1402580192-13937-13-git-send-email-vijay.kilari@gmail.com>	<539DC941.8010702@linaro.org>	<539EC846020000780001A829@mail.emea.novell.com>
	<CALicx6va=2js1JnMM_LdEjgGp-QZT7Z+bSmpxeBpGOwFdmNsfQ@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <CALicx6va=2js1JnMM_LdEjgGp-QZT7Z+bSmpxeBpGOwFdmNsfQ@mail.gmail.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Vijay Kilari <vijay.kilari@gmail.com>, Jan Beulich <JBeulich@suse.com>
Cc: Keir Fraser <keir@xen.org>, Ian Campbell <Ian.Campbell@citrix.com>, Stefano Stabellini <stefano.stabellini@eu.citrix.com>, Prasun Kapoor <Prasun.Kapoor@caviumnetworks.com>, Tim Deegan <tim@xen.org>, Vijaya Kumar K <vijaya.kumar@caviumnetworks.com>, "xen.org" <ian.jackson@eu.citrix.com>, george.dunlap@citrix.com, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>, Stefano Stabellini <stefano.stabellini@citrix.com>
List-Id: xen-devel@lists.xenproject.org

On 06/19/2014 11:58 AM, Vijay Kilari wrote:
> Hi Julien, Stefano, Jan,
> 
> On Mon, Jun 16, 2014 at 2:04 PM, Jan Beulich <JBeulich@suse.com> wrote:
>>>>> On 15.06.14 at 18:26, <julien.grall@linaro.org> wrote:
>>>> --- a/xen/include/xen/sched.h
>>>> +++ b/xen/include/xen/sched.h
>>>> @@ -826,6 +826,22 @@ void watchdog_domain_destroy(struct domain *d);
>>>>   #define need_iommu(d)    (0)
>>>>   #endif
>>>>
>>>> +static inline int is_vcpu_running(struct domain *d, int vcpuid)
>>>> +{
>>>> +    struct vcpu *v;
>>>> +
>>>> +    if ( vcpuid >= d->max_vcpus )
>>>> +        return 0;
>>>> +
>>>> +    v = d->vcpu[vcpuid];
>>>> +    if ( v == NULL )
>>>> +        return 0;
>>>> +    if (test_bit(_VPF_down, &v->pause_flags) )
>>>> +        return 0;
>>>> +
>>>> +    return 1;
>>>> +}
>>
>> If this function was to become common code, you'd have to not only
>> justify it (as requested by Julien), but also
>> - name it according to its function (VPF_down not set doesn't mean
>>   "running", it merely means "up")
> 
> IMO, is_vcpu_running() usage in vgic.c is just to check if vcpu is runnable
> and not down. So checking apart from VPF_down is enough.
> If so, we should change the name of this function

You are wrong, the usage in vgic.c is to check that the VCPU is online.
So we don't inject an IRQ to this VCPU.

We don't care if the VCPU is block/runnable or whatever.

What Jan was asking is to rename the function in is_vcpu_up (or smth
similar).

Regards,

-- 
Julien Grall