From:   "Bae, Chang Seok" <chang.seok.bae@intel.com>
To:     "Hansen, Dave" <dave.hansen@intel.com>
CC:     Linux Crypto Mailing List <linux-crypto@vger.kernel.org>,
        "dm-devel@redhat.com" <dm-devel@redhat.com>,
        Herbert Xu <herbert@gondor.apana.org.au>,
        Eric Biggers <ebiggers@kernel.org>,
        "Ard Biesheuvel" <ardb@kernel.org>,
        the arch/x86 maintainers <x86@kernel.org>,
        "Lutomirski, Andy" <luto@kernel.org>,
        "tglx@linutronix.de" <tglx@linutronix.de>,
        "bp@suse.de" <bp@suse.de>,
        "dave.hansen@linux.intel.com" <dave.hansen@linux.intel.com>,
        "mingo@kernel.org" <mingo@kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Williams, Dan J" <dan.j.williams@intel.com>,
        "Gairuboyina, Charishma1" <charishma1.gairuboyina@intel.com>,
        "Dwarakanath, Kumar N" <kumar.n.dwarakanath@intel.com>,
        "Shankar, Ravi V" <ravi.v.shankar@intel.com>
Subject: Re: [PATCH v5 00/12] x86: Support Key Locker
Thread-Topic: [PATCH v5 00/12] x86: Support Key Locker
Thread-Index: AQHYB/pCFarBJFjgXkafl7Hx49PA1KxhhmMAgAAFFYA=
Date:   Thu, 13 Jan 2022 22:34:26 +0000
Message-ID: <51D8F2DE-404E-4BED-8CE5-4DB1D2243556@intel.com>
References: <20220112211258.21115-1-chang.seok.bae@intel.com>
 <215bc107-aa47-62bd-65ad-f76e65ee75d5@intel.com>
In-Reply-To: <215bc107-aa47-62bd-65ad-f76e65ee75d5@intel.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?9k0LxA6w67rkJREQ3peII+DFkdCQWLvaJi636oBbtPLThtO+CsZGqS5Pjnwa?=
 =?us-ascii?Q?PJAjJ9KHmVktBS+uiLQPOnPLYlMmRe9yo65iCcc/fsxfVKrFejNMCb36ATkS?=
 =?us-ascii?Q?jRT+r+qUmcQnlzr1su8IYYPs0eBQX7LFffs9XwinuVTPsCIQquBRFlW8CODm?=
 =?us-ascii?Q?G8q5cAj/muCeb+P41263gsUBfeA2z7OzGgUJ9fdDmyuP0OI0I6kqQd621qMS?=
 =?us-ascii?Q?RL1DJxUYCQHf4OmBqAhbLOe/mxrLyYILOq6YD4UR7H5s7InR1fTfkZxtCOfH?=
 =?us-ascii?Q?t/6Hm5Db0SphNaZ1eW8gUJ8JxelQ/cNa2C00Leg612lWxuUae5JD8cYGXY1Q?=
 =?us-ascii?Q?LUDNyizaaN9ymgU+5mbeCSO+eWjdx7ZXQ2EA0/1u075sIkkdJXp2BeLZF6HF?=
 =?us-ascii?Q?cpKmWVcKPV+PKneMxvF16GMazdnuckamxVb/DTXrKNNE1RReEp24QdInLYbT?=
 =?us-ascii?Q?xuTwDCOi36xl/7T51rg6UNpu6Sonh+2pyTj3Z+cmcSYfA1KDR4j4iWWt42hl?=
 =?us-ascii?Q?LuPV1JUHMtA26g7GIJ7iYgU4u9L0Jw9cCtRIYVrQqkSemFyFhhXuVuxuWWpx?=
 =?us-ascii?Q?p1655EObVWQ4JtT5jZ8oP7iUr5hsoSvM0/rv/85O6GQ41bP7RUB18ATAzTuJ?=
 =?us-ascii?Q?kTduk56GlSNaQGWY39OFP0W1Q4M5eaKs1wQ1nKp1Bu50vtLTKmSCmhbj7SvJ?=
 =?us-ascii?Q?CMG+TV9aE8l/bDIjmvTi8YTMRNZMbFC5DY+RtVMz9/2JtaJ3p+eLqGYgtaAF?=
 =?us-ascii?Q?X/E+JhqY83R0UxZ6eKwtckX2hsxJ1uXEjwyrsiXFaiFb/Fl1KQGhyhDMAhR/?=
 =?us-ascii?Q?4ChzKRnJ3Jj0XuYrsGSOU1o/GY3U0P9CyHGmsTLOnjWJPw4tJ4Kyc/18AJn4?=
 =?us-ascii?Q?NRmr4D/8aUUfSjvCeYdJPhWudrqXdRKrh2LoAKEDFEeW74g1wHxNdvpkuTlj?=
 =?us-ascii?Q?8ep27MX/op+D6Z09zFhoVyjqYaA1rjGbrPPBDJYbQ4IUKadhLNOeMH5cd+Hf?=
 =?us-ascii?Q?fPVnPYU6ahzWt43/q5kxtwEfZxeauoVq31vw4+9R4mtEhWQkNLDfeAHXHqjS?=
 =?us-ascii?Q?qtRR3H9q2okgCmIOZF3dVwmx9K9QvbqzlKMDbGqZT6e6+nzpdagUO/qY09Ha?=
 =?us-ascii?Q?0d+1ztKzlRlurmwKmF8TgOHbgrIiZWtyrbkgqpSWhrzKuWxWVz1I9DbzE6/D?=
 =?us-ascii?Q?UdJJFxtyAOjaJr+q7ngp0FmznkwZ/krzGI8LZV+ziawEB7zXC0RWSu+WMMdn?=
 =?us-ascii?Q?r6lZky0TTh4zN8P5KwzYGqJBQuxyuzMpHSdXCKVKoBJKGLrheDzh/KrA5PH9?=
 =?us-ascii?Q?LvGeooYsC8pjvOpAUYPWC0u3sEjgx85dhdj4vC7LkIJ4Q3sw+ZbEDaZ9NslB?=
 =?us-ascii?Q?A2VBH6B8IZVAmTC/fl7h+8tyEFscj/HOo8FIIsNXub/sAuR1zZldz0EkWpmd?=
 =?us-ascii?Q?eYzWA8D1kcqRykMDTZojQKZCyn8PEdpnnc8IpfyNvzCQxiUb6sgbdoTFPNFr?=
 =?us-ascii?Q?mwISMgkGwmb+Y+lqps/U4H8QiVg9jSFsS6K0VD01gEMIpH6YG+x/8kEVuJ02?=
 =?us-ascii?Q?h+vkJSDmb9OFHcJ/0aG7TYmimKLpsao0t8Iz1Q17t79bkQwSBPbGSkB9YDLi?=
 =?us-ascii?Q?S2yxc7IDjDIDb5e2r4oDIfHA7+hNSGVBCviuJmuCduIz6mRdlIN1wm8XqtDE?=
 =?us-ascii?Q?M3TwECkIf3bZzhptVxuSuuBHRtI=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <1D0ECD449A0C3D47899F823B5A473FEF@namprd11.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB4855.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b012e708-b925-456d-db07-08d9d6e4db49
X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Jan 2022 22:34:26.8155
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 2pMybOAZQVFRJDTnRUfrxx/O1C/4UYxxNetwcYX4oi5Ww1FJc328OuUDsquZKNKMb2lE+Rr3NQtIqaxMxg1SLfnLD8Vri0AVmGaGa9lUqlw=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB3963
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-crypto.vger.kernel.org>
X-Mailing-List: linux-crypto@vger.kernel.org

On Jan 13, 2022, at 14:16, Hansen, Dave <dave.hansen@intel.com> wrote:
> On 1/12/22 1:12 PM, Chang S. Bae wrote:
>> A couple of other things outside of these patches are still in progress:
>> * Support DM-crypt/cryptsetup for Key Locker usage (Andy Lutomirski)
>>  [2].
>> * Understand decryption under-performance (Eric Biggers and Milan Broz)
>>  [3][4].
>=20
> I really like when contributors are clear about why they are posting
> their series and what their expectations are.  This posting leaves me a
> bit confused as to what you expect the maintainers to do.
>=20
> Should the maintainers ignore this series until those in-progress things
> are done?  Or, do you expect that this could be merged as-is before
> those are resolved?

Ah, right. Yeah, this is not super clear about that.

I think it makes sense to clarify those two points -- performance implicati=
on
and user interaction in the usage case, before considering this feature
support in the mainline.

But I wanted to address feedback on the patches with this posting.=20

Hopefully, this clarifies the status of this series.

Thanks,
Chang