Hi Stephen: Thanks for your review. On Sat, Feb 23, 2019 at 1:08 AM Stephen Hemminger <sthem...@microsoft.com> wrote: > > int hyperv_fill_flush_guest_mapping_list( > struct hv_guest_mapping_flush_list *flush, > - u64 start_gfn, u64 pages) > + int offset, u64 start_gfn, u64 pages) > { > u64 cur = start_gfn; > u64 additional_pages; > - int gpa_n = 0; > + int gpa_n = offset; > > do { > /* > > Do you mean to support negative offsets here? Maybe unsigned would be better?
Yes, this makes sense. Will update. Thanks. -- Best regards Tianyu Lan _______________________________________________ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm