From mboxrd@z Thu Jan 1 00:00:00 1970 From: keith.busch@intel.com (Keith Busch) Date: Wed, 7 Mar 2018 13:37:41 -0700 Subject: [PATCH] Revert "nvme: create 'slaves' and 'holders' entries for hidden controllers" In-Reply-To: References: <20180307131358.20275-1-hch@lst.de> <1c1332d4-1d92-af8f-974c-2eafcde741f2@suse.de> <20180307150344.GE30975@localhost.localdomain> <20180307171805.GA810@lst.de> <20180307173842.GA3766@localhost.localdomain> Message-ID: <20180307203741.GC3766@localhost.localdomain> On Wed, Mar 07, 2018@05:47:34PM +0000, Sitsofe Wheeler wrote: > The only thing that concerns me is where do the stats live? Over in > https://github.com/axboe/fio/pull/526#issuecomment-369521772 we found > the main device doesn't record stats. Will that still be the case? The stats still live in the same place at /sys/block/nvmeXcYnZ/queue/stats. This patch just removes symlinks to the each path block device in the multipath block device's slave directory, so 'fio' won't see them at the moment. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com ([134.134.136.20]:2696 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934506AbeCGUhg (ORCPT ); Wed, 7 Mar 2018 15:37:36 -0500 Date: Wed, 7 Mar 2018 13:37:41 -0700 From: Keith Busch To: Sitsofe Wheeler Cc: Christoph Hellwig , Hannes Reinecke , linux-nvme@lists.infradead.org, Jens Axboe , sagi@grimberg.me, Potnuri Bharat Teja , Karel Zak , stable Subject: Re: [PATCH] Revert "nvme: create 'slaves' and 'holders' entries for hidden controllers" Message-ID: <20180307203741.GC3766@localhost.localdomain> References: <20180307131358.20275-1-hch@lst.de> <1c1332d4-1d92-af8f-974c-2eafcde741f2@suse.de> <20180307150344.GE30975@localhost.localdomain> <20180307171805.GA810@lst.de> <20180307173842.GA3766@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: stable-owner@vger.kernel.org List-ID: On Wed, Mar 07, 2018 at 05:47:34PM +0000, Sitsofe Wheeler wrote: > The only thing that concerns me is where do the stats live? Over in > https://github.com/axboe/fio/pull/526#issuecomment-369521772 we found > the main device doesn't record stats. Will that still be the case? The stats still live in the same place at /sys/block/nvmeXcYnZ/queue/stats. This patch just removes symlinks to the each path block device in the multipath block device's slave directory, so 'fio' won't see them at the moment.