Hi Alexandru, On 12/1/20 4:01 PM, Alexandru Elisei wrote: > vgic_v3_map_resources() returns -EBUSY if the VGIC isn't initialized, > update the comment to kvm_vgic_map_resources() to match what the function > does. > > Signed-off-by: Alexandru Elisei <alexandru.elisei@xxxxxxx> > --- > arch/arm64/kvm/vgic/vgic-init.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/kvm/vgic/vgic-init.c b/arch/arm64/kvm/vgic/vgic-init.c > index a2f4d1c85f00..8942b2191bcf 100644 > --- a/arch/arm64/kvm/vgic/vgic-init.c > +++ b/arch/arm64/kvm/vgic/vgic-init.c > @@ -419,7 +419,8 @@ int vgic_lazy_init(struct kvm *kvm) > * Map the MMIO regions depending on the VGIC model exposed to the guest > * called on the first VCPU run. > * Also map the virtual CPU interface into the VM. > - * v2/v3 derivatives call vgic_init if not already done. > + * v2 calls vgic_init if not already done. nit: s/vgic_init/vgic_init()? > + * v3 and derivatives return an error if the VGIC is not initialized. > * vgic_ready() returns true if this function has succeeded. > * @kvm: kvm struct pointer > */ > Besides Reviewed-by: Eric Auger <eric.auger@xxxxxxxxxx> Thanks Eric _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
