Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp1647514imu; Tue, 20 Nov 2018 23:04:24 -0800 (PST) X-Google-Smtp-Source: AFSGD/VyruWQXzRpcp1cj8vYpBV0n+8so8VCQVw8rjh9gjoQqQRCrkR1U57OM+bvttDXvTHWXCp1 X-Received: by 2002:a65:6392:: with SMTP id h18mr4972214pgv.107.1542783864229; Tue, 20 Nov 2018 23:04:24 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1542783864; cv=none; d=google.com; s=arc-20160816; b=usPx6knhzKV2a5F7OPdp0dxWzq9OfbunDzNL0stymscwYNCrdzTPtW+Gl+BsvVqkC+ BJvrV4koiUocmzzTf2VqeGzvsEbltkqtFmyrVsdqdOY0BmHVFQZFovxexlBBfxIZgORF vxpGy6KWDFJT/nSmH2L/IVie7MgEUkQUF7qQKLbdIfAohf4KtkL7EDm8XJh2WHXhSqHa ij7M4aM9DX+KPX+KHm0thDlqlfU0vmyWMgC5Ofvomk+qCbWLdlic9FuS4fzJI8VC2yMy U+IODhJ2oQsyf0A8A36wFXSLMzncVIos6vZprgHN0HHMXuA8viQ1DvK7fz/AoEplJ24a A/qw== 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:cc :to:from:date; bh=uj216rMMUWXYEWRi3JlHVbTv8pDre1z69hfaAuJMMu4=; b=Dxw4D2TORSojEpNelprdhcDcjti6WD4fSQStWkBjjxbyfqwAUltyu8Y1DYmD9mqQrv mLnujl6fVPYLAtcx2gvOBmCdeCVDC7/+fZ35xdrva2FgrHTEe02V4AG+KvSZKSRWy1Fr qU2TcihXSYxjFWy/1E/K+Lhky5Ggz1TNdvZauEGCOxecb4nKqLcDC/M0vjfS1XGiux4C 5bEGDkynCXrrSRwH0o22BWelqOBojmOyRFv8OXMtdgdUBMTOtXcEcSjjkn/oZqJBfbuX afs7qhClqfEbzmGmlIZcswGP0Of1+w3rufF8y5BKRN8x8QyWbKRxvDYHT9EHJStUfL1n jEgQ== 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 16-v6si49559020pfy.64.2018.11.20.23.04.09; Tue, 20 Nov 2018 23:04:24 -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 S1727589AbeKURS7 (ORCPT + 99 others); Wed, 21 Nov 2018 12:18:59 -0500 Received: from mga17.intel.com ([192.55.52.151]:9312 "EHLO mga17.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725939AbeKURS7 (ORCPT ); Wed, 21 Nov 2018 12:18:59 -0500 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga107.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 20 Nov 2018 22:45:46 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,260,1539673200"; d="scan'208";a="108047964" Received: from ncanderx-mobl.ger.corp.intel.com (HELO localhost) ([10.249.254.86]) by fmsmga004.fm.intel.com with ESMTP; 20 Nov 2018 22:45:40 -0800 Date: Wed, 21 Nov 2018 08:45:39 +0200 From: Jarkko Sakkinen To: Tadeusz Struk Cc: jgg@ziepe.ca, linux-integrity@vger.kernel.org, linux-security-module@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v5] tpm: add support for partial reads Message-ID: <20181121064539.GB3640@linux.intel.com> References: <154265371562.23494.10113870855345906523.stgit@tstruk-mobl1.jf.intel.com> <20181120124810.GB8813@linux.intel.com> <77b031b2-7fa2-f057-1e6a-70f0427ec6c4@intel.com> <20181120230759.GF8391@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: 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 Tue, Nov 20, 2018 at 03:13:59PM -0800, Tadeusz Struk wrote: > On 11/20/18 3:07 PM, Jarkko Sakkinen wrote: > >>>> + /* Holds the resul of the last successful call to tpm_transmit() */ > >>> This comment is cruft. > >> Do you want me to remove it? This is the comment you proposed. > > As I explained before it made sense before you made the remark that > > it can only get positive values i.e. the length. > > > So what do you want me to do with this one? I don't think it needs now any comment because the variable describes itself so well. /Jarkko