Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 31 May 2023 01:04:36 +0300
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Bjorn Helgaas <helgaas@kernel.org>
Cc:     linux-pci@vger.kernel.org, netdev@vger.kernel.org,
        Bjorn Helgaas <bhelgaas@google.com>,
        Rob Herring <robh@kernel.org>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Michael Walle <michael@walle.cc>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH pci] PCI: don't skip probing entire device if first fn OF
 node has status = "disabled"
Message-ID: <20230530220436.fooxifm47irxqlrj@skbuf>
References: <20230521115141.2384444-1-vladimir.oltean@nxp.com>
 <ZHZxn0a3/EJbthYO@bhelgaas>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <ZHZxn0a3/EJbthYO@bhelgaas>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?gOx1NtRnJel12Y3LxS0twQtymKlcMh/WlN0+g27r6L/q7KnA9QAJd2syIcH8?=
 =?us-ascii?Q?F7njaYrALTgg74/q7FPaHnAk8AEWLMYGy2/GLor9lDlLhdlNB6GSHS21nnXt?=
 =?us-ascii?Q?ZOjWYK4ya8dpxwuTBn7F9nk7zT+G8JLtZj63/+OAjm1da8PSgogF5EVJkMZv?=
 =?us-ascii?Q?qmuVDcXp65P3Oj3izq0kI18L8UMuM4zFdeKWZpdyJvUPyvf+UHsSPJ+lZZXd?=
 =?us-ascii?Q?jVLxbNzIIlua0H43l6oKtRWn18Avtmh/8WtP55SfmpTMr2dEx0shuFvCwC6W?=
 =?us-ascii?Q?Wg0hkGnGeq6FIUOawGQFtWL4tFbntp66PlRSugrHk3y+YGph8A8tLmE6mBc4?=
 =?us-ascii?Q?gyLEyhZk8GgJ2brjsJN1//GGB0VqETSrt7QDH9fAMUGVAFEQxZ8yvM6uS6nc?=
 =?us-ascii?Q?qh1blQBwUu3Mt26O6aVBvS+NIS0UfqQp2bds5aAu77lG7YejGloukoXDA4kH?=
 =?us-ascii?Q?TjEY5V50SJ2EfDJlmyEP7r5OJHcl7IAIaKElmZzA4ybiK/e4XMN6hy7FDdRM?=
 =?us-ascii?Q?K0/Poj+FKxW1eVfJUYlpwiNdv1hV7PoOhmvIDetTt2AMGwSNwymtA9/Z+Ixg?=
 =?us-ascii?Q?GA/p7MdD6lGLdyOJxyoTy/w66lOpCYif2SAU2XdoWZ3cAdiHFvCdfzFRM5Lp?=
 =?us-ascii?Q?P9k+H2zmKpN4LawL68e4qMyh1b6TTDVuMyxs9vZSIlRSCQohgVmSfZFvskxr?=
 =?us-ascii?Q?u1LqQqpU8eZf5fQQtsxuvjcfHQNZEx4kEZw4a9asN8JlaZcZDfcQPZhQ1k2s?=
 =?us-ascii?Q?opo4hWjbYTvjqx91GjUp9xxXY2tl+KUwLwKUMbWML5WnCOGoDE7E0c0eRiW7?=
 =?us-ascii?Q?0qOd2BSR77sl7hYsiMCoTPDItzEP4HD7XjAd9wUvvLtmOmJy6JguoAMZlks9?=
 =?us-ascii?Q?NRLf1cKxbgfEqef74sGulQdd217HbpD1FIzAn1WgW8r0MxOz4lerjv1D1ijZ?=
 =?us-ascii?Q?YlEC1V4w8qvftYcyYoG7yIrktdJZAAa7uHi8cFgeU8qNNsC9ZIkfzN3L6SWm?=
 =?us-ascii?Q?d2NIbkP/CUyi7QQhapZ7ohCbBuFvyc5iFAYLNtrxJQIK/arMU8L9HxKnJgHj?=
 =?us-ascii?Q?lRXTTyhg/dLHC/irM/LlyvQGRMVIHX43MdCTU+IbfNWIJe1pkv7Mf5EcrK40?=
 =?us-ascii?Q?iQZTy5w7LzeU6o0Be6umuQlAb9HqjH37yVOeyT2GPQZGykAgj7FAYa62NbII?=
 =?us-ascii?Q?wj09ZTnLSDqkMxo+XqkgoJQixqpMEaHg7L6oPhbI/gmnLw6L7UHvvIwLZvmq?=
 =?us-ascii?Q?MkwJP6ID+Bf29DUEQ4LXvvuAqEP7nj48vVEwHsKl4IEz9dIRS1LFEDiCEGnv?=
 =?us-ascii?Q?T/s/l6uNLzmuqoP8k+tSg6ZQW1nBQ4DdxWrUJufmQ3M8ZFENE1I/ZL9cpmwK?=
 =?us-ascii?Q?/MzzftaTgwhFnEpZf1WhrxZh4xeDtvfNLoQPd5t0MABW6zSkompKb56Pzudq?=
 =?us-ascii?Q?6jwa/Gga7rFGkLtJHofPkSdABghX0lKZ4/me8YVmaraz6DR3bnbIYUjQfK5v?=
 =?us-ascii?Q?HsRbLQh+vWRjBlYs+duAsk9KSu05BaJIopY+x5mP5xR/tsW/znXCRFhfPFvt?=
 =?us-ascii?Q?w+cRpkFVLioNc43gP4wqrYSXlDsxZ5t1wsIJjaerfObaFd4bhGGaGBBh0lPJ?=
 =?us-ascii?Q?5Q=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: dd769c17-599e-4637-6277-08db6159dd51
X-MS-Exchange-CrossTenant-AuthSource: DB8PR04MB6459.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 May 2023 22:04:39.9357
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: i9fHBWtG1j2AgVD4wvEca4293TtsKPPhuAm/0PkJtYA2qpbyH4GlocVxcvk9uccSkyLT+k/mDsWCYsOEaG/jcg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU2PR04MB9131
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE,
        URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Tue, May 30, 2023 at 04:58:55PM -0500, Bjorn Helgaas wrote:
> Can you write this description in terms of PCI topology?  The
> nitty-gritty SERDES details are not relevant at this level, except to
> say that Function 0 is present in some cases but not others, and when
> it is not present, *other* functions may be present.

No. It is to say that within the device, all PCIe functions (including 0)
are always available and have the same number, but depending on SERDES
configuration, their PCIe presence might be practically useful or not.
So that's how function 0 may end having status = "disabled" in the
device tree.

> Sigh.  Per spec (PCIe r6.0, sec 7.5.1.1.9), software is not permitted
> to probe for Functions other than 0 unless "explicitly indicated by
> another mechanism, such as an ARI or SR-IOV Capability."
> 
> Does it "work" to probe when the spec prohibits it?  Probably.  Does
> it lead to some breakage elsewhere eventually?  Quite possibly.  They
> didn't put "software must not probe" in the spec just to make
> enumeration faster.
> 
> So I'm a little grumpy about further complicating this already messy
> path just to accommodate a new non-compliant SoC.  Everybody pays the
> price of understanding all this stuff, and it doesn't seem in balance.
> 
> Can you take advantage of some existing mechanism like
> PCI_SCAN_ALL_PCIE_DEVS or hypervisor_isolated_pci_functions() (which
> could be renamed and made more general)?

Not responding yet to the rest of the email since it's not clear to me
that you've understood function 0 is absolutely present and responds
to all config space accesses - it's just disabled in the device tree
because the user doesn't have something useful to do with it.