On Sat, Sep 16, 2017 at 09:59:21AM +0800, wanghaibin wrote: > On 2017/9/14 3:14, Christoffer Dall wrote: > > > On Wed, Sep 06, 2017 at 09:05:08PM +0800, wanghaibin wrote: > >> We slightly refactor vgic_its_destroy, separate vgic_its_free_list() > >> function for later patch invoke. > >> > >> The patch also take a functional change. If the its->device_list.next > > > > I don't see a functional change in this patch? > > > I mean, if this check can its->device_list.next happened, this patch will > free the its structure compared with the original implementation. > I really cannot see how this patch alone changes anything at all. Thanks, -Christoffer _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/mailman/listinfo/kvmarm