Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5;
Date:   Thu, 5 Oct 2023 16:19:44 +0530
From:   Pavan Kondeti <quic_pkondeti@quicinc.com>
To:     Greg KH <gregkh@linuxfoundation.org>
CC:     Zhenhua Huang <quic_zhenhuah@quicinc.com>, <rafael@kernel.org>,
        <linux-kernel@vger.kernel.org>, <quic_pkondeti@quicinc.com>,
        <quic_tingweiz@quicinc.com>, <saravanak@google.com>
Subject: Re: [RESEND PATCH] driver core: Clear FWNODE_FLAG_LINKS_ADDED in
 device_links_purge()
Message-ID: <50508cf9-fd4f-4199-9df3-e42ebb489499@quicinc.com>
References: <1695781810-5700-1-git-send-email-quic_zhenhuah@quicinc.com>
 <2023100501-entourage-bounce-3364@gregkh>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <2023100501-entourage-bounce-3364@gregkh>
Precedence: bulk

On Thu, Oct 05, 2023 at 11:32:22AM +0200, Greg KH wrote:
> On Wed, Sep 27, 2023 at 10:30:10AM +0800, Zhenhua Huang wrote:
> > Flag FWNODE_FLAG_LINKS_ADDED stops fwnode links creation. Current kernel
> > only adds it once after fwnode links creation in fw_devlink_parse_fwnode().
> > After that even device links being purged, the flag will not be cleared.
> > 
> > Fwnode links are converted to device links and will not be added back
> > forever in normal case. Essentially if a device is registered and
> > unregisted (also deleted) before it is probed (due to missing fwlink
> > dependencies, abort in device_links_check_suppliers), the fwlink is not
> > setup next when device is newly created again. This means the probe gets
> > called without meeting all dependencies.
> > 
> > It usuallly happens in the case of a glue driver.
> 
> What exact glue driver is causing this to happen?  Why can't we fix that
> up instead?
> 

Yes, we are looking at fixing the glue driver and/or the probe order of drivers
involed by controlling modules load order. To answer your question,

It is observed with dwc3-qcom on a downstream kernel based on
Android GKI 6.1 kernel.

usb_1: usb@a6f8800 {
	...
	usb_1_dwc3: usb@a600000 {
		compatible = "snps,dwc3";
		iommus = <&apps_smmu 0x40 0x0>;
		...
	};
};

The parent device has no dependency on IOMMU but the child has
dependency. The parent probe gets called before IOMMU is probed.
The parent glue driver adds the child device and removes it since its
probe is not completed. Once the child is unregistered, all the fwlinks
are lost. Next time when the child is added, its probe gets called
before IOMMU is ready (since the fwlinks are lost). 

Thanks,
Pavan