Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757242AbdHYR2Z (ORCPT ); Fri, 25 Aug 2017 13:28:25 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:38066 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755082AbdHYR2X (ORCPT ); Fri, 25 Aug 2017 13:28:23 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 034E66025C Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=anjiandi@codeaurora.org Subject: Re: [PATCH] tpm/tpm_crb: Access locality for non-ACPI and non-SMC start method To: Jason Gunthorpe , Jarkko Sakkinen References: <1503029736-591-1-git-send-email-anjiandi@codeaurora.org> <20170822173956.zpqe4scdnv7plrhj@linux.intel.com> <20170824122630.75sxjmkj5f7p7tv5@linux.intel.com> <7478e964-6553-9d1e-3d8f-83b044a9a562@codeaurora.org> <20170825162139.s5oasztecntzg223@linux.intel.com> <20170825165309.GB5254@obsidianresearch.com> Cc: peterhuewe@gmx.de, tpmdd@selhorst.net, tpmdd-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org, linux-security-module@vger.kernel.org From: Jiandi An Message-ID: Date: Fri, 25 Aug 2017 12:28:21 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: <20170825165309.GB5254@obsidianresearch.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 781 Lines: 24 On 08/25/2017 11:53 AM, Jason Gunthorpe wrote: > On Fri, Aug 25, 2017 at 07:21:39PM +0300, Jarkko Sakkinen wrote: >> As I said before it would make much more sense to make code always deal >> with sm and remove flags completely. That would help maintaining code >> easier as new logic for TZ is introduced. > > Yes please > > Jason > Okay, I'll work on moving sm to priv and getting rid of flags from priv. Will submit a new patch with appropriate patch name and not a v2 of this patch since the scope of changes have changed. Thanks for your comments and feedback. -- Jiandi An Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc. Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.