From mboxrd@z Thu Jan 1 00:00:00 1970 From: hch@lst.de (Christoph Hellwig) Date: Thu, 17 Aug 2017 09:36:20 +0200 Subject: Centralize nvme controller reset, delete and fabrics periodic reconnects In-Reply-To: References: <1502790745-12569-1-git-send-email-sagi@grimberg.me> <20170816104935.GA30525@lst.de> <20170816151723.GA18932@lst.de> Message-ID: <20170817073620.GA24372@lst.de> On Thu, Aug 17, 2017@10:21:08AM +0300, Sagi Grimberg wrote: > >>> Having said that, I'm very much open to suggestions if you have anything >>> better in mind... >> >> Not too many suggestions - you want to see it done before we go on >> merging the code.. > > Something is not adding up in your reply :) > > You want to see it done before we merge? Or are you asking if I > want to see it done? I designed it to be incremental and not > break existing transports. > > Personally I think that pci conversion will have to be incremental > and gradual as we need some preparations in place first. So if you are > reluctant to get it in without full conversion its ok, just note it will > take me some time to get both pci and fc converted. I'd really like to see at least a demo PCI conversion. Guess I need to take care of it then. I'd also like to see FC, but I'm not that concerned about it as it better fits into the Fabrics scheme.