From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.9 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 653F3C432C3 for ; Thu, 28 Nov 2019 00:14:13 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 3BF0B2075C for ; Thu, 28 Nov 2019 00:14:13 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="C9hGflyA" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726947AbfK1AOM (ORCPT ); Wed, 27 Nov 2019 19:14:12 -0500 Received: from us-smtp-delivery-1.mimecast.com ([207.211.31.120]:55202 "EHLO us-smtp-1.mimecast.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727071AbfK1AOM (ORCPT ); Wed, 27 Nov 2019 19:14:12 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1574900051; h=from:from:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=bI3FgBWOF7wCQIIRv2T8Vht0Y2ID7iTwByZG4uRF2TY=; b=C9hGflyA8dPTSRY8yNzI/haK/B942652U+THL98mFvtyOM1jdpQuovldgOTnahoOdOibjL B/313YeumbOihO15FZscZTBK6Vs+bQN2Twx9oxup8LyjHGt422SVyNqUNXYhAJKp9cxw6B /OY573Hs0HbDXAap2QbxvO3X83OUhJs= Received: from mail-pg1-f198.google.com (mail-pg1-f198.google.com [209.85.215.198]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-201-fNT15UPwOySjOHj6gIGeqA-1; Wed, 27 Nov 2019 19:14:08 -0500 Received: by mail-pg1-f198.google.com with SMTP id k7so13664738pgq.10 for ; Wed, 27 Nov 2019 16:14:08 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:reply-to :mail-followup-to:references:mime-version:content-disposition :in-reply-to; bh=81TflNEkIggalSb9xJoZHH5dIwhWptMNuIwNrPhaU8E=; b=i8rY2JvbxMag5p0+gGYVecST1cwQmxqMGaT5LNkwgyo+jttLU2wuQUTSM6ZIHeMlZX 7TWxICMxKTWGWv2pzmN0kRLIUwrd7FS9er1xvq13iMLRu5OTTsHhd1dEZ10XdLiXrgSk +FWfvE+ftrKkJPsixyd1b8ogB1fflQf9HihUIv4cNj0dAhyiNwZXJ6qQ97eR+QM4OIbH FVwuo7wwKJR3ss+EFanmCMWPq3t7Fx1gcOiypquMQ6YR9UcxtijhIIliypqXBpf/7b2g kWKFbhBYZTK2F9aoXIR0ZUAXrKVBx7g5gdb/GTNj+wSNPjqY/dMcDe/o8wwGEb86JUBc onBw== X-Gm-Message-State: APjAAAWrDeIYa7DPWEALLzmXzm5SyPtffe0S0W59bwLZGUYaBFA7U2v4 73WHQsEquFivD1LwixU4ramyRAGr0ex4MehW4nNJxEmYEQufXhjOq71LOvFohEWBruCZVMWOQWU 3KTUu0YSrROVXkoX+THZX6Hv+ZoTU X-Received: by 2002:a62:16d3:: with SMTP id 202mr6980031pfw.118.1574900047372; Wed, 27 Nov 2019 16:14:07 -0800 (PST) X-Google-Smtp-Source: APXvYqzJc+VcxwW5kWr8ouolCeyP8fu1uzQijQ/p273FgvMFRNGogDQLIK+RwXvnf+WFq5+P1kvN6w== X-Received: by 2002:a62:16d3:: with SMTP id 202mr6980008pfw.118.1574900047098; Wed, 27 Nov 2019 16:14:07 -0800 (PST) Received: from localhost (ip70-163-223-149.ph.ph.cox.net. [70.163.223.149]) by smtp.gmail.com with ESMTPSA id b1sm7823320pfi.74.2019.11.27.16.14.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2019 16:14:06 -0800 (PST) Date: Wed, 27 Nov 2019 17:14:04 -0700 From: Jerry Snitselaar To: Dan Williams Cc: Jarkko Sakkinen , Stefan Berger , Stefan Berger , linux-integrity@vger.kernel.org, Linux Kernel Mailing List , linux-security-module@vger.kernel.org Subject: Re: [PATCH] tpm_tis: Move setting of TPM_CHIP_FLAG_IRQ into tpm_tis_probe_irq_single Message-ID: <20191128001404.3y4wh4hbhqpzxmfx@cantor> Reply-To: Jerry Snitselaar Mail-Followup-To: Dan Williams , Jarkko Sakkinen , Stefan Berger , Stefan Berger , linux-integrity@vger.kernel.org, Linux Kernel Mailing List , linux-security-module@vger.kernel.org References: <20191112202725.3009814-1-stefanb@linux.vnet.ibm.com> <20191114164151.GB9528@linux.intel.com> <20191114164426.GC9528@linux.intel.com> <185664a9-58f2-2a4b-4e6b-8d7750a35690@linux.ibm.com> <20191121184949.yvw2gwzlkhjzko64@cantor> <20191127211109.GF14290@linux.intel.com> MIME-Version: 1.0 In-Reply-To: X-MC-Unique: fNT15UPwOySjOHj6gIGeqA-1 X-Mimecast-Spam-Score: 0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: quoted-printable Content-Disposition: inline Sender: linux-integrity-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-integrity@vger.kernel.org On Wed Nov 27 19, Dan Williams wrote: >[ add Jerry ] > >On Wed, Nov 27, 2019 at 1:11 PM Jarkko Sakkinen > wrote: >> >> On Thu, Nov 21, 2019 at 11:49:49AM -0700, Jerry Snitselaar wrote: >> > On Sat Nov 16 19, Stefan Berger wrote: >> > > On 11/14/19 11:44 AM, Jarkko Sakkinen wrote: >> > > > On Thu, Nov 14, 2019 at 06:41:51PM +0200, Jarkko Sakkinen wrote: >> > > > > On Tue, Nov 12, 2019 at 03:27:25PM -0500, Stefan Berger wrote: >> > > > > > From: Stefan Berger >> > > > > > >> > > > > > Move the setting of the TPM_CHIP_FLAG_IRQ for irq probing into >> > > > > > tpm_tis_probe_irq_single before calling tpm_tis_gen_interrupt. >> > > > > > This move handles error conditions better that may arise if an= ything >> > > > > > before fails in tpm_tis_probe_irq_single. >> > > > > > >> > > > > > Signed-off-by: Stefan Berger >> > > > > > Suggested-by: Jerry Snitselaar >> > > > > What about just changing the condition? >> > > > Also cannot take this since it is not a bug (no fixes tag). >> > > >> > > I'll repost but will wait until Jerry has tested it on that machine. >> > > >> > > Stefan >> > > >> > > >> > > > >> > > > /Jarkko >> > > >> > > >> > >> > It appears they still have the problem. I'm still waiting on logistics >> > to send me a system to debug. >> >> Which hardware is guaranteed to ignite this? I can try to get test hw >> for this from somewhere. Kind of looking into this blinded ATM. Dan? > >Jerry had mentioned that this was also occurring on T490s. Otherwise >I'll ping you offline about the system I saw this on internally. > I've been trying for about 3 weeks now to get one of the laptops from logistics here, but unfortunately they have been silent. Pinged one of their email addresses today to see if they could respond to the ticket, so hopefully next week I will have something.