Received: by 2002:a25:ef43:0:0:0:0:0 with SMTP id w3csp128474ybm; Wed, 27 May 2020 21:44:31 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyPn7gSHE/+eD+F+Ccqr5WB4JH4DTZtKT3oEJQu6CbkrQWiu20WTDnAZcP96+S2AUDGc8qh X-Received: by 2002:a17:906:6bc5:: with SMTP id t5mr1308697ejs.459.1590641071233; Wed, 27 May 2020 21:44:31 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1590641071; cv=none; d=google.com; s=arc-20160816; b=w74cnLuiHjgQ+ROmtxR2BLXwBoqEZoGAdhr6qog8m2dCHwEVmKx5mu0CqEQysEJLLt 4WuvZc9/Kcf01oA6IGmApuuY62LKumylsJb6e/vTGZlCbYiuxZ53NEHgXxcmW31B/Qe+ NuKcYNgN4EriFhZKz/QYCueDLCVdhE2bC87bHc3FJ72h0c1m8UyilbDY/DGsSON5ZmF+ Fv5Bt7xSK8IqdSGxcGjSsieSP7P9tg2jJonGLQR6rYx//NZl1tNS6izomlYbHCRdlhO6 I7PaPHhTaSZkWcnrBqDP8Xa6DBAocDGIdMzpqP5PJpX+lNoHp1pZC0cdIEEDkbPmXIIS x5aQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature; bh=SEFEjOeVaCb6nRVJJoHwrwcnuKr7Yd2WmIU8A9Wz24w=; b=dmVZEZT7fdz94KdgcunlIiFuiMa44cdTpxxBp1jW3A/8F3ftAIRmruat8O9vhg7601 ys3WZuWGgw6dGQNCMgf1s4WAFEtqEC3gand7OthKVtBBghxIfLD+2LgEDAp7sp7n5HNJ cyoZ2TzpE7mwjgWT1lX7Vk4U8Mw5WxtgSkZrwtYcAs0w2EE91ySUhOfVgFtgkzLP6RTJ SNGCIxHkke6j6uDc78kgVazUpYcPJpahz3nlfglfuaUoa+MReNWpmzethybuiZclwhEt wnqrB96UL8OdmzOnwOOXuPmmhiGU6FHDLDuFEEFgsSwrsGhCC9BIuGMB7Xn3ONYJqSAi GaKg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=gUMu+Kpz; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id u20si3032007ejg.680.2020.05.27.21.44.08; Wed, 27 May 2020 21:44:31 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=gUMu+Kpz; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727030AbgE1Ek3 (ORCPT + 99 others); Thu, 28 May 2020 00:40:29 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35322 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725298AbgE1Ek2 (ORCPT ); Thu, 28 May 2020 00:40:28 -0400 Received: from mail-pl1-x641.google.com (mail-pl1-x641.google.com [IPv6:2607:f8b0:4864:20::641]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 59ED5C05BD1E; Wed, 27 May 2020 21:40:27 -0700 (PDT) Received: by mail-pl1-x641.google.com with SMTP id x10so11011973plr.4; Wed, 27 May 2020 21:40:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=SEFEjOeVaCb6nRVJJoHwrwcnuKr7Yd2WmIU8A9Wz24w=; b=gUMu+KpzmcRaYBU3gFRGvFKQqtwZpCg/gRJKUF0aRETBDeFJ7Ei88kCtWIUF1VzI/N DUXzKkdoRhcH1MC3CsrigOaqUpojb/ol9B1tL5RY4VXO6UPyGb6xwamjluCvDtbRzyhp gpHUXBDBZtMYK1qlCvFSDoSEmlTlboRJxvRFN8JmfV8orlCJwGKMeXKZ3s31Mb20qHXH JFIs+4cLo7FXGjYkoM4Nvrj0d7+HVq/PUNdJFLifxie0t7YJlVq3eYbPMffgPS8zcj9P 9x+X0ERXkoa87dOfsTKOS1RR4sfGzzgHKxoZbUV/UC4Cu4qBlpgcNX1ehhKVJcol8Rsr B+Tg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=SEFEjOeVaCb6nRVJJoHwrwcnuKr7Yd2WmIU8A9Wz24w=; b=HRdy11BAVkmmSDhxWgFwbzv/HZj94nedyIFq3nhM+thQxC9g0ChWQRbFM3r5NeTenh WNymQRm13VUg0ENnZ08rQ0wiogTU/hk0Pl6+BOJFx7t+MK7bTc3HZ5X1MTaZKd0k+oxz gaBfZtPxJE9em1fWzIvid0pdXtTJdcgEJ9/DTV/vocp6TYzol+XbbENzj2BDPlEgGH5O bFV3X2R5BrB+9g5x6YLHwiyK9sDyk312cVUhsMYwT0uzdaul0AAoXb3vguUhYZdpCokH DWJ6y0wq02rFAvCS120dPpKLOngCPT2jHhje+pfDGtUFmVLOJj0kedQ3A5x1oClqcGmR 2Reg== X-Gm-Message-State: AOAM53098QPHuCQ/Dz82hOFRJhqFSlQyjsq6K5jBfDMOdFG2Gpt943JP cQYIbsugFVYjTJszOl74Kqw= X-Received: by 2002:a17:902:9b88:: with SMTP id y8mr1754669plp.171.1590640826833; Wed, 27 May 2020 21:40:26 -0700 (PDT) Received: from [192.168.1.5] ([75.164.7.205]) by smtp.gmail.com with ESMTPSA id i29sm3450648pfk.38.2020.05.27.21.40.25 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 27 May 2020 21:40:26 -0700 (PDT) Subject: Re: [PATCH] tpm: Revert "tpm: fix invalid locking in NONBLOCKING mode" To: Jarkko Sakkinen Cc: James Bottomley , Mario Limonciello , Peter Huewe , Jason Gunthorpe , Arnd Bergmann , Greg Kroah-Hartman , linux-integrity@vger.kernel.org, linux-kernel@vger.kernel.org, Jeffrin Jose T , Alex Guzman References: <20200526183213.20720-1-mario.limonciello@dell.com> <1590520454.11810.40.camel@HansenPartnership.com> <20200528003040.GA4781@linux.intel.com> From: Tadeusz Struk Message-ID: Date: Wed, 27 May 2020 21:40:25 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.8.0 MIME-Version: 1.0 In-Reply-To: <20200528003040.GA4781@linux.intel.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 5/27/20 5:30 PM, Jarkko Sakkinen wrote: >> This won't help if the message is read by an async tcti. If the problem lies >> in the chip get locality code, perhaps this could help to debug the root-cause >> instead of masking it out in the upper layer code: > What is TCTI and async TCTI? Not following. TPM Command Transmission Interface (TCTI) as defined by TCG in https://trustedcomputinggroup.org/resource/tss-tcti-specification/ the reason we added the O_NONBLOCK mode was to satisfy the TCG spec for async TCTI. Thanks, Tadeusz