On Tue, Mar 29, 2022 at 02:14:16PM +0800, yuzhe wrote: > > > @@ -229,7 +229,7 @@ static void print_irq_state(struct seq_file *s, struct vgic_irq *irq, > > > static int vgic_debug_show(struct seq_file *s, void *v) > > > { > > > - struct kvm *kvm = (struct kvm *)s->private; > > > + struct kvm *kvm = s->private; > > > struct vgic_state_iter *iter = (struct vgic_state_iter *)v; ^^^^^^^^^^^^^^^^^^^^^^^^^^ This cast is unnecessary. Look for others as well. regards, dan carpenter _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/mailman/listinfo/kvmarm