>>> Paul Durrant <paul.durr...@citrix.com> 10/12/17 6:28 PM >>>
>@@ -1608,7 +1608,8 @@ fault:
>}
>
>static int
>-gnttab_populate_status_frames(struct domain *d, struct grant_table *gt,
>+gnttab_populate_status_frames(struct domain *d,
>+ struct grant_table *gt,
>unsigned int req_nr_frames)
What is this change about?
>+int gnttab_get_grant_frame(struct domain *d, unsigned long idx,
>+ mfn_t *mfn)
>+{
>+ struct grant_table *gt = d->grant_table;
>+ int rc;
>+
>+ /* write lock required as version may change and/or table may grow */
>+ grant_write_lock(gt);
>+
>+ rc = (gt->gt_version == 2 &&
>+ idx > XENMAPIDX_grant_table_status) ?
I don't understand this check - why does XENMAPIDX_grant_table_status
matter here at all? Same in gnttab_get_status_frame() then.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
