Cover Letter Detail
Show a cover letter.
GET /api/covers/810106/?format=api
{ "id": 810106, "url": "http://patchwork.ozlabs.org/api/covers/810106/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-imx/cover/1504615144-29770-1-git-send-email-gilad@benyossef.com/", "project": { "id": 19, "url": "http://patchwork.ozlabs.org/api/projects/19/?format=api", "name": "Linux IMX development", "link_name": "linux-imx", "list_id": "linux-imx-kernel.lists.patchwork.ozlabs.org", "list_email": "linux-imx-kernel@lists.patchwork.ozlabs.org", "web_url": null, "scm_url": null, "webscm_url": null, "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<1504615144-29770-1-git-send-email-gilad@benyossef.com>", "list_archive_url": null, "date": "2017-09-05T12:38:39", "name": "[v8,00/20] simplify crypto wait for async op", "submitter": { "id": 11198, "url": "http://patchwork.ozlabs.org/api/people/11198/?format=api", "name": "Gilad Ben-Yossef", "email": "gilad@benyossef.com" }, "mbox": "http://patchwork.ozlabs.org/project/linux-imx/cover/1504615144-29770-1-git-send-email-gilad@benyossef.com/mbox/", "series": [ { "id": 1574, "url": "http://patchwork.ozlabs.org/api/series/1574/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-imx/list/?series=1574", "date": "2017-09-05T12:38:39", "name": "simplify crypto wait for async op", "version": 8, "mbox": "http://patchwork.ozlabs.org/series/1574/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/810106/comments/", "headers": { "Return-Path": "<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>", "X-Original-To": "incoming-imx@patchwork.ozlabs.org", "Delivered-To": "patchwork-incoming-imx@bilbo.ozlabs.org", "Authentication-Results": [ "ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)", "ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"r2uHKEC/\"; dkim-atps=neutral" ], "Received": [ "from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xmmWY5Bqvz9sRV\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tTue, 5 Sep 2017 22:39:51 +1000 (AEST)", "from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dpD8t-0006OC-5k; Tue, 05 Sep 2017 12:39:47 +0000", "from foss.arm.com ([217.140.101.70])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dpD8p-0006Gr-3F; Tue, 05 Sep 2017 12:39:44 +0000", "from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249])\n\tby usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id BBCBC2B;\n\tTue, 5 Sep 2017 05:39:20 -0700 (PDT)", "from gby.kfn.arm.com (unknown [10.45.48.140])\n\tby usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id\n\tDDDE13F3E1; Tue, 5 Sep 2017 05:39:10 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe:\n\tList-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date:\n\tSubject:To:From:Reply-To:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:\n\tReferences:List-Owner; bh=F8aHb04x2oOMM57KrManKJ2ODmfJ3bwN9ng5ddUgWfI=;\n\tb=r2u\n\tHKEC/gMqQ/v8HzI09qdrZHdM2sifs+paM/b1a8nG75OFqtv3Rzg2MVWp5cNf8Qa7K/ZOaZ1ObsFE7\n\tun+ajYY6IxghC/OXthO7LQ5DyF9HyO80W3ewnSyfXXejw1aS90Sf1tKd9Fug6Rai4WuuxF9ZPdGv0\n\tFGORphUzTytXdAkSuhTndn2cKCIjhektwbZJtXW0cA09n9sIQedY3abPKVWUOnq8q5DdN0akT/NWh\n\t9ZPH4nZOr9vWdtmS8FKczQe72V/6kVwUNSltViSO4BHLGKBt1hAD9tHBeIOjbd1g/uuvqkP3UiViC\n\t6sxgAJca5xrjUPv0042Ol+ME/KfjKZw==;", "From": "Gilad Ben-Yossef <gilad@benyossef.com>", "To": "Herbert Xu <herbert@gondor.apana.org.au>,\n\t\"David S. Miller\" <davem@davemloft.net>,\n\tJonathan Corbet <corbet@lwn.net>, David Howells <dhowells@redhat.com>,\n\tTom Lendacky <thomas.lendacky@amd.com>, Gary Hook <gary.hook@amd.com>,\n\tBoris Brezillon <boris.brezillon@free-electrons.com>,\n\tArnaud Ebalard <arno@natisbad.org>,\n\tMatthias Brugger <matthias.bgg@gmail.com>,\n\tAlasdair Kergon <agk@redhat.com>, Mike Snitzer <snitzer@redhat.com>, \n\tdm-devel@redhat.com, Shaohua Li <shli@kernel.org>,\n\tSteve French <sfrench@samba.org>, \"Theodore Y. Ts'o\" <tytso@mit.edu>, \n\tJaegeuk Kim <jaegeuk@kernel.org>,\n\tSteffen Klassert <steffen.klassert@secunet.com>,\n\tAlexey Kuznetsov <kuznet@ms2.inr.ac.ru>,\n\tHideaki YOSHIFUJI <yoshfuji@linux-ipv6.org>,\n\tMimi Zohar <zohar@linux.vnet.ibm.com>,\n\tDmitry Kasatkin <dmitry.kasatkin@gmail.com>,\n\tJames Morris <james.l.morris@oracle.com>,\n\t\"Serge E. Hallyn\" <serge@hallyn.com>, linux-crypto@vger.kernel.org,\n\tlinux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tkeyrings@vger.kernel.org, linux-arm-kernel@lists.infradead.org,\n\tlinux-mediatek@lists.infradead.org, linux-raid@vger.kernel.org,\n\tlinux-cifs@vger.kernel.org, samba-technical@lists.samba.org,\n\tlinux-fscrypt@vger.kernel.org, netdev@vger.kernel.org,\n\tlinux-ima-devel@lists.sourceforge.net,\n\tlinux-ima-user@lists.sourceforge.net,\n\tlinux-security-module@vger.kernel.org", "Subject": "[PATCH v8 00/20] simplify crypto wait for async op", "Date": "Tue, 5 Sep 2017 15:38:39 +0300", "Message-Id": "<1504615144-29770-1-git-send-email-gilad@benyossef.com>", "X-Mailer": "git-send-email 2.1.4", "X-CRM114-Version": "20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ", "X-CRM114-CacheID": "sfid-20170905_053943_163610_C1E17A48 ", "X-CRM114-Status": "GOOD ( 15.33 )", "X-Spam-Score": "-6.9 (------)", "X-Spam-Report": "SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-6.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/,\n\thigh trust [217.140.101.70 listed in list.dnswl.org]\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]", "X-BeenThere": "linux-arm-kernel@lists.infradead.org", "X-Mailman-Version": "2.1.21", "Precedence": "list", "List-Unsubscribe": "<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>", "List-Archive": "<http://lists.infradead.org/pipermail/linux-arm-kernel/>", "List-Post": "<mailto:linux-arm-kernel@lists.infradead.org>", "List-Help": "<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>", "List-Subscribe": "<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>", "Cc": "Ofir Drang <ofir.drang@arm.com>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"us-ascii\"", "Content-Transfer-Encoding": "7bit", "Sender": "\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>", "Errors-To": "linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org", "List-Id": "linux-imx-kernel.lists.patchwork.ozlabs.org" }, "content": "Many users of kernel async. crypto services have a pattern of\nstarting an async. crypto op and than using a completion\nto wait for it to end.\n\nThis patch set simplifies this common use case in two ways:\n\nFirst, by separating the return codes of the case where a\nrequest is queued to a backlog due to the provider being\nbusy (-EBUSY) from the case the request has failed due\nto the provider being busy and backlogging is not enabled\n(-EAGAIN).\n\nNext, this change is than built on to create a generic API\nto wait for a async. crypto operation to complete.\n\nThe end result is a smaller code base and an API that is\neasier to use and more difficult to get wrong.\n\nThe patch set was boot tested on x86_64 and arm64 which\nat the very least tests the crypto users via testmgr and\ntcrypt but I do note that I do not have access to some\nof the HW whose drivers are modified nor do I claim I was\nable to test all of the corner cases.\n\nThe patch set is based upon linux-next release tagged\nnext-20170905.\n\nChanges from v7:\n- Turn -EBUSY to -EAGAIN also in crypto using net\n code which I missed before, as has been pointed\n out by Harsh Jain.\n\nChanges from v6:\n- Fix brown paper bag compile error on marvell/cesa\n code.\n\nChanges from v5:\n- Remove redundant new line as spotted by Jonathan\n Cameron.\n- Reworded dm-verity change commit message to better\n clarify potential issue averted by change as\n pointed out by Mikulas Patocka.\n\nChanges from v4:\n- Rebase on top of latest algif changes from Stephan\n Mueller.\n- Fix typo in ccp patch title.\n\nChanges from v3:\n- Instead of changing the return code to indicate\n backlog queueing, change the return code to indicate\n transient busy state, as suggested by Herbert Xu.\n\nChanges from v2:\n- Patch title changed from \"introduce crypto wait for\n async op\" to better reflect the current state.\n- Rebase on top of latest linux-next.\n- Add a new return code of -EIOCBQUEUED for backlog\n queueing, as suggested by Herbert Xu.\n- Transform more users to the new API.\n- Update the drbg change to account for new init as\n indicated by Stephan Muller.\n\nChanges from v1:\n- Address review comments from Eric Biggers.\n- Separated out bug fixes of existing code and rebase\n on top of that patch set.\n- Rename 'ecr' to 'wait' in fscrypto code.\n- Split patch introducing the new API from the change\n moving over the algif code which it originated from\n to the new API.\n- Inline crypto_wait_req().\n- Some code indentation fixes.\n\nGilad Ben-Yossef (20):\n crypto: change transient busy return code to -EAGAIN\n crypto: ccp: use -EAGAIN for transient busy indication\n net: use -EAGAIN for transient busy indication\n crypto: remove redundant backlog checks on EBUSY\n crypto: marvell/cesa: remove redundant backlog checks on EBUSY\n crypto: introduce crypto wait for async op\n crypto: move algif to generic async completion\n crypto: move pub key to generic async completion\n crypto: move drbg to generic async completion\n crypto: move gcm to generic async completion\n crypto: move testmgr to generic async completion\n fscrypt: move to generic async completion\n dm: move dm-verity to generic async completion\n cifs: move to generic async completion\n ima: move to generic async completion\n crypto: tcrypt: move to generic async completion\n crypto: talitos: move to generic async completion\n crypto: qce: move to generic async completion\n crypto: mediatek: move to generic async completion\n crypto: adapt api sample to use async. op wait\n\n Documentation/crypto/api-samples.rst | 52 ++-------\n crypto/af_alg.c | 27 -----\n crypto/ahash.c | 12 +--\n crypto/algapi.c | 6 +-\n crypto/algif_aead.c | 8 +-\n crypto/algif_hash.c | 50 +++++----\n crypto/algif_skcipher.c | 9 +-\n crypto/api.c | 13 +++\n crypto/asymmetric_keys/public_key.c | 28 +----\n crypto/cryptd.c | 4 +-\n crypto/cts.c | 6 +-\n crypto/drbg.c | 36 ++-----\n crypto/gcm.c | 32 ++----\n crypto/lrw.c | 8 +-\n crypto/rsa-pkcs1pad.c | 16 +--\n crypto/tcrypt.c | 84 +++++----------\n crypto/testmgr.c | 204 ++++++++++++-----------------------\n crypto/xts.c | 8 +-\n drivers/crypto/ccp/ccp-crypto-main.c | 8 +-\n drivers/crypto/ccp/ccp-dev.c | 7 +-\n drivers/crypto/marvell/cesa.c | 3 +-\n drivers/crypto/marvell/cesa.h | 2 +-\n drivers/crypto/mediatek/mtk-aes.c | 31 +-----\n drivers/crypto/qce/sha.c | 30 +-----\n drivers/crypto/talitos.c | 38 +------\n drivers/md/dm-verity-target.c | 81 ++++----------\n drivers/md/dm-verity.h | 5 -\n fs/cifs/smb2ops.c | 30 +-----\n fs/crypto/crypto.c | 28 +----\n fs/crypto/fname.c | 36 ++-----\n fs/crypto/fscrypt_private.h | 10 --\n fs/crypto/keyinfo.c | 21 +---\n include/crypto/drbg.h | 3 +-\n include/crypto/if_alg.h | 15 +--\n include/linux/crypto.h | 40 +++++++\n net/ipv4/ah4.c | 2 +-\n net/ipv4/esp4.c | 2 +-\n net/ipv6/ah6.c | 2 +-\n net/ipv6/esp6.c | 2 +-\n security/integrity/ima/ima_crypto.c | 56 +++-------\n 40 files changed, 314 insertions(+), 741 deletions(-)" }