From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from verein.lst.de (verein.lst.de [213.95.11.211]) (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 CE20054F95 for ; Wed, 13 Nov 2024 14:30:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=213.95.11.211 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1731508252; cv=none; b=SK1fw/cF/26VEtj/Tkgo9EmHscX6eC7DL62S+EJtdgUJeKVzXQhnf7+EPc6jfJk7loR4kj5pUxRE4e2c6ro3/MFkf7YNwkhZm4kbn4elGhgclwut8aKYpw/okmG4OiMRIYS5qDi73l/cwKYMrzRhJzIR96RHW9JF8taw9BMmQCc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1731508252; c=relaxed/simple; bh=ZfjbghJSvVTLjJH8eDY6fgXNlZmBxkdlyNnBXN+eGSE=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=TttJB8gtmw/+k+qJ5fktE7hKPHDoFq+rJwVQz1p+TzM3AfuQH3I4xArGizaRLvixa9y/lp2MIfw54xIUoB7o3KojxLHFrcvOzuDn8spDe5F6cKOb6cds95Q+uwrr6KhmJ+fKIURNOGMVZS99cwS7by+NgsrC1k7v2npVnlUkBgo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=lst.de; spf=pass smtp.mailfrom=lst.de; arc=none smtp.client-ip=213.95.11.211 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=lst.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=lst.de Received: by verein.lst.de (Postfix, from userid 2407) id C525C68C4E; Wed, 13 Nov 2024 15:30:39 +0100 (CET) Date: Wed, 13 Nov 2024 15:30:39 +0100 From: Christoph Hellwig To: Greg Kroah-Hartman Cc: John Garry , Daniel Wagner , Daniel Wagner , Jens Axboe , Bjorn Helgaas , "Michael S. Tsirkin" , Jason Wang , Xuan Zhuo , Eugenio =?iso-8859-1?Q?P=E9rez?= , "Martin K. Petersen" , Keith Busch , Christoph Hellwig , Sagi Grimberg , linux-block@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, virtualization@lists.linux.dev, linux-scsi@vger.kernel.org, megaraidlinux.pdl@broadcom.com, mpi3mr-linuxdrv.pdl@broadcom.com, MPT-FusionLinux.pdl@broadcom.com, storagedev@microchip.com, linux-nvme@lists.infradead.org Subject: Re: [PATCH v3 1/8] driver core: bus: add irq_get_affinity callback to bus_type Message-ID: <20241113143039.GA20331@lst.de> References: <20241112-refactor-blk-affinity-helpers-v3-0-573bfca0cbd8@kernel.org> <20241112-refactor-blk-affinity-helpers-v3-1-573bfca0cbd8@kernel.org> <76da6c05-4f28-41cc-a48e-da2ae16c64c4@oracle.com> <2d85aa5e-037a-45c3-9f2d-e46b2159b697@flourine.local> <2024111323-darkening-sappy-23fa@gregkh> Precedence: bulk X-Mailing-List: virtualization@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2024111323-darkening-sappy-23fa@gregkh> User-Agent: Mutt/1.5.17 (2007-11-01) On Wed, Nov 13, 2024 at 02:54:23PM +0100, Greg Kroah-Hartman wrote: > bus types are good to set it at a bus level so you don't have to > explicitly set it at each-and-every-driver. Depends on what you want > this to be, if it is a "all drivers of this bus type will have the same > callback" then put it on the bus. otherwise if you are going to > mix/match on a same bus, then put it in the driver structure. ... and that is exactly the case here. The driver itself has no business being involved.