[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/3] gnttab: GTF_sub_page is a v2-only flag
- To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
- Date: Thu, 18 Feb 2021 14:22:49 +0000
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
- 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-SenderADCheck; bh=+j0B3tTIqYe23JUhyzIlAO1rYhkU6ufEHOyZT/ZzWXg=; b=At1P1c2iShszeFkOpN9tSgleqvEMmJhoFwSPfs0fBY8Idic88SxBjXyb2OtMwbozBk/+IA+3wMUfKeG/4yY5xPkYfoDZCNrmJoNhaXojflIstNNaA+akQF8c26WMNaWIvNStU59fWUO0+e2it9V9Epn6VrTa9yQGNCPAfh512yh1U2iWsZkUQxfpq06EvM5zNQ8crDbvha1YMRNKveCpDzxtPva12QFeefDbXab/petz+XJIoXU3qvCgHJdMY2D0jwxen7lh2VuOiuWIeMu2xCGYT7TSVGds+ZNeE1IWZnNO9c6bmp9x91bcTuth2Q1N5Hn2dYI+Mk0X3vXHn2ygCw==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ENhQGFML8nTZt0X7KD0LMERyNwfXz1aTfw5QB0s6aZunq4o+8RzUwFMjntuLh+Zl43R0UqM3xByULQPpq+NHXr/Z2vF5zquCaYSNAWnsALoiEnzi2U7Tgfo9eGtFcs1xgd1GmnDGkE2stflseI7Mvx3Ci6xpxUi2h1ZoyU3017peo15PLzEQ3BFgAWF/z96b9KnTpfKuYw4NoBjbx+lfz2K+wladantjk2n/uwEzeGP9UZ9Q79vsUcC4aPq8S4WFvXOGnv6N26lVnYp7nTH1B6it52zJPIr49ElxcEmIUPyzdwgysNAlruUobumOkmgOFL3wDmAySloCirN8FtshDg==
- Authentication-results: esa2.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
- Cc: George Dunlap <george.dunlap@xxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
- Delivery-date: Thu, 18 Feb 2021 14:23:19 +0000
- Ironport-sdr: oAQDl4fKCScSRfgYOmya+DHqRM0Xh4jDni0L6Md7I9GAwYKSz9gyFsX0+mQRpQHZPoYFlrnGZa XBrfVC73KThkpA9ndqDCPQfNLKZLzia+8ue9ngkpof0t39/ivTre8RZKUJvpnhU4WxSBSWr798 tK63G3B040fSBU17gmfzFnGGksBEr4JdsBH71N9YgtiJMegDqE6G1PvF/tLWTrVM2pV3bYNDk0 zHeRYYSZPQACMQlvHn5yURQWlmv5uRRCYIEaUGa7g3COs43mGFb4O2qisBtVReWormD7ncg/bp dsY=
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 17/02/2021 10:46, Jan Beulich wrote:
> Prior to its introduction, v1 entries weren't checked for this flag, and
> the flag also has been meaningless for v1 entries. Therefore it also
> shouldn't be checked. (The only consistent alternative would be to also
> check for all currently undefined flags to be clear.)
We recently had a similar discussion for the stable libs.
Whatever we do, an unexpected corner case needs to break. Checking for
all undefined flags up front is far cleaner - absolutely nothing good
can come for a guest which set GTF_sub_page with v1, and is expecting it
to work, and this way, we do all breaking in one go, rather than
breaking $N times in the future as new flags get added.
~Andrew
|