Received: by 2002:a25:c205:0:0:0:0:0 with SMTP id s5csp5584263ybf; Thu, 5 Mar 2020 03:22:19 -0800 (PST) X-Google-Smtp-Source: ADFU+vtf6/Uen/0MX3kCuFspnmZZdUXNDLP5XeZzf+rILb4b0DxF0MS+1xMLPLC3rpHxg+3Ya1ZF X-Received: by 2002:a9d:264:: with SMTP id 91mr6427079otb.216.1583407339535; Thu, 05 Mar 2020 03:22:19 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1583407339; cv=none; d=google.com; s=arc-20160816; b=0FiZSdw0V5ebGLCRzdxnLXPfggar6eYvKvmMfj0agiWhpl7LDQote4Ivca4glxYSau ZE0rtrS1+f1rzvP9LuRZivrq8FtmJyRw1x+w4aoduR9MG5DvIOPBukXou4TElitq10SK QRZMSYtzen4yuqzixO7TB9y/Yj3bCd2PnH/TIuLHfS7t283327INXpM8XxSK5t84BBcp GwL3uPOgMk52V34IOcEc9ariBGZHiG8iYghi8SfHVPXNN0QT9FyrCRh7sjr18QZ25MaW bkQI0+yc8r3yhcw/TCN+JEHHsf4Jy5f6UiVal3vmB+QzQP18ts/MmpJ5+aEJ9rH9Hhss nxSA== 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:mime-version :user-agent:organization:references:in-reply-to:date:cc:to:from :subject:message-id; bh=6No6XFNYaFLIJkRtq3Np6WG+P/3IwWY1rDeZkF00xs0=; b=RusXuWJQmkWQmQRTkRziRAhkakIMGl+uOq+tIpkHt7m11N2LprxoS555dqFYM0AfdZ JWpP2PCkI8SM2bZs8rzqfeol5czh8WIervv8nIBux2upoFtczruHRQJ288mnObIgyH2x tykWJkuUoPuAIBQO7cbZWZEjvUHVg4pASiiXO5C8J1DXKlIc+UdLcCXjvfnqPVdjPhOQ erlHlz2tndzXooMSOsFw1B60ijdvgOAUVptLV66XKyTsvWrMCm4QlGd0SkF5VXIg0Mo+ uAoAmEWHmhJrhdekl/p8iKkW3ETnLfvTbHV27XGMeoFqdxyaj0BghfgXoIt4zafd8giI ld6Q== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id l19si3146245oii.54.2020.03.05.03.22.07; Thu, 05 Mar 2020 03:22:19 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727370AbgCELVG (ORCPT + 99 others); Thu, 5 Mar 2020 06:21:06 -0500 Received: from mga03.intel.com ([134.134.136.65]:13005 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725953AbgCELVG (ORCPT ); Thu, 5 Mar 2020 06:21:06 -0500 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 05 Mar 2020 03:21:05 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.70,517,1574150400"; d="scan'208";a="234385081" Received: from unknown (HELO jsakkine-mobl1) ([10.237.50.161]) by orsmga008.jf.intel.com with ESMTP; 05 Mar 2020 03:21:02 -0800 Message-ID: Subject: Re: [PATCH v6 2/3] tpm: ibmvtpm: Wait for buffer to be set before proceeding From: Jarkko Sakkinen To: Stefan Berger , linux-integrity@vger.kernel.org Cc: aik@ozlabs.ru, david@gibson.dropbear.id.au, linux-kernel@vger.kernel.org, nayna@linux.vnet.ibm.com, gcwilson@linux.ibm.com, jgg@ziepe.ca, Stefan Berger Date: Thu, 05 Mar 2020 13:21:03 +0200 In-Reply-To: <20200304132243.179402-3-stefanb@linux.vnet.ibm.com> References: <20200304132243.179402-1-stefanb@linux.vnet.ibm.com> <20200304132243.179402-3-stefanb@linux.vnet.ibm.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.35.92-1 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2020-03-04 at 08:22 -0500, Stefan Berger wrote: > From: Stefan Berger > > Synchronize with the results from the CRQs before continuing with > the initialization. This avoids trying to send TPM commands while > the rtce buffer has not been allocated, yet. > > This patch fixes an existing race condition that may occurr if the > hypervisor does not quickly respond to the VTPM_GET_RTCE_BUFFER_SIZE > request sent during initialization and therefore the ibmvtpm->rtce_buf > has not been allocated at the time the first TPM command is sent. > > Fixes: 132f76294744 ("Add new device driver to support IBM vTPM") > Signed-off-by: Stefan Berger Reviewed-by: Jarkko Sakkinen /Jarkko