Re: [PATCH v4 1/9] driver core: add a faux bus for use when a simple device/bus is needed

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

On Tue, Feb 11, 2025 at 08:27:26AM +0100, Greg Kroah-Hartman wrote:
> On Mon, Feb 10, 2025 at 12:56:46PM -0500, Kurt Borja wrote:
> I'll work on adding "if probe failed, don't let the device be created"
> logic as it's a simple change, BUT it is a functionally different change
> from what the current api that this code is replacing is doing (i.e. the
> current platform device creation code does this today and no one has
> ever hit this in their use of it in the past few decades.)

How about something as simple as this change, does that provide what you
are thinking about here?  Only compile tested, not runtime tested at
all:


diff --git a/drivers/base/faux.c b/drivers/base/faux.c
index 531e9d789ee0..e2de0697c0e3 100644
--- a/drivers/base/faux.c
+++ b/drivers/base/faux.c
@@ -25,6 +25,7 @@
 struct faux_object {
 	struct faux_device faux_dev;
 	const struct faux_device_ops *faux_ops;
+	bool probe_was_successful;
 };
 #define to_faux_object(dev) container_of_const(dev, struct faux_object, faux_dev.dev)
 
@@ -48,6 +49,9 @@ static int faux_probe(struct device *dev)
 	if (faux_ops && faux_ops->probe)
 		ret = faux_ops->probe(faux_dev);
 
+	if (!ret)
+		faux_obj->probe_was_successful = true;
+
 	return ret;
 }
 
@@ -147,6 +151,15 @@ struct faux_device *faux_device_create_with_groups(const char *name,
 		return NULL;
 	}
 
+	/*
+	 * The probe callback will set probe_was_successful if it
+	 * succeeded, if not, then we need to tear things down here
+	 */
+	if (!faux_obj->probe_was_successful) {
+		faux_device_destroy(faux_dev);
+		return NULL;
+	}
+
 	return faux_dev;
 }
 EXPORT_SYMBOL_GPL(faux_device_create_with_groups);