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:07:24 -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 12/29] drm/i915/gvt: remove vgpu->handle
Message-ID: <20211102160724.GR2744544@nvidia.com>
References: <20211102070601.155501-1-hch@lst.de>
 <20211102070601.155501-13-hch@lst.de>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20211102070601.155501-13-hch@lst.de>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?fu7K3kuH1bgl2Atnyhw3cAxdSsa9eWYtr7JsUdo2hqsghm0x3fIjxGhP/W7k?=
 =?us-ascii?Q?Gt75qKWh13VXBx7ZNYGjyNWKroJnc4s4gDfFKg0NPAh0qOf9okShCRNH9fo4?=
 =?us-ascii?Q?aYXrr+Mxhtctte4ofxBz/5yN7fiY0dM28rNamSo6mQcjInyp/aBCUc98T9mX?=
 =?us-ascii?Q?cAUK2eXaIRI2SOHWjkNGY0QwGkHfC5RrTqe3uI1HhygsMBzmIcy6SfyXfdLM?=
 =?us-ascii?Q?atiHnYpANHvWFdRdsJHlB0vsRaV+COTv/M1TrQWSo9XhhpOP7el5jAX2uoHb?=
 =?us-ascii?Q?BXycU2MTF6RqeZtOjU3WGBvyDSjIlU7qlS/okYOHn6KeV1Ib9uM183NdhzMv?=
 =?us-ascii?Q?fVoXRLarI8bSLnlHLRBXiOaJzQM4JAbLf1bVANCXyrVBYkbzDSOlKSPfwEhP?=
 =?us-ascii?Q?Zlz5IyBjdPgBdlVpQxTRak5+Hy6CtD+MPFya2fAydDEVinc4zhG93SxNhmwN?=
 =?us-ascii?Q?+GkfNqapM6jFafUpdtGyBOLRV8dPBbUGNEDlxsifusN7rpxrk4jkTgm1JdEY?=
 =?us-ascii?Q?2G2330s4EkcO6RE1Qvs/VBF25lDZ7N3nyL7ko5hopIiLB7tL1EknoD1wz+BB?=
 =?us-ascii?Q?0UGmtqzT0oPE1qNgj35Yr87OpkhFkwaibWiWLLEbMKE8ph2JTxcYkJcvXqOU?=
 =?us-ascii?Q?Sf+fQYU4Ggprl0F8XFia0Vws3znBAAiwF3oRlg0yvG7mrW2QxmsjH+vBMAiI?=
 =?us-ascii?Q?BSR/Phdj+0qUhNyBUbHG9rxRBZGWrAcBqibfZvYvs+eAPjdcQgcJXOmtxgWc?=
 =?us-ascii?Q?vLQ7qWiCQtIOlYIjze3mojxYnAU/OYc5LA+BpR2SJVo4bkJZTLnlWCOUPgnJ?=
 =?us-ascii?Q?Q0PGwNRMXaKfPIr88RPUplH0r2WdjWkBWdvDdODS85HUKq0BdCg5ZJVNc3Di?=
 =?us-ascii?Q?crq4n0xsfKFukLrEymNNj2LdLWpvu5s9OJFeeq5oAUmpx+c0dMo0u2UYmLPk?=
 =?us-ascii?Q?pRpxv6hSp0Flzp/C97RGe03J8QaKRVE5s5E5oaacFttBvgogztJeVyXU/y6P?=
 =?us-ascii?Q?83p8sSe5sLsbfZML3G9eJk3Poy6x63WbWw3cWDjRVV1WRRDIDGW6ennfo4di?=
 =?us-ascii?Q?zZ0mS5XGGSVVE+Z8kOrr6dSve/dxOHfR+Byc7o0ramDGDN0h+xqPqfcKj2Tk?=
 =?us-ascii?Q?eOr2EPnQoRVt+DNVi78dfkGc2+7h1IdWW3DDJk4KuX0odtZHGCd8K6r6kX1z?=
 =?us-ascii?Q?logGzBKuLc7ay5jWu8AoxHLhhDjo9WIyZYqtJ4s4qGErgAEty3wHcm+mvts3?=
 =?us-ascii?Q?EimLBsj8bJoplOhbX9bSySoDj7+qYMUguj+xrrS1k+5bJ1YGpgjkaEp1zN1w?=
 =?us-ascii?Q?t/MRk+1d5CldC7wtfHk66M6niwrZxKFXv/GmpAmb7mMjSDbK0duhpShBisJ0?=
 =?us-ascii?Q?VPL8rLaqSqx0lK1PZGFoXsQH5pWN2aUDeppOWLxnjILJ5X+9Zi5hLca+4i5f?=
 =?us-ascii?Q?a+uOhA/5mV3MYJZg2kx7wWwBB4kofNQd1ViYzQEN/YMiHhn+ZZCkj3P66Apg?=
 =?us-ascii?Q?QKZKPrGaHWMctXduLc8PYS5PezxPKG6j+vuQDFU/JWcMGA9i+qyLqdV8PCMk?=
 =?us-ascii?Q?1rWcgKXjhPATwBtlhRU=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5e715fe6-f436-46b6-9269-08d99e1adcb8
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:07:26.0292
 (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: JZLm7vF6JLwMkLiXL4lLVq/yFQSypSSPZiEYe3qlSDvmaEtM9Zhnqdm3wPmm0do0
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL1PR12MB5110
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:44AM +0100, Christoph Hellwig wrote:
> Always pass the actual vgpu structure instead of encoding it as a
> "handle" and add a bool flag to denote if a VGPU is attached.
> 
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
>  drivers/gpu/drm/i915/gvt/gvt.h       |   3 +-
>  drivers/gpu/drm/i915/gvt/hypercall.h |  32 +++----
>  drivers/gpu/drm/i915/gvt/kvmgt.c     | 126 +++++++++------------------
>  drivers/gpu/drm/i915/gvt/mpt.h       |  20 ++---
>  drivers/gpu/drm/i915/gvt/vgpu.c      |   6 +-
>  5 files changed, 71 insertions(+), 116 deletions(-)

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