Received: by 2002:a25:824b:0:0:0:0:0 with SMTP id d11csp236580ybn; Thu, 3 Oct 2019 04:27:00 -0700 (PDT) X-Google-Smtp-Source: APXvYqxdhMb0JM8FvjrWFinFD+G7dN1eTI4JbBkhhXPWvS3PPMUBEsZckWUuTYHP7Fmk2nsnw4xX X-Received: by 2002:a17:906:b804:: with SMTP id dv4mr7069257ejb.243.1570102019957; Thu, 03 Oct 2019 04:26:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1570102019; cv=none; d=google.com; s=arc-20160816; b=F8n1PVWsPwVpn2QXWdSADdqHGs1mMD8vZQ0s8NIhAg3cJD98XvkWNTborPZ9oT/itf gI9N4dTKq58ls11Dsj+UsHB7G3vrNQ1CjfJqZqSxwvc2PiXEUyUrTRcdfqIS1YmcGgkl cHUkNW1wMa9d2b0SGQ3PPg6UCMA9Cat6zYxJRWBXEYFPOD1odUkBiKocb/fAtMOrPKW3 llwyGaiSXVpHeUwXpOaqgVLxVdOd4f+kguly1qk8wxt7q4sknTWlaHP3p4q3QOYbpPAW xGQLS2vlzZQU6v5UskAyNhDxe2e0q7wxXDfucynJ4qMtQaB0OZYQoGbmXLvzUry9BrnS LiEg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:organization:in-reply-to :content-disposition:mime-version:references:message-id:subject:to :from:date; bh=s6RUbXMbaX49bH262wASATHUcKq1DqUcaKmGX9imuqg=; b=c6yPs8v0GRlSceeWpOEVrsr/LFSl5SwMd66Ifjnb73rTzaSzYXDBC/RvgzjkGMRgvP y/1eGqHfpakOCa9xP6jFMvSswScCcsLpsvglEEpxKo2lw/ADzr53ICSPskF0kVz1vKXs +PZlqnHD1fnPh2QO7rovyPTyIMhxSiOfBkcHlE/nhgPmjlYEbjuX1ud7gOXXN7Usrgw6 TijtebEKG7ZE8BuQHgHxDNk+yFGCbDXtb5gubO2tW1GaHh6BW5fvW20qjQFqH/s6nQbH +rpD7KUcXb4Az3b8M5xj1r+Ys39tpZeMKlwz3GAC8dheKiSTbntsM5ge+H7T4riLqzFX yPBA== 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 h42si1323885eda.90.2019.10.03.04.26.35; Thu, 03 Oct 2019 04:26:59 -0700 (PDT) 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 S1729989AbfJCLZn (ORCPT + 99 others); Thu, 3 Oct 2019 07:25:43 -0400 Received: from mga03.intel.com ([134.134.136.65]:51919 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725827AbfJCLZn (ORCPT ); Thu, 3 Oct 2019 07:25:43 -0400 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 03 Oct 2019 04:25:42 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.67,251,1566889200"; d="scan'208";a="198503176" Received: from jsakkine-mobl1.tm.intel.com (HELO localhost) ([10.237.50.161]) by FMSMGA003.fm.intel.com with ESMTP; 03 Oct 2019 04:25:39 -0700 Date: Thu, 3 Oct 2019 14:25:39 +0300 From: Jarkko Sakkinen To: Sasha Levin , Greg KH , linux-stabley@vger.kernel.org, Vadim Sukhomlinov , stable@vger.kernel.org, Douglas Anderson , Peter Huewe , Jason Gunthorpe , "open list:TPM DEVICE DRIVER" , open list Subject: Re: [PATCH 3/3] tpm: Fix TPM 1.2 Shutdown sequence to prevent future TPM operations Message-ID: <20191003112539.GB8933@linux.intel.com> References: <20191002131445.7793-1-jarkko.sakkinen@linux.intel.com> <20191002131445.7793-4-jarkko.sakkinen@linux.intel.com> <20191002135758.GA1738718@kroah.com> <20191002151751.GP17454@sasha-vm> <20191002153123.wcguist4okoxckis@cantor> <20191002154204.me4lzgx2l4r6zkpy@cantor> <20191003112442.GA8933@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20191003112442.GA8933@linux.intel.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Oct 03, 2019 at 02:24:42PM +0300, Jarkko Sakkinen wrote: > On Wed, Oct 02, 2019 at 08:42:04AM -0700, Jerry Snitselaar wrote: > > On Wed Oct 02 19, Jerry Snitselaar wrote: > > > On Wed Oct 02 19, Sasha Levin wrote: > > > > On Wed, Oct 02, 2019 at 03:57:58PM +0200, Greg KH wrote: > > > > > On Wed, Oct 02, 2019 at 04:14:44PM +0300, Jarkko Sakkinen wrote: > > > > > > From: Vadim Sukhomlinov > > > > > > > > > > > > commit db4d8cb9c9f2af71c4d087817160d866ed572cc9 upstream > > > > > > > > > > > > TPM 2.0 Shutdown involve sending TPM2_Shutdown to TPM chip and disabling > > > > > > future TPM operations. TPM 1.2 behavior was different, future TPM > > > > > > operations weren't disabled, causing rare issues. This patch ensures > > > > > > that future TPM operations are disabled. > > > > > > > > > > > > Fixes: d1bd4a792d39 ("tpm: Issue a TPM2_Shutdown for TPM2 devices.") > > > > > > Cc: stable@vger.kernel.org > > > > > > Signed-off-by: Vadim Sukhomlinov > > > > > > [dianders: resolved merge conflicts with mainline] > > > > > > Signed-off-by: Douglas Anderson > > > > > > Reviewed-by: Jarkko Sakkinen > > > > > > Signed-off-by: Jarkko Sakkinen > > > > > > --- > > > > > > drivers/char/tpm/tpm-chip.c | 5 +++-- > > > > > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > > > > > > > > What kernel version(s) is this for? > > > > > > > > It would go to 4.19, we've recently reverted an incorrect backport of > > > > this patch. > > > > > > > > Jarkko, why is this patch 3/3? We haven't seen the first two on the > > > > mailing list, do we need anything besides this patch? > > > > > > > > -- > > > > Thanks, > > > > Sasha > > > > > > It looks like there was a problem mailing the earlier patchset, and patches 1 and 2 > > > weren't cc'd to stable, but patch 3 was. > > > > Is linux-stabley@vger.kernel.org a valid address? > > > > No, did a resend :-( New version sent to stableish@vger.kernel.org. /Jarkko