[PATCH v2 01/15] KVM: arm/arm64: VGIC: don't track used LRs in the distributor
p.fedin at samsung.com
Fri Oct 2 02:55:05 PDT 2015
Hello! One more concern.
> Currently we track which IRQ has been mapped to which VGIC list
> register and also have to synchronize both. We used to do this
> to hold some extra state (for instance the active bit).
> It turns out that this extra state in the LRs is no longer needed and
> this extra tracking causes some pain later.
Returning to the beginning, can you explain, what kind of pain exactly does it bring?
For some reasons here i had to keep all this tracking mechanism, and modified your patchset. I see
no significant problems, except memory usage. I have to allocate vgic_irq_lr_map large enough to
hold indexes up to 16384, and indexes from dist->nr_irqs to 8192 appear to be not used.
Since the map itself is actually used only for piggy-back optimization, it is possible to easily
get rid of it using for_each_set_bit(lr, vgic_cpu->lr_used, vgic->nr_lr) iteration instead. The rest
of mechanism will work as it is, there's no need to remove the state tracking bitmap and
I am currently testing this approach and preparing my alternative patch for upstreaming.
Samsung Electronics Research center Russia
More information about the linux-arm-kernel