[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [kvm-unit-tests PATCH v6 07/13] arm/arm64: ITS: its_enable_defaults
From: |
Andrew Jones |
Subject: |
Re: [kvm-unit-tests PATCH v6 07/13] arm/arm64: ITS: its_enable_defaults |
Date: |
Wed, 11 Mar 2020 16:38:42 +0100 |
On Wed, Mar 11, 2020 at 02:51:11PM +0100, Eric Auger wrote:
> +/* must be called after gicv3_enable_defaults */
> +void its_enable_defaults(void)
> +{
> + int i;
> +
> + /* Allocate LPI config and pending tables */
> + gicv3_lpi_alloc_tables();
> +
> + for (i = 0; i < nr_cpus; i++)
> + gicv3_lpi_rdist_enable(i);
You still haven't explained what's wrong with for_each_present_cpu. Also,
I see you've added 'i < nr_cpus' loops in arm/gic.c too. I'd prefer we not
assume that all cpu's are present (even though, currently, they must be),
because we may want to integrate cpu hotplug tests with these tests at
some point.
> +
> + writel(GITS_CTLR_ENABLE, its_data.base + GITS_CTLR);
> +}
> --
> 2.20.1
>
>
- [kvm-unit-tests PATCH v6 01/13] libcflat: Add other size defines, (continued)
- [kvm-unit-tests PATCH v6 01/13] libcflat: Add other size defines, Eric Auger, 2020/03/11
- [kvm-unit-tests PATCH v6 02/13] page_alloc: Introduce get_order(), Eric Auger, 2020/03/11
- [kvm-unit-tests PATCH v6 03/13] arm/arm64: gic: Introduce setup_irq() helper, Eric Auger, 2020/03/11
- [kvm-unit-tests PATCH v6 04/13] arm/arm64: gicv3: Add some re-distributor defines, Eric Auger, 2020/03/11
- [kvm-unit-tests PATCH v6 05/13] arm/arm64: gicv3: Set the LPI config and pending tables, Eric Auger, 2020/03/11
- [kvm-unit-tests PATCH v6 06/13] arm/arm64: ITS: Introspection tests, Eric Auger, 2020/03/11
- [kvm-unit-tests PATCH v6 08/13] arm/arm64: ITS: Device and collection Initialization, Eric Auger, 2020/03/11
- [kvm-unit-tests PATCH v6 07/13] arm/arm64: ITS: its_enable_defaults, Eric Auger, 2020/03/11
- Re: [kvm-unit-tests PATCH v6 07/13] arm/arm64: ITS: its_enable_defaults,
Andrew Jones <=
- [kvm-unit-tests PATCH v6 09/13] arm/arm64: ITS: Commands, Eric Auger, 2020/03/11
- [kvm-unit-tests PATCH v6 10/13] arm/arm64: ITS: INT functional tests, Eric Auger, 2020/03/11
- [kvm-unit-tests PATCH v6 11/13] arm/run: Allow Migration tests, Eric Auger, 2020/03/11
- [kvm-unit-tests PATCH v6 12/13] arm/arm64: ITS: migration tests, Eric Auger, 2020/03/11
- [kvm-unit-tests PATCH v6 13/13] arm/arm64: ITS: pending table migration test, Eric Auger, 2020/03/11
- Re: [kvm-unit-tests PATCH v6 00/13] arm/arm64: Add ITS tests, Andrew Jones, 2020/03/11