From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Jan Beulich" Subject: [PATCH 02/11] gnttab: remove redundant xenheap check from gnttab_transfer() Date: Wed, 21 Jun 2017 03:31:18 -0600 Message-ID: <594A59060200007800165038@prv-mh.provo.novell.com> References: <594A57B10200007800165012@prv-mh.provo.novell.com> <594A57B10200007800165012@prv-mh.provo.novell.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=__PartB189AFF6.3__=" Return-path: Received: from mail6.bemta6.messagelabs.com ([193.109.254.103]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dNbyt-0007oy-0l for xen-devel@lists.xenproject.org; Wed, 21 Jun 2017 09:31:23 +0000 In-Reply-To: <594A57B10200007800165012@prv-mh.provo.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: xen-devel Cc: Stefano Stabellini , Wei Liu , George Dunlap , Andrew Cooper , Ian Jackson , Tim Deegan List-Id: xen-devel@lists.xenproject.org This is a MIME message. If you are reading this text, you may want to consider changing to a mail reader or gateway that understands how to properly handle MIME multipart messages. --=__PartB189AFF6.3__= Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Content-Disposition: inline The message isn't very useful, and the check is being done by steal_page() anyway. Signed-off-by: Jan Beulich --- a/xen/common/grant_table.c +++ b/xen/common/grant_table.c @@ -1843,15 +1843,6 @@ gnttab_transfer( } =20 page =3D mfn_to_page(mfn); - if ( unlikely(is_xen_heap_page(page)) ) - {=20 - put_gfn(d, gop.mfn); - gdprintk(XENLOG_INFO, "gnttab_transfer: xen frame %lx\n", - (unsigned long)gop.mfn); - gop.status =3D GNTST_bad_page; - goto copyback; - } - if ( steal_page(d, page, 0) < 0 ) { put_gfn(d, gop.mfn); --=__PartB189AFF6.3__= Content-Type: text/plain; name="gnttab-xfer-xenheap.patch" Content-Transfer-Encoding: quoted-printable Content-Disposition: attachment; filename="gnttab-xfer-xenheap.patch" gnttab: remove redundant xenheap check from gnttab_transfer()=0A=0AThe = message isn't very useful, and the check is being done by=0Asteal_page() = anyway.=0A=0ASigned-off-by: Jan Beulich =0A=0A--- = a/xen/common/grant_table.c=0A+++ b/xen/common/grant_table.c=0A@@ -1843,15 = +1843,6 @@ gnttab_transfer(=0A }=0A =0A page =3D mfn_to_pag= e(mfn);=0A- if ( unlikely(is_xen_heap_page(page)) )=0A- { = =0A- put_gfn(d, gop.mfn);=0A- gdprintk(XENLOG_INFO, = "gnttab_transfer: xen frame %lx\n",=0A- (unsigned = long)gop.mfn);=0A- gop.status =3D GNTST_bad_page;=0A- = goto copyback;=0A- }=0A-=0A if ( steal_page(d, page, 0) < = 0 )=0A {=0A put_gfn(d, gop.mfn);=0A --=__PartB189AFF6.3__= Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVs IG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVuLm9yZwpodHRwczovL2xpc3RzLnhlbi5v cmcveGVuLWRldmVsCg== --=__PartB189AFF6.3__=--