From: Julien Grall <julien.grall@arm.com>
To: Shanker Donthineni <shankerd@codeaurora.org>,
xen-devel <xen-devel@lists.xensource.com>,
Stefano Stabellini <sstabellini@kernel.org>
Cc: Philip Elcan <pelcan@codeaurora.org>,
Wei Liu <wei.liu2@citrix.com>,
Vikram Sethi <vikrams@codeaurora.org>,
George Dunlap <George.Dunlap@eu.citrix.com>,
Andrew Cooper <andrew.cooper3@citrix.com>,
Ian Jackson <ian.jackson@eu.citrix.com>, Tim Deegan <tim@xen.org>,
Jan Beulich <jbeulich@suse.com>
Subject: Re: [PATCH V3 4/4] arm/vgic: Change fixed number of mmio handlers to variable number
Date: Tue, 26 Jul 2016 22:02:07 +0100 [thread overview]
Message-ID: <0bc5022d-e44e-f38c-6d51-59fbba30bf07@arm.com> (raw)
In-Reply-To: <1469023256-6487-5-git-send-email-shankerd@codeaurora.org>
Hi Shanker,
On 20/07/2016 15:00, Shanker Donthineni wrote:
> Compute the number of mmio handlers that are required for vGICv3 and
> vGICv2 emulation drivers in vgic_v3_init()/vgic_v2_init(). Augment
> this variable number of mmio handers to a fixed number MAX_IO_HANDLER
s/handers/handlers/
> and pass it to domain_io_init() to allocate enough memory.
>
> New code path:
> domain_vgic_register(&count)
> domain_io_init(count + MAX_IO_HANDLER)
> domain_vgic_init()
>
> Signed-off-by: Shanker Donthineni <shankerd@codeaurora.org>
Acked-by: Julien Grall <julien.grall@arm.com>
Regards,
> ---
> xen/arch/arm/domain.c | 12 +++++++-----
> xen/arch/arm/vgic-v2.c | 3 ++-
> xen/arch/arm/vgic-v3.c | 5 ++++-
> xen/arch/arm/vgic.c | 10 +++-------
> xen/include/asm-arm/vgic.h | 5 +++--
> 5 files changed, 19 insertions(+), 16 deletions(-)
>
> diff --git a/xen/arch/arm/domain.c b/xen/arch/arm/domain.c
> index 0170cee..4e5259b 100644
> --- a/xen/arch/arm/domain.c
> +++ b/xen/arch/arm/domain.c
> @@ -546,7 +546,7 @@ void vcpu_destroy(struct vcpu *v)
> int arch_domain_create(struct domain *d, unsigned int domcr_flags,
> struct xen_arch_domainconfig *config)
> {
> - int rc, count;
> + int rc, count = 0;
>
> d->arch.relmem = RELMEM_not_started;
>
> @@ -569,10 +569,6 @@ int arch_domain_create(struct domain *d, unsigned int domcr_flags,
> share_xen_page_with_guest(
> virt_to_page(d->shared_info), d, XENSHARE_writable);
>
> - count = MAX_IO_HANDLER;
> - if ( (rc = domain_io_init(d, count)) != 0 )
> - goto fail;
> -
> if ( (rc = p2m_alloc_table(d)) != 0 )
> goto fail;
>
> @@ -609,6 +605,12 @@ int arch_domain_create(struct domain *d, unsigned int domcr_flags,
> goto fail;
> }
>
> + if ( (rc = domain_vgic_register(d, &count)) != 0 )
> + goto fail;
> +
> + if ( (rc = domain_io_init(d, count + MAX_IO_HANDLER)) != 0 )
> + goto fail;
> +
> if ( (rc = domain_vgic_init(d, config->nr_spis)) != 0 )
> goto fail;
>
> diff --git a/xen/arch/arm/vgic-v2.c b/xen/arch/arm/vgic-v2.c
> index 6a5e67b..c6d280e 100644
> --- a/xen/arch/arm/vgic-v2.c
> +++ b/xen/arch/arm/vgic-v2.c
> @@ -711,7 +711,7 @@ static const struct vgic_ops vgic_v2_ops = {
> .max_vcpus = 8,
> };
>
> -int vgic_v2_init(struct domain *d)
> +int vgic_v2_init(struct domain *d, int *mmio_count)
> {
> if ( !vgic_v2_hw.enabled )
> {
> @@ -721,6 +721,7 @@ int vgic_v2_init(struct domain *d)
> return -ENODEV;
> }
>
> + *mmio_count = 1; /* Only GICD region */
> register_vgic_ops(d, &vgic_v2_ops);
>
> return 0;
> diff --git a/xen/arch/arm/vgic-v3.c b/xen/arch/arm/vgic-v3.c
> index be9a9a3..ec038a3 100644
> --- a/xen/arch/arm/vgic-v3.c
> +++ b/xen/arch/arm/vgic-v3.c
> @@ -1499,7 +1499,7 @@ static const struct vgic_ops v3_ops = {
> .max_vcpus = 4096,
> };
>
> -int vgic_v3_init(struct domain *d)
> +int vgic_v3_init(struct domain *d, int *mmio_count)
> {
> if ( !vgic_v3_hw.enabled )
> {
> @@ -1509,6 +1509,9 @@ int vgic_v3_init(struct domain *d)
> return -ENODEV;
> }
>
> + /* GICD region + number of Redistributors */
> + *mmio_count = vgic_v3_rdist_count(d) + 1;
> +
> register_vgic_ops(d, &v3_ops);
>
> return 0;
> diff --git a/xen/arch/arm/vgic.c b/xen/arch/arm/vgic.c
> index a7ccfe7..768cb91 100644
> --- a/xen/arch/arm/vgic.c
> +++ b/xen/arch/arm/vgic.c
> @@ -88,18 +88,18 @@ static void vgic_rank_init(struct vgic_irq_rank *rank, uint8_t index,
> rank->vcpu[i] = vcpu;
> }
>
> -static int domain_vgic_register(struct domain *d)
> +int domain_vgic_register(struct domain *d, int *mmio_count)
> {
> switch ( d->arch.vgic.version )
> {
> #ifdef CONFIG_HAS_GICV3
> case GIC_V3:
> - if ( vgic_v3_init(d) )
> + if ( vgic_v3_init(d, mmio_count) )
> return -ENODEV;
> break;
> #endif
> case GIC_V2:
> - if ( vgic_v2_init(d) )
> + if ( vgic_v2_init(d, mmio_count) )
> return -ENODEV;
> break;
> default:
> @@ -124,10 +124,6 @@ int domain_vgic_init(struct domain *d, unsigned int nr_spis)
>
> d->arch.vgic.nr_spis = nr_spis;
>
> - ret = domain_vgic_register(d);
> - if ( ret < 0 )
> - return ret;
> -
> spin_lock_init(&d->arch.vgic.lock);
>
> d->arch.vgic.shared_irqs =
> diff --git a/xen/include/asm-arm/vgic.h b/xen/include/asm-arm/vgic.h
> index c3cc4f6..300f461 100644
> --- a/xen/include/asm-arm/vgic.h
> +++ b/xen/include/asm-arm/vgic.h
> @@ -304,9 +304,10 @@ extern int vgic_emulate(struct cpu_user_regs *regs, union hsr hsr);
> extern void vgic_disable_irqs(struct vcpu *v, uint32_t r, int n);
> extern void vgic_enable_irqs(struct vcpu *v, uint32_t r, int n);
> extern void register_vgic_ops(struct domain *d, const struct vgic_ops *ops);
> -int vgic_v2_init(struct domain *d);
> -int vgic_v3_init(struct domain *d);
> +int vgic_v2_init(struct domain *d, int *mmio_count);
> +int vgic_v3_init(struct domain *d, int *mmio_count);
>
> +extern int domain_vgic_register(struct domain *d, int *mmio_count);
> extern int vcpu_vgic_free(struct vcpu *v);
> extern int vgic_to_sgi(struct vcpu *v, register_t sgir,
> enum gic_sgi_mode irqmode, int virq,
>
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
next prev parent reply other threads:[~2016-07-26 21:02 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-20 14:00 [PATCH V3 0/4] Change fixed mmio handlers to a variable number Shanker Donthineni
2016-07-20 14:00 ` [PATCH V3 1/4] arm/io: Use separate memory allocation for mmio handlers Shanker Donthineni
2016-07-20 14:00 ` [PATCH V3 2/4] xen: Add generic implementation of binary search Shanker Donthineni
2016-08-01 12:46 ` Jan Beulich
2016-08-01 12:49 ` Andrew Cooper
2016-07-20 14:00 ` [PATCH V3 3/4] xen/arm: io: Use binary search for mmio handler lookup Shanker Donthineni
2016-07-26 21:02 ` Julien Grall
2016-07-20 14:00 ` [PATCH V3 4/4] arm/vgic: Change fixed number of mmio handlers to variable number Shanker Donthineni
2016-07-26 21:02 ` Julien Grall [this message]
2016-07-26 22:16 ` [PATCH V3 0/4] Change fixed mmio handlers to a " Stefano Stabellini
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=0bc5022d-e44e-f38c-6d51-59fbba30bf07@arm.com \
--to=julien.grall@arm.com \
--cc=George.Dunlap@eu.citrix.com \
--cc=andrew.cooper3@citrix.com \
--cc=ian.jackson@eu.citrix.com \
--cc=jbeulich@suse.com \
--cc=pelcan@codeaurora.org \
--cc=shankerd@codeaurora.org \
--cc=sstabellini@kernel.org \
--cc=tim@xen.org \
--cc=vikrams@codeaurora.org \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xensource.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).