[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [PATCH v8 1/2] IOMMU/VT-d: wire common device reserved memory API


  • To: Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
  • From: Henry Wang <Henry.Wang@xxxxxxx>
  • Date: Fri, 4 Nov 2022 02:34:16 +0000
  • Accept-language: zh-CN, en-US
  • Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=23qKp8ButlWL4cXYmnm1YOIdtG+G16PRwEGbMk8Vgmg=; b=IwJdtFjtm6wD4GuwpdcAtriwOYOicDUf0Ob6qBWWb7pyju3twrAPfQTDL9OvfV320Wg0Uo+4cltlnwwNfU8dFAXJ21hJTZ+4lhNYpQPeceLC09zfYDeAFitl/XexSgb0gB67RaXv3Tt6EGw0IPHRS+FHwU8MR9FL8XfME840gPi5+LB0AWu4LtihZQ31nrM07fkdZPABX7/16tne4BsEJrY46/DDP1tGSjO8x3uxVXMTuUZxxYuVNwP5v5w/x74VVVwfq/FIONUufoc58zSqOdATXEu5fMB3WFpHcnk90u3HlmCvECvpnSotXe8hWh2EnVI8Am6w5HEvvRciTukIpw==
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=23qKp8ButlWL4cXYmnm1YOIdtG+G16PRwEGbMk8Vgmg=; b=SOZ3W4KSdy3nufyDxgmEUImHiUFSS97yln5YqUEqccYyV2o+LkHDXq1oooKmrCVMmpx2e9S7NkRWtwqskYL83yyU+rvgTI3Fgi5fYeAwcF0YtDd7mRWZEem2PxubZbaCNTVI/1s8ZqiIMAsRuS1SoF0KHuBY19HHH+5dJb0seVrBfXtin/dbFg12aazbnnUscyVmwRQ8ms/6BqSodScZznB1o81wK9VrEp3xMx+GfAoBDv3ke44sr6wN3ZGv0hnCBxTOpFI4MU76/1Tw0ify9WYyyVCBwZ0frPYyKUpRXStnP7WEPQPOx3NajF2dbwItY4KDJ/pnl/OjIY/p8LorLg==
  • Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=Wbo0WZMES99eWhGddc52bNu59go2GkSyaPEvGmFGkRYq/slp0drhXt8LaVhHqaMS8p5Ym+dgAKJRuXypUCV8m9mWU6MlOpzDmnSnroR9lJ7lQo+pvgUKawmTyLwss/IneZP85+jZt7aQfwlMwzcZvlYBBqqiocAqFq0XyHXEvJUsNF93FwWDkB9OwD/mJaCDNWeD5pTeUsN2IOaRC8Of6ywzWZ6ToaqFGzhS8gaEFCiEhvz0GmeGXkaf0nUgLZ/FZq9rNJGAuKea8P6erKLJMgOYSRlKLpCNNxYNM73Oa443q+o3Ah1eso6O4+AMQoe8mbbLHedQkw4OlUaDb2X34A==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=MruC0lOsSv3abyAPShdVLDD5FGQ8S7ltajoPpG+1FgxAJI2pH0S/PQTITh+NzQ4AcxbUFUBGnz7pPoDcFz7rAt9TYfaob+rmlwrqigY/QoMhLwYFBEzdOsbDuovN6/SfzWq8NBHw3av10B1pnMk1wGGIjprnri9uxbI2oENTHsch2vLpyqM9SlvGmF0C60sNNF0TWc8aGQ8ljCru1ExhKL9M2o67Bqqj54g2mHPOrU/6JrBCHme0zgebvvFeWGHt2pfFSGrEqcIX97DR6/LbR90LXxeBhFIYm9MS2AMlOxt6st2291IwaEUeVEA5nA2KebgoYlkah8goGt7CFuF22Q==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: Jan Beulich <jbeulich@xxxxxxxx>, Kevin Tian <kevin.tian@xxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Fri, 04 Nov 2022 02:34:45 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHY1AgeJwq4WhKefkGyVS1JhkdRo64QKSwAgAG7SUCAAH/pgIAZrF0AgAClVGCAAR6tgIAAbuMg
  • Thread-topic: [PATCH v8 1/2] IOMMU/VT-d: wire common device reserved memory API

Hi Marek,

> -----Original Message-----
> 
> FWIW, most of the diff is just extracting loop body into a function, the
> only functional difference is a new called for this function, and moving
> one of the checks (MAX_USER_RMRR_PAGES enforcement) outside of it. So,
> my (biased) opinion is, it's rather low risk.

I think yesterday Jan has committed this patch :)

Kind regards,
Henry

> 
> --
> Best Regards,
> Marek Marczykowski-Górecki
> Invisible Things Lab

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.