[{"id":1803186,"web_url":"http://patchwork.ozlabs.org/comment/1803186/","msgid":"<f4e93b8e-cd8e-3c67-106d-652b5a60456e@linux.vnet.ibm.com>","list_archive_url":null,"date":"2017-11-11T17:36:49","subject":"Re: [Skiboot] [PATCH v2 00/14] libstb: simplify the initialization\n\tof cvc drivers","submitter":{"id":69305,"url":"http://patchwork.ozlabs.org/api/people/69305/","name":"Claudio Carvalho","email":"cclaudio@linux.vnet.ibm.com"},"content":"Patch series dropped. Superseded by this:\n\n[Skiboot] [PATCH 00/19] libstb: add support for secure and trusted boot \nin P9\n\nClaudio\n\nOn 31/08/2017 04:24, Claudio Carvalho wrote:\n> v2:\n>\n>   - new patch: remove macros used to force secure-mode and trusted-mode\n>\n> v1:\n>\n> This series moves the initialization of container-verification-code drivers to\n> stb.c with the intent to simplify the current initialization process and also\n> the addition of new drivers in the future.\n>\n> Claudio Carvalho (14):\n>    libstb/stb.c: remove macros used to force secure-mode and trusted-mode\n>    core/flash: extern function to get the name of a PNOR partition\n>    libstb/stb.c: change sb_verify() to use flash_lookup_resource_name()\n>    libstb/stb.c: change tb_measure() to use flash_lookup_resource_name()\n>    libstb: rename rom_driver_ops struct to container_verification_code\n>    libstb: rename drivers/romcode.* to cvc/c1vc.*\n>    libstb: rename drivers/sw_driver.* to cvc/c1vc_mbedtls.*\n>    libstb: move drivers/sha512.* to mbedtls/sha512.*\n>    libstb: clean up the force-secure-mode and force-trusted-mode from\n>      nvram\n>    libstb: move cvc initialization to stb.c\n>    libstb/stb.c: free cvc allocated memory\n>    libstb: check container version before using it\n>    libstb/stb.c: fix trustedboot abort\n>    libstb/stb.c: fix log messages\n>\n>   asm/Makefile.inc            |   2 +-\n>   asm/c1vc_entry.S            |  51 +++++\n>   asm/rom_entry.S             |  52 -----\n>   core/flash.c                |  10 +\n>   include/skiboot.h           |   1 +\n>   libstb/Makefile.inc         |   6 +-\n>   libstb/container.c          |  47 ++++-\n>   libstb/container.h          |  11 +-\n>   libstb/cvc/Makefile.inc     |  11 +\n>   libstb/cvc/c1vc.c           |  69 +++++++\n>   libstb/cvc/c1vc.h           |  31 +++\n>   libstb/cvc/c1vc_mbedtls.c   |  46 +++++\n>   libstb/cvc/c1vc_mbedtls.h   |  34 ++++\n>   libstb/drivers/Makefile.inc |   2 +-\n>   libstb/drivers/romcode.c    | 138 -------------\n>   libstb/drivers/romcode.h    |  24 ---\n>   libstb/drivers/sha512.c     | 480 --------------------------------------------\n>   libstb/drivers/sha512.h     | 141 -------------\n>   libstb/drivers/sw_driver.c  |  76 -------\n>   libstb/drivers/sw_driver.h  |  24 ---\n>   libstb/mbedtls/Makefile.inc |  11 +\n>   libstb/mbedtls/sha512.c     | 480 ++++++++++++++++++++++++++++++++++++++++++++\n>   libstb/mbedtls/sha512.h     | 141 +++++++++++++\n>   libstb/rom.c                |  55 -----\n>   libstb/rom.h                |  43 ----\n>   libstb/stb.c                | 407 +++++++++++++++++++++++--------------\n>   26 files changed, 1197 insertions(+), 1196 deletions(-)\n>   create mode 100644 asm/c1vc_entry.S\n>   delete mode 100644 asm/rom_entry.S\n>   create mode 100644 libstb/cvc/Makefile.inc\n>   create mode 100644 libstb/cvc/c1vc.c\n>   create mode 100644 libstb/cvc/c1vc.h\n>   create mode 100644 libstb/cvc/c1vc_mbedtls.c\n>   create mode 100644 libstb/cvc/c1vc_mbedtls.h\n>   delete mode 100644 libstb/drivers/romcode.c\n>   delete mode 100644 libstb/drivers/romcode.h\n>   delete mode 100644 libstb/drivers/sha512.c\n>   delete mode 100644 libstb/drivers/sha512.h\n>   delete mode 100644 libstb/drivers/sw_driver.c\n>   delete mode 100644 libstb/drivers/sw_driver.h\n>   create mode 100644 libstb/mbedtls/Makefile.inc\n>   create mode 100644 libstb/mbedtls/sha512.c\n>   create mode 100644 libstb/mbedtls/sha512.h\n>   delete mode 100644 libstb/rom.c\n>   delete mode 100644 libstb/rom.h\n>","headers":{"Return-Path":"<skiboot-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org>","X-Original-To":["incoming@patchwork.ozlabs.org","skiboot@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@bilbo.ozlabs.org","skiboot@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3yZ3xh4rwVz9sCZ\n\tfor <incoming@patchwork.ozlabs.org>;\n\tSun, 12 Nov 2017 04:37:12 +1100 (AEDT)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3yZ3xh3q08zDrLX\n\tfor <incoming@patchwork.ozlabs.org>;\n\tSun, 12 Nov 2017 04:37:12 +1100 (AEDT)","from mx0a-001b2d01.pphosted.com (mx0a-001b2d01.pphosted.com\n\t[148.163.156.1])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3yZ3xP36xNzDrLT\n\tfor <skiboot@lists.ozlabs.org>; Sun, 12 Nov 2017 04:36:57 +1100 (AEDT)","from pps.filterd (m0098394.ppops.net [127.0.0.1])\n\tby mx0a-001b2d01.pphosted.com (8.16.0.21/8.16.0.21) with SMTP id\n\tvABHYScu038353\n\tfor <skiboot@lists.ozlabs.org>; Sat, 11 Nov 2017 12:36:55 -0500","from e17.ny.us.ibm.com (e17.ny.us.ibm.com [129.33.205.207])\n\tby mx0a-001b2d01.pphosted.com with ESMTP id 2e5xkm4cvh-1\n\t(version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT)\n\tfor <skiboot@lists.ozlabs.org>; Sat, 11 Nov 2017 12:36:55 -0500","from localhost\n\tby e17.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use\n\tOnly! Violators will be prosecuted\n\tfor <skiboot@lists.ozlabs.org> from <cclaudio@linux.vnet.ibm.com>;\n\tSat, 11 Nov 2017 12:36:54 -0500","from b01cxnp22034.gho.pok.ibm.com (9.57.198.24)\n\tby e17.ny.us.ibm.com (146.89.104.204) with IBM ESMTP SMTP Gateway:\n\tAuthorized Use Only! Violators will be prosecuted; \n\tSat, 11 Nov 2017 12:36:50 -0500","from b01ledav006.gho.pok.ibm.com (b01ledav006.gho.pok.ibm.com\n\t[9.57.199.111])\n\tby b01cxnp22034.gho.pok.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP\n\tid vABHaoOJ45744210\n\tfor <skiboot@lists.ozlabs.org>; Sat, 11 Nov 2017 17:36:50 GMT","from b01ledav006.gho.pok.ibm.com (unknown [127.0.0.1])\n\tby IMSVA (Postfix) with ESMTP id 2ACDBAC03F\n\tfor <skiboot@lists.ozlabs.org>; Sat, 11 Nov 2017 12:37:41 -0500 (EST)","from [9.85.175.178] (unknown [9.85.175.178])\n\tby b01ledav006.gho.pok.ibm.com (Postfix) with ESMTP id CD262AC03A\n\tfor <skiboot@lists.ozlabs.org>; Sat, 11 Nov 2017 12:37:40 -0500 (EST)"],"Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=linux.vnet.ibm.com\n\t(client-ip=148.163.156.1; helo=mx0a-001b2d01.pphosted.com;\n\tenvelope-from=cclaudio@linux.vnet.ibm.com; receiver=<UNKNOWN>)","To":"skiboot@lists.ozlabs.org","References":"<1504164285-15095-1-git-send-email-cclaudio@linux.vnet.ibm.com>","From":"Claudio Carvalho <cclaudio@linux.vnet.ibm.com>","Date":"Sat, 11 Nov 2017 15:36:49 -0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.4.0","MIME-Version":"1.0","In-Reply-To":"<1504164285-15095-1-git-send-email-cclaudio@linux.vnet.ibm.com>","Content-Language":"en-US","X-TM-AS-GCONF":"00","x-cbid":"17111117-0040-0000-0000-000003C1156A","X-IBM-SpamModules-Scores":"","X-IBM-SpamModules-Versions":"BY=3.00008052; HX=3.00000241; KW=3.00000007;\n\tPH=3.00000004; SC=3.00000239; SDB=6.00944427; UDB=6.00476527;\n\tIPR=6.00724688; \n\tBA=6.00005688; NDR=6.00000001; ZLA=6.00000005; ZF=6.00000009;\n\tZB=6.00000000; \n\tZP=6.00000000; ZH=6.00000000; ZU=6.00000002; MB=3.00017970;\n\tXFM=3.00000015; UTC=2017-11-11 17:36:52","X-IBM-AV-DETECTION":"SAVI=unused REMOTE=unused XFE=unused","x-cbparentid":"17111117-0041-0000-0000-000007B631CD","Message-Id":"<f4e93b8e-cd8e-3c67-106d-652b5a60456e@linux.vnet.ibm.com>","X-Proofpoint-Virus-Version":"vendor=fsecure engine=2.50.10432:, ,\n\tdefinitions=2017-11-11_06:, , signatures=0","X-Proofpoint-Spam-Details":"rule=outbound_notspam policy=outbound score=0\n\tpriorityscore=1501\n\tmalwarescore=0 suspectscore=3 phishscore=0 bulkscore=0 spamscore=0\n\tclxscore=1015 lowpriorityscore=0 impostorscore=0 adultscore=0\n\tclassifier=spam adjust=0 reason=mlx scancount=1\n\tengine=8.0.1-1707230000\n\tdefinitions=main-1711110252","Subject":"Re: [Skiboot] [PATCH v2 00/14] libstb: simplify the initialization\n\tof cvc drivers","X-BeenThere":"skiboot@lists.ozlabs.org","X-Mailman-Version":"2.1.24","Precedence":"list","List-Id":"Mailing list for skiboot development <skiboot.lists.ozlabs.org>","List-Unsubscribe":"<https://lists.ozlabs.org/options/skiboot>,\n\t<mailto:skiboot-request@lists.ozlabs.org?subject=unsubscribe>","List-Archive":"<http://lists.ozlabs.org/pipermail/skiboot/>","List-Post":"<mailto:skiboot@lists.ozlabs.org>","List-Help":"<mailto:skiboot-request@lists.ozlabs.org?subject=help>","List-Subscribe":"<https://lists.ozlabs.org/listinfo/skiboot>,\n\t<mailto:skiboot-request@lists.ozlabs.org?subject=subscribe>","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"skiboot-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org","Sender":"\"Skiboot\"\n\t<skiboot-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org>"}}]