Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp4012881imm; Mon, 30 Jul 2018 07:16:07 -0700 (PDT) X-Google-Smtp-Source: AAOMgpfHjD4BMadsviCWK3T+7KSNSGEHcl9VdCKkzHDpT2Cnr0iGmxcktUt2u0v0E2t93Hvw+jz5 X-Received: by 2002:a62:d75b:: with SMTP id v27-v6mr17902705pfl.79.1532960167259; Mon, 30 Jul 2018 07:16:07 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1532960167; cv=none; d=google.com; s=arc-20160816; b=geU0NrwUmlejvntKmJxLnz/OKoGoydRNyCq3A+I/ai/mN42CcLIAPwk0I32gDQ/xYY gAk9RcBx705mZKeHhHZy8pjx+9DzQ9og8BxLeqUZk07IKF4tJhDRMX966+yk0+3fYHcL S9dxHSd2bTulRct0z7l54HHp4h97z79s0KCwRXz5UDahcZkQhR/GfiIfuuIF+vVwD/hH /RIdnoWkHGJgYwGbFfJ9xQP+TavJ3p4b9CwOKa5uRt5biGvIQbmAbGm46cHG2aTXymvZ J3tS+mjxmdVbnw20jRrWvd0EkPFwHazrL33PbDMbPky4Tu2+L0yppa5groMqskZ09Zlx 0hmQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature:arc-authentication-results; bh=tG8neWtEwSQXojUNlDXoDhTrhS3Z7nAz+fqZisYlY/A=; b=OUbR0jkAj2I2JVaFnrJ4bNZ8kZXSGz20zCpISCEAhlnDbQYs9lZAdHNWCOW4huQWiu qoHn7yaCxlslOulM/+ABeZFfgyE3y7I5J7D93h1Moi7aTnslYCZ9P1gB+vtb57QOlhvh SscQ0Wl2uGBIBCM62DlOBMpYItS/47zxtDJPBWOeUNc0vCVR/qvTTHMu/yOYGo5Y7PE2 VhlOIfSEDtTgVRPgetYxwi5IZFnGS0jTTYEmpb8DfP9w0HSn6ziPoi+vCgRJBUxAj0N8 DyoEg9TkpTp9cyUtqrB7FvrHyOi/du06iOiLDNxNUP/otOwjpaqn1GGI1L92SExw28kZ 7rnw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=Kg36b2MN; 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=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 31-v6si10426780plg.260.2018.07.30.07.15.53; Mon, 30 Jul 2018 07:16:07 -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; dkim=pass header.i=@kernel.org header.s=default header.b=Kg36b2MN; 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=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731165AbeG3Ptm (ORCPT + 99 others); Mon, 30 Jul 2018 11:49:42 -0400 Received: from mail.kernel.org ([198.145.29.99]:42924 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726740AbeG3Ptm (ORCPT ); Mon, 30 Jul 2018 11:49:42 -0400 Received: from localhost (173-25-171-118.client.mchsi.com [173.25.171.118]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id DFE9020870; Mon, 30 Jul 2018 14:14:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1532960070; bh=w5Kc8XXkLNCRgUM+w62o6gYcaDSSaDvD/7D3LDS0yRk=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Kg36b2MNwR4T6w0oKB72fimR3YvQWQcuz3JT9PeeM7LgFycFK+6Tko0H8qF6f6nmo 68emZjxcScVfo7gK8JOYN3GLSpM8Gr3XWbiOWK2cnP1ytcHz29VMp9uYYjWMKfOQ+h xHJwRZ3+X/g3DgWyQFwG9aGIJu3ESBajpose2Jjw= Date: Mon, 30 Jul 2018 09:14:28 -0500 From: Bjorn Helgaas To: Sinan Kaya Cc: Rajat Jain , Bjorn Helgaas , Keith Busch , Vidya Sagar , Philippe Ombredanne , Kees Cook , "Gustavo A. R. Silva" , Ard Biesheuvel , Sinan Kaya , Frederick Lawler , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, mayurkumar.patel@intel.com, rajatxjain@gmail.com, Richard Hughes , Carlos Garnacho , "Rafael J. Wysocki" , Pali Rohar , Takashi Iwai , Andy Whitcroft , Colin Ian King Subject: Re: [PATCH v2] pci/aspm: Remove CONFIG_PCIEASPM_DEBUG Message-ID: <20180730141428.GK173328@bhelgaas-glaptop.roam.corp.google.com> References: <20180508230148.121852-1-rajatja@google.com> <20180510233912.96454-1-rajatja@google.com> <20180727202619.GD173328@bhelgaas-glaptop.roam.corp.google.com> <5a102163-73bc-c596-8266-186e222e7ceb@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5a102163-73bc-c596-8266-186e222e7ceb@kernel.org> User-Agent: Mutt/1.9.2 (2017-12-15) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Jul 28, 2018 at 05:16:13PM -0700, Sinan Kaya wrote: > On 7/27/2018 1:26 PM, Bjorn Helgaas wrote: > > - A link can lead to a multi-function device, and the spec allows > > those functions to have different ASPM settings (see PCIe r4.0, > > sec 5.4.1). With the sysfs files at the upstream end of the link, > > we have no way to configure those functions individually. > > Even though we can set them individually, there is only one PCIe link > for single function as well as multi-function devices. > > It would be a user mistake to allow individual PCIe functions with > different ASPM policies. (maybe, we should enforce it if we are not > doing it already). It's true that multi-function devices share a single PCIe link. However, the end of sec 5.4.1 does make it clear that the functions need not have the same ASPM configuration, and it gives rules for how those different settings should affect the shared link. Since it mentions different ASPM Control fields for the different functions, I assume the policy combining those per-function settings into the single link behavior must be implemented in the hardware. Obviously we don't support per-function ASPM settings today. I don't know whether there's any value in supporting them or not, but putting the control files at the upstream end basically precludes us from ever supporting them. Bjorn