gicv3_its: do LPI init only once per CPU

[jchandra]

Summary

The initialization required for LPIs (setting up pending tables etc.) has to be done just once per CPU, even in the case where there are multiple ITS blocks associated with the CPU.

Add a variable lpi_enabled in the per-cpu distributor info for this and use it ensure that we call its_init_cpu_lpi() just once.

This enables us to support the case where multiple ITS blocks can generate LPIs to a CPU.

[jchandra]

This fixes https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=236333

[andrew]

sys/arm64/arm64/gic_v3_var.h

46 ↗

(On Diff #55932)

Just make this an int or bool. The struct will be padded any way so it doesn't save any space.

sys/arm64/arm64/gicv3_its.c

674 ↗

(On Diff #55932)

How do we get into its_init_cpu on a given CPU? I assume it's when there are multiple ITS devices.

[jchandra]

Let me know if you need a new rev with bool.

sys/arm64/arm64/gic_v3_var.h

46 ↗

(On Diff #55932)

The idea was to use other bits for more flags if needed. I think bool would work as well, if the bit field is looks like an unnecessary complication.

sys/arm64/arm64/gicv3_its.c

674 ↗

(On Diff #55932)

gicv3_its_init_secondary calls it for each secondary CPU.

This is to handle the case when there are multiple ITS blocks associated with a CPU. We need to do the LPI init once per CPU (per- redistributor actually). The collection init has to be done for all the ITS blocks.

So the idea is to get a per-redistributor flag to mark LPI initialization status.

[andrew]

If you think more flags will be needed later you can make it a u_int with a per-flag macro. If not just make it a bool.

sys/arm64/arm64/gic_v3_var.h

46 ↗

(On Diff #55932)

You could also make it a u_int and define a flag. e.g.:

struct redist_pcpu {
  ...
  u_int flags;
};

#define RDIST_PCPU_LPI_ENBLED 0x01

sys/arm64/arm64/gicv3_its.c

674 ↗

(On Diff #55932)

Ok, it would pay to explain this in a comment.