Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Tue, 2 Nov 2021 13:17:31 -0300
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     Christoph Hellwig <hch@lst.de>
Cc:     Jani Nikula <jani.nikula@linux.intel.com>,
        Joonas Lahtinen <joonas.lahtinen@linux.intel.com>,
        Rodrigo Vivi <rodrigo.vivi@intel.com>,
        Zhenyu Wang <zhenyuw@linux.intel.com>,
        Zhi Wang <zhi.a.wang@intel.com>,
        intel-gfx@lists.freedesktop.org,
        intel-gvt-dev@lists.freedesktop.org,
        dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 21/29] drm/i915/gvt: devirtualize
 ->dma_{,un}map_guest_page
Message-ID: <20211102161731.GA2744544@nvidia.com>
References: <20211102070601.155501-1-hch@lst.de>
 <20211102070601.155501-22-hch@lst.de>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20211102070601.155501-22-hch@lst.de>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?QEg+3kHz4ccbNaQHBt6TQAnq+1BFIZ/QaJdTC30fJ12Ourf3huRL6d1PUZOl?=
 =?us-ascii?Q?JMUWlrZXhSbSKJjnnUO6rdFDhCWaPGj377XZl4zOOjAo/q0DwzZ+BTTrobGV?=
 =?us-ascii?Q?ai7qLlkeM+wgeSax+y70ki7lnXba3IHjvPS2HizS4lfe//mBPhd5+rUtyLba?=
 =?us-ascii?Q?MwkNw7TLWq38o4RvobLbGvWM51CJC9DQVR4YXpywBBZR9I7t6nGmOyD90FKE?=
 =?us-ascii?Q?tGEFAPdOwB8F2/wAKVr1xM/1YLRf1/l1v6/nIQ7b16w1FPJ2b0zYxKfuww7b?=
 =?us-ascii?Q?Aov2huCpR+4Q8sMZ5gij7Gunb4bAF9WyK0VYEL6pEuYYYanbxnXSSrmhkqw7?=
 =?us-ascii?Q?hGGOBniNk7qty0mIjvcvF74ani9xZHitEweRnJXRIZ2k6vSNcYYdxfIReUfx?=
 =?us-ascii?Q?PFs4uxeMs7dIhCbNvvDFvBC1JYae/i0pzp8So/tR+k4QI/Nf8+72GY4i3VXL?=
 =?us-ascii?Q?kaxP5wJoSVU4bKllTt+I70nn+wSqw8NWq4vkGMbREtc/dXIY9vn4vYTdBh55?=
 =?us-ascii?Q?Yxr71Dd0bFoDfwxEDM5pqdAN4LnRoULasrEdF5DrdKZp3yZ62Av2L3rJMpXk?=
 =?us-ascii?Q?0nSBCNX7HAzagQOpPsqbWWQb4wpYx2nr6ClXc/JwqleJJRzgowuMm8Slxh43?=
 =?us-ascii?Q?3uvKVkJsU+hkt4s1ORuEKa+knUdp52jMum1fd/TwDWmRwk1oMGsjJnUGIKyD?=
 =?us-ascii?Q?l0Vn8tDWqwUgIxuph4k/YPT+ak65iUKhaAD4eyrGy9kdkDuJ9RpXTFmyU3/E?=
 =?us-ascii?Q?wGCaOBg+bQLmRC9BczBVr/BrPEOK193agY06d8s3aMOH8gPEghOdyw7udOr/?=
 =?us-ascii?Q?hIpJFEZmyLTQQeJR80Pgynuxzy4COCIhlsrTsulJOum8yBSUlP5b1weG0rNE?=
 =?us-ascii?Q?4+XCqbv/sAz8opb2JNPa/Rprdr6XtKJLJQowEVJ2SFd5apBogzJ6JTMZLSYt?=
 =?us-ascii?Q?2HMTcJpP0dY50sRXaBLbzQN5ri8/dX6B0kaQMijVAOMLmfwdcLDpu3QfEeiy?=
 =?us-ascii?Q?Z+Sj6bHvua9rFoS7dNiHTOpt2AvJxZ/PwW23Y/z8Z8OV5LUb77sJgvOq4NIf?=
 =?us-ascii?Q?aSyGxMetj/TvUvwlxTbyVRIX5e0FE7sZG4UfPNLAJgrw9T16ikNx/qej+akg?=
 =?us-ascii?Q?5dqHKWqXjslPv0CQdBw2Jl9vn7t3CPjy8DhCvsP/eq+gtq1cCmyMaG9cRpSn?=
 =?us-ascii?Q?iDyGpuBHXAXbG3ztntp+WrQdk5WrJk4JPpCAweZEYb5k/mZvO1IX+uqU/fgv?=
 =?us-ascii?Q?Ghu3khcTmTL/0jRmsvmWjiiC59ee3T4tM3htlwjnVRW3JS9mF+q5ZcM4piNm?=
 =?us-ascii?Q?fV3zEZ1x2VUseHsmdYH5MYWol1YrNFBqCJogfGnsbSoZMxbQmSydA3BJYR74?=
 =?us-ascii?Q?jzXMRX2FYZtNZpAdg3e2Y2WL28xOU4zEEoTYPlOQBXQ9p/JubCAGwzgYqFtW?=
 =?us-ascii?Q?YLiWTb++ploUnf8OmqAxulItNkloLsJjVxvz/KVRbfXYFni08WnRYXKwZ9fJ?=
 =?us-ascii?Q?bFm1BiXnyxbtTTxbjk3xZqPgN9J25J3lzX5TkLks/nbm2Kv0XGSmzlXl20Su?=
 =?us-ascii?Q?UPccDKUn4eSkGb+JxDo=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5cf5aecc-5cff-474d-a1c3-08d99e1c4635
X-MS-Exchange-CrossTenant-AuthSource: BL0PR12MB5506.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Nov 2021 16:17:32.5272
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: HrVWYnVSzaLodaVg8clqNt9tLt0hWNKgvx7abcymnlSi9fkXeaCQ9NOwg11oK46T
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL1PR12MB5364
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Tue, Nov 02, 2021 at 08:05:53AM +0100, Christoph Hellwig wrote:
> Just call the functions directly.  Also remove a pointless wrapper.
> 
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
>  drivers/gpu/drm/i915/gvt/dmabuf.c    | 10 ++--------
>  drivers/gpu/drm/i915/gvt/gtt.c       | 20 +++++++++----------
>  drivers/gpu/drm/i915/gvt/gvt.h       |  4 ++++
>  drivers/gpu/drm/i915/gvt/hypercall.h |  5 -----
>  drivers/gpu/drm/i915/gvt/kvmgt.c     |  6 ++----
>  drivers/gpu/drm/i915/gvt/mpt.h       | 29 ----------------------------
>  6 files changed, 17 insertions(+), 57 deletions(-)

Reviewed-by: Jason Gunthorpe <jgg@nvidia.com>

Jason