From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753252AbbLNSJm (ORCPT ); Mon, 14 Dec 2015 13:09:42 -0500 Received: from mga04.intel.com ([192.55.52.120]:61439 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752898AbbLNSJk (ORCPT ); Mon, 14 Dec 2015 13:09:40 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,428,1444719600"; d="scan'208";a="12982190" Subject: Re: [PATCH v2 0/5] crypto: add algif_akcipher user space API To: Marcel Holtmann , Stephan Mueller , David Woodhouse References: <1831785.BBs8Hj3CxY@myon.chronox.de> Cc: Herbert Xu , linux-crypto@vger.kernel.org, linux-kernel , linux-api@vger.kernel.org, David Howells From: Tadeusz Struk Message-ID: <566F051D.4080408@intel.com> Date: Mon, 14 Dec 2015 10:06:21 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On 10/26/2015 09:54 PM, Marcel Holtmann wrote: > Hi Stephan, > >> This patch set adds the AF_ALG user space API to externalize the >> asymmetric cipher API recently added to the kernel crypto API. >> >> The patch set is tested with the user space library of libkcapi [1]. >> Use [1] test/test.sh for a full test run. The test covers the >> following scenarios: >> >> * sendmsg of one IOVEC >> >> * sendmsg of 16 IOVECs with non-linear buffer >> >> * vmsplice of one IOVEC >> >> * vmsplice of 15 IOVECs with non-linear buffer >> >> * invoking multiple separate cipher operations with one >> open cipher handle >> >> * encryption with private key (using vector from testmgr.h) >> >> * encryption with public key (using vector from testmgr.h) >> >> * decryption with private key (using vector from testmgr.h) > > after having discussions with David Howells and David Woodhouse, I don't think we should expose akcipher via AF_ALG at all. I think the akcipher operations for sign/verify/encrypt/decrypt should operate on asymmetric keys in the first place. With akcipher you are pretty much bound to public and private keys and the key is the important part and not the akcipher itself. Especially since we want to support private keys in hardware (like TPM for example). > > It seems more appropriate to use keyctl to derive the symmetric session key from your asymmetric key. And then use the symmetric session key id with skcipher via AF_ALG. Especially once symmetric key type has been introduced this seems to be trivial then. > > I am not really in favor of having two userspace facing APIs for asymmetric cipher usage. And we need to have an API that is capable to work with hardware keys. If we would have something like this: diff --git a/include/uapi/linux/if_alg.h b/include/uapi/linux/if_alg.h index f2acd2f..02e6162 100644 --- a/include/uapi/linux/if_alg.h +++ b/include/uapi/linux/if_alg.h @@ -34,9 +34,12 @@ struct af_alg_iv { #define ALG_SET_OP 3 #define ALG_SET_AEAD_ASSOCLEN 4 #define ALG_SET_AEAD_AUTHSIZE 5 +#define ALG_SET_PUBKEY 6 +#define ALG_SET_PUBKEY_ID 7 in case of ALG_SET_PUBKEY the key will be provided by user space and in case of ALG_SET_PUBKEY_ID the PF_ALG layer will retrieve the key from the keyring using the ID provided form user space. Will this be ok with you Marcel and David? Thanks, -- TS