From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bart Van Assche Date: Tue, 31 Jul 2018 20:18:30 +0000 Subject: Re: [PATCH v2 3/9] scsi: build scsi_common.o for all scsi passthrough request users Message-Id: <6c04fc087bae896bf00b94184def3d7697a22d6c.camel@wdc.com> List-Id: References: <20180731195155.46664-1-keescook@chromium.org> <20180731195155.46664-4-keescook@chromium.org> <20180731201225.GA1989@infradead.org> In-Reply-To: <20180731201225.GA1989@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "hch@infradead.org" Cc: "vinod.koul@intel.com" , "david.kershner@unisys.com" , "linux-block@vger.kernel.org" , "keescook@chromium.org" , "rdunlap@infradead.org" , "u.kleine-koenig@pengutronix.de" , "nab@linux-iscsi.org" , "tglx@linutronix.de" , "hch@lst.de" , "martin.petersen@oracle.com" , "manoj@linux.vnet.ibm.com" , "axboe@kernel.dk" , "jgross@suse.com" , "cyrille.pitchen@free-electrons.com" , "sdharia@codeaurora.org" , "viresh.kumar@linaro.org" , "linux-scsi@vger.kernel.org" , "target-devel@vger.kernel.org" , "pombredanne@nexb.com" , "linux-ide@vger.kernel.org" , "bp@alien8.de" , "mrochs@linux.vnet.ibm.com" , "tj@kernel.org" , "davem@davemloft.net" , "jejb@linux.vnet.ibm.com" , "linux-kernel@vger.kernel.org" , "sboyd@codeaurora.org" , "ukrishn@linux.vnet.ibm.com" On Tue, 2018-07-31 at 13:12 -0700, hch+AEA-infradead.org wrote: +AD4- It shouldn't. All these are built into scsi+AF8-mod.o, which is only built +AD4- when CONFIG+AF8-SCSI is set. Under what circumstances do you see them built? Although I think you are right, I still prefer that the scsi+AF8-common.c file would be moved to a new directory. That will prevent confusion later on for people who want to add additional code. This patch makes it nontrivial to figure out which code is built when SCSI target functionality is enabled but SCSI initiator functionality is not selected. I think moving scsi+AF8-common.c into a new directory would make it much easier to figure out which code is built depending on the kernel configuration. Bart.