From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3B0A232D441 for ; Wed, 25 Mar 2026 09:29:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.133.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774430999; cv=none; b=Mw+7Xaeuvrts3D43B5e+0kSDEGURl/tVINQfdFKowjQxrPaYpKEX3t2dKuwBxHTgnSjvI5pa1EdKbG/htmpu1H1h2qslfNNaVzNKVVZBoQqBailIGDFW+UB0KH+J68lt2dSdDnohnsRkua/kepSd//HkeiJqakUyiqQ6QrDgMgY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774430999; c=relaxed/simple; bh=hzAY3ZoPFk3ktxoVDsUh6drVK7glBEle7ZgqK1lKcjk=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: In-Reply-To:Content-Type:Content-Disposition; b=Z5RKF5fQmKtyfYosvl3Y6Wt5NyK05Hv4oLIiwl/khBx6AZ1cJGIa32/672WOhf/g99wn4D84dYYQokIbIhAW9samPTiaPDxraKXqh0eM/tNA4v97xHhaKEA1LL7F3XSHdLgSIT9SGO4/mqTBOsXqUs3FVvkrexU3JfPpDtmTjzQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=UIIwNU+n; arc=none smtp.client-ip=170.10.133.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="UIIwNU+n" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1774430996; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=/X6OQFIMMVlzQpHVcxvz5zk00vazKEddYu1a8TXBALA=; b=UIIwNU+nPUJJvHBBie22wcCjzDVg3JpRuHuXBHiy0rjpN6rDxNAzNlOeQY2QG9m8DWuxEm iaNmUUXhpXPFoYwUfpFYc/GchGtfpCQlnmZs/FYSLP7O1jEI3zV30Rr3AOr8Y9H0wsXSVL jcPPi+BUVu4IYinrziQaqTXzZ8OPLkY= Received: from mail-wr1-f71.google.com (mail-wr1-f71.google.com [209.85.221.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-572-QKpX5u6QMSKKxR9jZoqkug-1; Wed, 25 Mar 2026 05:29:55 -0400 X-MC-Unique: QKpX5u6QMSKKxR9jZoqkug-1 X-Mimecast-MFC-AGG-ID: QKpX5u6QMSKKxR9jZoqkug_1774430994 Received: by mail-wr1-f71.google.com with SMTP id ffacd0b85a97d-43b3e0a73f8so2562135f8f.3 for ; Wed, 25 Mar 2026 02:29:54 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1774430994; x=1775035794; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=/X6OQFIMMVlzQpHVcxvz5zk00vazKEddYu1a8TXBALA=; b=s8bRc3TsMtfvAr8Pk6DBqRqRU7LDBbAO91tOyJzsHtp76rqXl5WPBIiF9ws1gF/rcO n2HtcwLxP9f0brvrCiiIqR4voQ+wqCIeB3rpiTdYCfSyh33wMhatsmCVFfHhwik99UoG BY5eaPEo7jze9eBVi7/3EGG7Ap3qZRxLm1mOIO6hgI3gst3AIu/jdOcT6y7uY8L+8++h /NHV57vwvpvX+ieWt5db5CiKHelbJpQEr2BfZkWYZEWneB0omhvjCi2Y40tidN1zip7s IwOLzV/KwzpViV1lArgHhMvr1EK8RSldwwbLQCXYoqfivWSIJsUvYkBLMGFFDi1i3XDN LC6A== X-Forwarded-Encrypted: i=1; AJvYcCU7AISGFK9rlaw7O9eTPyDOa/w2G9Wno04YKttcIBVPk9+MOmNGyYWTSXqS4r9I39/Y1yGCdmSnOJM=@vger.kernel.org X-Gm-Message-State: AOJu0YxdNrTrU8gSyKoi/EqXK5MWJVVbXv2FHVeO3Uz4tq940yxzRMGQ y9Jvc6yOZ1eGKwvUy3Gk7dM4DN5WqYGVtBm6bGTnlxfLxK0KW3LrnOZFq0phXPH7ImVuWXsTw2F sYd2joghCxj3zQlihwHTtqsyjkWLo7VBPn6mj6eTqsP5M0G58WSvElRrYCOF8Tg== X-Gm-Gg: ATEYQzylp13VjHDfav2E1pWxq0r1TfqdEbxwBO8pm/1zpowC4CSLDSDM5McqsJoSFsR VWaZj+fVZ+nIY17o7bWYU7id6lTGIhpHxHdX6RGtBJzz9FSnYcmXJ4jyM4G7IYrzXeZ0alS8R6Q HojkUdBzDqTmI/FQdySSNP+0hueA1DPg0mby1Ws9AF2MedMH2UmBCbxXKh1EQOmd4+HJrD2Ihkn z7ExOqntfv356s5PxKM/KagBDXcYQRfHu1Du1Eym/9rBmpXGWcmjFhbuinmPb6lguoawQB9j76m faUAGGz4RSkLNt1JoxtIc1jwUod1gtWE4saWCVUDzXYMNIZq4j0VKQZ3kmReJJl3t7eCn88V7gA 7QxcHq7PtGNd1ivOy X-Received: by 2002:a05:6000:400c:b0:439:be78:e1e9 with SMTP id ffacd0b85a97d-43b88a3d3cdmr3692802f8f.14.1774430993569; Wed, 25 Mar 2026 02:29:53 -0700 (PDT) X-Received: by 2002:a05:6000:400c:b0:439:be78:e1e9 with SMTP id ffacd0b85a97d-43b88a3d3cdmr3692727f8f.14.1774430992928; Wed, 25 Mar 2026 02:29:52 -0700 (PDT) Received: from redhat.com ([2a0d:6fc0:1525:da00:3ac2:1a22:72ff:4256]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-43b6470380asm44280922f8f.24.2026.03.25.02.29.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 25 Mar 2026 02:29:52 -0700 (PDT) Date: Wed, 25 Mar 2026 05:29:45 -0400 From: "Michael S. Tsirkin" To: Danilo Krummrich Cc: Russell King , Greg Kroah-Hartman , "Rafael J. Wysocki" , Ioana Ciornei , Nipun Gupta , Nikhil Agarwal , "K. Y. Srinivasan" , Haiyang Zhang , Wei Liu , Dexuan Cui , Long Li , Bjorn Helgaas , Armin Wolf , Bjorn Andersson , Mathieu Poirier , Vineeth Vijayan , Peter Oberparleiter , Heiko Carstens , Vasily Gorbik , Alexander Gordeev , Christian Borntraeger , Sven Schnelle , Harald Freudenberger , Holger Dengler , Mark Brown , Jason Wang , Xuan Zhuo , Eugenio =?iso-8859-1?Q?P=E9rez?= , Alex Williamson , Juergen Gross , Stefano Stabellini , Oleksandr Tyshchenko , "Christophe Leroy (CS GROUP)" , linux-kernel@vger.kernel.org, driver-core@lists.linux.dev, linuxppc-dev@lists.ozlabs.org, linux-hyperv@vger.kernel.org, linux-pci@vger.kernel.org, platform-driver-x86@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-remoteproc@vger.kernel.org, linux-s390@vger.kernel.org, linux-spi@vger.kernel.org, virtualization@lists.linux.dev, kvm@vger.kernel.org, xen-devel@lists.xenproject.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 00/12] treewide: Convert buses to use generic driver_override Message-ID: <20260325052919-mutt-send-email-mst@kernel.org> References: <20260324005919.2408620-1-dakr@kernel.org> Precedence: bulk X-Mailing-List: linux-spi@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 In-Reply-To: <20260324005919.2408620-1-dakr@kernel.org> X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: 8akcHUCN1x7g0EgcoigZ6dbEEBwCTKMv5RHL6KWZNRw_1774430994 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Tue, Mar 24, 2026 at 01:59:04AM +0100, Danilo Krummrich wrote: > This is the follow-up of the driver_override generalization in [1], converting > the remaining 11 busses and removing the now-unused driver_set_override() > helper. > > All of them (except AP, which has a different race condition) are prone to the > potential UAF described in [2], caused by accessing the driver_override field > from their corresponding match() callback. > > In order to address this, the generalized driver_override field in struct device > is protected with a spinlock. The driver-core provides accessors, such as > device_match_driver_override(), device_has_driver_override() and > device_set_driver_override(), which all ensure proper locking internally. > > Additionally, the driver-core provides a driver_override flag in struct > bus_type, which, once enabled, automatically registers generic sysfs callbacks, > allowing userspace to modify the driver_override field. > > SPI and AP are a bit special; both print "\n" when driver_override is not set, > whereas all other buses (and thus the driver-core) produce "(null)\n" in this > case. > > Hence, SPI and AP do not take advantage of the driver_override flag in struct > bus_type; AP additionally maintains a counter in its custom sysfs store(). > > Technically, we could support a custom fallback string when driver_override is > unset in struct bus_type, but only SPI would benefit from this, since AP has > additional custom logic in store() anyways. > > (I'm not sure if there are userspace programs that strictly rely on this; > driverctl seems to check for both, but I rather not break some userspace tool > I'm not aware of. :) > > This series is based on v7.0-rc5 with no additional dependencies, hence those > patches can be picked up by subsystems individually. > > [1] https://lore.kernel.org/driver-core/20260303115720.48783-1-dakr@kernel.org/ > [2] https://bugzilla.kernel.org/show_bug.cgi?id=220789 > [3] https://gitlab.com/driverctl/driverctl/-/blob/0.121/driverctl?ref_type=tags#L99 vdpa bits: Acked-by: Michael S. Tsirkin I assume it'll all be merged together? > Danilo Krummrich (12): > amba: use generic driver_override infrastructure > bus: fsl-mc: use generic driver_override infrastructure > cdx: use generic driver_override infrastructure > hv: vmbus: use generic driver_override infrastructure > PCI: use generic driver_override infrastructure > platform/wmi: use generic driver_override infrastructure > rpmsg: use generic driver_override infrastructure > vdpa: use generic driver_override infrastructure > s390/cio: use generic driver_override infrastructure > s390/ap: use generic driver_override infrastructure > spi: use generic driver_override infrastructure > driver core: remove driver_set_override() > > drivers/amba/bus.c | 37 +++------------ > drivers/base/driver.c | 75 ------------------------------ > drivers/bus/fsl-mc/fsl-mc-bus.c | 43 +++-------------- > drivers/cdx/cdx.c | 40 ++-------------- > drivers/hv/vmbus_drv.c | 36 ++------------ > drivers/pci/pci-driver.c | 11 +++-- > drivers/pci/pci-sysfs.c | 28 ----------- > drivers/pci/probe.c | 1 - > drivers/platform/wmi/core.c | 36 ++------------ > drivers/rpmsg/qcom_glink_native.c | 2 - > drivers/rpmsg/rpmsg_core.c | 43 +++-------------- > drivers/rpmsg/virtio_rpmsg_bus.c | 1 - > drivers/s390/cio/cio.h | 5 -- > drivers/s390/cio/css.c | 34 ++------------ > drivers/s390/crypto/ap_bus.c | 34 +++++++------- > drivers/s390/crypto/ap_bus.h | 1 - > drivers/s390/crypto/ap_queue.c | 24 +++------- > drivers/spi/spi.c | 19 +++----- > drivers/vdpa/vdpa.c | 48 ++----------------- > drivers/vfio/fsl-mc/vfio_fsl_mc.c | 4 +- > drivers/vfio/pci/vfio_pci_core.c | 5 +- > drivers/xen/xen-pciback/pci_stub.c | 6 ++- > include/linux/amba/bus.h | 5 -- > include/linux/cdx/cdx_bus.h | 4 -- > include/linux/device/driver.h | 2 - > include/linux/fsl/mc.h | 4 -- > include/linux/hyperv.h | 5 -- > include/linux/pci.h | 6 --- > include/linux/rpmsg.h | 4 -- > include/linux/spi/spi.h | 5 -- > include/linux/vdpa.h | 4 -- > include/linux/wmi.h | 4 -- > 32 files changed, 88 insertions(+), 488 deletions(-) > > > base-commit: c369299895a591d96745d6492d4888259b004a9e > -- > 2.53.0