On Tue, Sep 04, 2018 at 01:49:11AM +0100, Peter Maydell wrote: > On 3 September 2018 at 17:45, Andrew Jones <drjones@xxxxxxxxxx> wrote: > > We need to support two redistributor regions in order to support > > more than 123 vcpus. Also bump NR_CPUS to 512, since that's what > > KVM currently supports. > > Wouldn't it be neater to support the arbitrary number > of redistributor regions that the h/w spec allows and > the dt can describe, rather than just the specific case > of 2 regions that QEMU happens to use ? > It would be nicer, and I suppose it would be pretty easy to do as well. kvm-unit-tests/arm has always been a balance between being over-engineered for its purposes and being too hacky. In this case, I was probably too hacky. Although KVM only supports up to 512 VCPUS right now and these two regions support up to 635, so it'll be a while before a third region will be needed. If I find time I'll send a v2, but in the meantime it'd be good to get test results with this version to be sure I didn't miss anything else. Thanks, drew _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
