All of lore.kernel.org
 help / color / mirror / Atom feed
From: Julien Grall <julien.grall@arm.com>
To: Christoffer Dall <christoffer.dall@linaro.org>
Cc: kvmarm@lists.cs.columbia.edu, marc.zyngier@arm.com,
	fu.wei@linaro.org, kvm@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, wei@redhat.com,
	al.stone@linaro.org, gg@slimlogic.co.uk,
	Thomas Gleixner <tglx@linutronix.de>,
	Jason Cooper <jason@lakedaemon.net>
Subject: Re: [PATCH v5 6/9] irqchip/gic-v3: Parse and export virtual GIC information
Date: Wed, 6 Apr 2016 18:45:19 +0100	[thread overview]
Message-ID: <57054B2F.8040907@arm.com> (raw)
In-Reply-To: <20160406172228.GE8961@cbox>

Hi Christoffer,

On 06/04/2016 18:22, Christoffer Dall wrote:
> On Mon, Apr 04, 2016 at 12:37:37PM +0100, Julien Grall wrote:
>> +static void __init gic_acpi_setup_kvm_info(void)
>> +{
>> +	int irq;
>> +
>> +	if (!gic_acpi_collect_virt_info()) {
>> +		pr_warn("Unable to get hardware information used for virtualization\n");
>
> perhaps this should say something about gicv3 as well?

Actually, none of the pr_* call contain GICv3 and there is no pr_fmt.

I'm thinking to send a patch to add pr_fmt in the GICv3 driver.

>
> Otherwise (with my extremely limited acpi understanding):
>
> Reviewed-by: Christoffer Dall <christoffer.dall@linaro.org>

Thank you!

Regards,

-- 
Julien Grall

WARNING: multiple messages have this Message-ID (diff)
From: Julien Grall <julien.grall@arm.com>
To: Christoffer Dall <christoffer.dall@linaro.org>
Cc: al.stone@linaro.org, kvm@vger.kernel.org, marc.zyngier@arm.com,
	Jason Cooper <jason@lakedaemon.net>,
	linux-kernel@vger.kernel.org, fu.wei@linaro.org,
	Thomas Gleixner <tglx@linutronix.de>,
	kvmarm@lists.cs.columbia.edu,
	linux-arm-kernel@lists.infradead.org, gg@slimlogic.co.uk
Subject: Re: [PATCH v5 6/9] irqchip/gic-v3: Parse and export virtual GIC information
Date: Wed, 6 Apr 2016 18:45:19 +0100	[thread overview]
Message-ID: <57054B2F.8040907@arm.com> (raw)
In-Reply-To: <20160406172228.GE8961@cbox>

Hi Christoffer,

On 06/04/2016 18:22, Christoffer Dall wrote:
> On Mon, Apr 04, 2016 at 12:37:37PM +0100, Julien Grall wrote:
>> +static void __init gic_acpi_setup_kvm_info(void)
>> +{
>> +	int irq;
>> +
>> +	if (!gic_acpi_collect_virt_info()) {
>> +		pr_warn("Unable to get hardware information used for virtualization\n");
>
> perhaps this should say something about gicv3 as well?

Actually, none of the pr_* call contain GICv3 and there is no pr_fmt.

I'm thinking to send a patch to add pr_fmt in the GICv3 driver.

>
> Otherwise (with my extremely limited acpi understanding):
>
> Reviewed-by: Christoffer Dall <christoffer.dall@linaro.org>

Thank you!

Regards,

-- 
Julien Grall

WARNING: multiple messages have this Message-ID (diff)
From: julien.grall@arm.com (Julien Grall)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v5 6/9] irqchip/gic-v3: Parse and export virtual GIC information
Date: Wed, 6 Apr 2016 18:45:19 +0100	[thread overview]
Message-ID: <57054B2F.8040907@arm.com> (raw)
In-Reply-To: <20160406172228.GE8961@cbox>

Hi Christoffer,

On 06/04/2016 18:22, Christoffer Dall wrote:
> On Mon, Apr 04, 2016 at 12:37:37PM +0100, Julien Grall wrote:
>> +static void __init gic_acpi_setup_kvm_info(void)
>> +{
>> +	int irq;
>> +
>> +	if (!gic_acpi_collect_virt_info()) {
>> +		pr_warn("Unable to get hardware information used for virtualization\n");
>
> perhaps this should say something about gicv3 as well?

Actually, none of the pr_* call contain GICv3 and there is no pr_fmt.

I'm thinking to send a patch to add pr_fmt in the GICv3 driver.

>
> Otherwise (with my extremely limited acpi understanding):
>
> Reviewed-by: Christoffer Dall <christoffer.dall@linaro.org>

Thank you!

Regards,

-- 
Julien Grall

  reply	other threads:[~2016-04-06 17:45 UTC|newest]

Thread overview: 90+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-04 11:37 [PATCH v5 0/9] arm64: Add support for KVM with ACPI Julien Grall
2016-04-04 11:37 ` Julien Grall
2016-04-04 11:37 ` Julien Grall
2016-04-04 11:37 ` [PATCH v5 1/9] clocksource: arm_arch_timer: Gather KVM specific information in a structure Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-06 17:26   ` Christoffer Dall
2016-04-06 17:26     ` Christoffer Dall
2016-04-06 17:26     ` Christoffer Dall
2016-04-04 11:37 ` [PATCH v5 2/9] clocksource: arm_arch_timer: Extend arch_timer_kvm_info to get the virtual IRQ Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-06 17:26   ` Christoffer Dall
2016-04-06 17:26     ` Christoffer Dall
2016-04-06 17:26     ` Christoffer Dall
2016-04-04 11:37 ` [PATCH v5 3/9] irqchip/gic-v2: Gather ACPI specific data in a single structure Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-11  2:47   ` Hanjun Guo
2016-04-11  2:47     ` Hanjun Guo
2016-04-11  2:47     ` Hanjun Guo
2016-04-04 11:37 ` [PATCH v5 4/9] irqchip/gic-v2: Parse and export virtual GIC information Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-06 17:04   ` Christoffer Dall
2016-04-06 17:04     ` Christoffer Dall
2016-04-06 17:04     ` Christoffer Dall
2016-04-06 17:40     ` Julien Grall
2016-04-06 17:40       ` Julien Grall
2016-04-06 17:40       ` Julien Grall
2016-04-11  3:17   ` Hanjun Guo
2016-04-11  3:17     ` Hanjun Guo
2016-04-11  3:17     ` Hanjun Guo
2016-04-11 13:54     ` Julien Grall
2016-04-11 13:54       ` Julien Grall
2016-04-11 13:54       ` Julien Grall
2016-04-04 11:37 ` [PATCH v5 5/9] irqchip/gic-v3: Gather all ACPI specific data in a single structure Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-06 17:26   ` Christoffer Dall
2016-04-06 17:26     ` Christoffer Dall
2016-04-06 17:26     ` Christoffer Dall
2016-04-11  5:21   ` Hanjun Guo
2016-04-11  5:21     ` Hanjun Guo
2016-04-11  5:21     ` Hanjun Guo
2016-04-04 11:37 ` [PATCH v5 6/9] irqchip/gic-v3: Parse and export virtual GIC information Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-06 17:22   ` Christoffer Dall
2016-04-06 17:22     ` Christoffer Dall
2016-04-06 17:22     ` Christoffer Dall
2016-04-06 17:45     ` Julien Grall [this message]
2016-04-06 17:45       ` Julien Grall
2016-04-06 17:45       ` Julien Grall
2016-04-09  2:29   ` Shanker Donthineni
2016-04-09  2:29     ` Shanker Donthineni
2016-04-09  2:29     ` Shanker Donthineni
2016-04-11 14:03     ` Julien Grall
2016-04-11 14:03       ` Julien Grall
2016-04-11 14:03       ` Julien Grall
2016-04-11  5:27   ` Hanjun Guo
2016-04-11  5:27     ` Hanjun Guo
2016-04-11  5:27     ` Hanjun Guo
2016-04-11 14:27     ` Julien Grall
2016-04-11 14:27       ` Julien Grall
2016-04-11 14:27       ` Julien Grall
2016-04-11 15:25       ` Shanker Donthineni
2016-04-11 15:25         ` Shanker Donthineni
2016-04-11 15:25         ` Shanker Donthineni
2016-04-13 10:11         ` Hanjun Guo
2016-04-13 10:11           ` Hanjun Guo
2016-04-13 10:11           ` Hanjun Guo
2016-04-04 11:37 ` [PATCH v5 7/9] KVM: arm/arm64: arch_timer: Rely on the arch timer to parse the firmware tables Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-04 11:37 ` [PATCH v5 8/9] KVM: arm/arm64: vgic: Rely on the GIC driver " Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-06 17:25   ` Christoffer Dall
2016-04-06 17:25     ` Christoffer Dall
2016-04-06 17:25     ` Christoffer Dall
2016-04-04 11:37 ` [PATCH v5 9/9] clocksource: arm_arch_timer: Remove arch_timer_get_timecounter Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-04 11:37   ` Julien Grall
2016-04-06 17:28 ` [PATCH v5 0/9] arm64: Add support for KVM with ACPI Christoffer Dall
2016-04-06 17:28   ` Christoffer Dall
2016-04-06 17:28   ` Christoffer Dall
2016-04-06 17:49   ` Julien Grall
2016-04-06 17:49     ` Julien Grall
2016-04-06 17:49     ` Julien Grall

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=57054B2F.8040907@arm.com \
    --to=julien.grall@arm.com \
    --cc=al.stone@linaro.org \
    --cc=christoffer.dall@linaro.org \
    --cc=fu.wei@linaro.org \
    --cc=gg@slimlogic.co.uk \
    --cc=jason@lakedaemon.net \
    --cc=kvm@vger.kernel.org \
    --cc=kvmarm@lists.cs.columbia.edu \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marc.zyngier@arm.com \
    --cc=tglx@linutronix.de \
    --cc=wei@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.