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:42:09 -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 29/29] drm/i915/gvt: merge gvt.c into kvmgvt.c
Message-ID: <20211102164209.GJ2744544@nvidia.com>
References: <20211102070601.155501-1-hch@lst.de>
 <20211102070601.155501-30-hch@lst.de>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20211102070601.155501-30-hch@lst.de>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?gsXF2qCVWMbooezl3a0nrJkTWsNxYTDZVQLWpZkdzwMpp9Covdw5HaiMbPpn?=
 =?us-ascii?Q?P5BA5Q3wSCFWPSGRvpxwso5q0Zz+HvcRG0wKrHc2QrLrONbzxJMxb5N/GyMy?=
 =?us-ascii?Q?A+l9pF59sBil8fXCR04ja2HSHyrD0gsyqyOJwZ/XGzDgVVcOvSv+B2VW5RBD?=
 =?us-ascii?Q?MjLCh7ye0BqhDGqqlk/Gk/D1wYwflDVk8jU2Qty+TsoWqnDixqSjCnycjax9?=
 =?us-ascii?Q?z9qytQUnuPVO0nItbJvKKke9TqTBGI5dPIYzZ0rGtWFR5bD5v8WlSFFvZUuK?=
 =?us-ascii?Q?EmWkGCJUCH4SBhNoPJMBSxNdK6pg8w6sqF2fvlwaDMn6QTGQzx+ubKsvp987?=
 =?us-ascii?Q?6fjpxZ42X7nNw0vAVs2ev7TsSZFoJ4/+3iZMKWQP7S7+FQBF//80e/CcyBSX?=
 =?us-ascii?Q?IuDlshSiUEp2RKrvHFbnEONKhXVxP9+PqFg3mbYVPZOrWqIVagCUrbJ4PBk6?=
 =?us-ascii?Q?waktIgYFCucqpVSbRIiVez/PHWWD80jJzN8Lc7BFv5HFj57pc3cjZ5JCPFyf?=
 =?us-ascii?Q?w3VswPAn566zEVtxBmd00Ri171SLhVyLUu42ON9dZ5yxtnAreeM0VX83Xetz?=
 =?us-ascii?Q?RugwJdGkPZsPs35exLwjZIPtiPCWwqjiVL7Tp7zuwj/mVtLcNj40tola3BsN?=
 =?us-ascii?Q?L+gPDhLhj9VGQbWI8hW4xUxLJC02gwZytmoKsMcus97sogqugN+P28FU4A/f?=
 =?us-ascii?Q?O6Cq3JfAjGEWAZCtVQuaFuZ6MUDeoKHyGeEQJBs47sOmqN7e2AQQJ7v3TUmk?=
 =?us-ascii?Q?MrSFMhMOhK/OQI11g6G/mFL/PSedL3ykITOG61cBGG6lZ122H4GX/zVvRIsH?=
 =?us-ascii?Q?CeWpFmN70TDd3nnxW9nQRqrhAfrCN7NasCt1Trg423Kny1VVhAsWXUwY5QXT?=
 =?us-ascii?Q?VUuE1zwOUtDzEBbiyH1Pasuy5JjjBULkXnGdsgvOlaOSuRqutEmdEmNwPGTY?=
 =?us-ascii?Q?IMK0Wxl3mOymB39XPerskQQ2W5nCDBkprdfme9NOR5wAgFRViIhhnWogjzdS?=
 =?us-ascii?Q?CdjRTtIFOxcbf8mrZqxAKp1AGIsmHXeoR/8YLE2DIPTgHSSNHBCAiXwo+StL?=
 =?us-ascii?Q?bTXBIZg3oDD7/7hZLDD/MY0NrivVHxLJ0wEh4LE9oB1tSNN5QRriMJEwf3x1?=
 =?us-ascii?Q?cgJDBkgnEfAFg+z5xLBu6yzkVdofHQyjKt//XiNhkHBpTJxp8B7wr4Ig99cN?=
 =?us-ascii?Q?H8ZQWzxCzAly6SbS9WJ4XYxwMllEGEPgZUt1G+BnnJUGU5HYHivu6nhdPv5B?=
 =?us-ascii?Q?Rr3BDe3/t/+sR1z6jW8OZATl/NIpcNURbeCJ2IihJXYhZI2qbXVg+WpL9OYZ?=
 =?us-ascii?Q?CR0wrJctlXgIxHr1FMotukzL4g31ZM9HtE8JNJ25deB3NIzb2l8l1MEehgxn?=
 =?us-ascii?Q?59M0EJH8jOs+bc4XQ5NmC3w0QuYFOxBGTr4n52YYkDqQ4XAOMDMOQnnJgNed?=
 =?us-ascii?Q?ycINk62JuM6UFhkV478JOwYobTCW5UB51WDomxemKndcYuO8rRkMunVH56ae?=
 =?us-ascii?Q?HtQA1sYcyf0rAZeFjbJLTPcGl451MNWxZHp/5MGYNghvTixl+7BuEARQ+YvN?=
 =?us-ascii?Q?w+4JenhP09uor1lBTOc=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 838471ab-a84e-432d-8f50-08d99e1fb757
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:42:10.9009
 (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: twO4aJclD8LzJ110paRYbXRU42K7D3yNzE907N1QlLEmLfB3IxdB2wpPaLgm6paf
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL1PR12MB5253
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Tue, Nov 02, 2021 at 08:06:01AM +0100, Christoph Hellwig wrote:
> The code in both files is deeply interconnected, so merge it and
> keep a bunch of structures and functions static.
> 
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
>  drivers/gpu/drm/i915/Makefile    |   1 -
>  drivers/gpu/drm/i915/gvt/gvt.c   | 291 -------------------------------
>  drivers/gpu/drm/i915/gvt/gvt.h   |   6 -
>  drivers/gpu/drm/i915/gvt/kvmgt.c | 264 +++++++++++++++++++++++++++-
>  4 files changed, 260 insertions(+), 302 deletions(-)
>  delete mode 100644 drivers/gpu/drm/i915/gvt/gvt.c

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

Jason