On Sun, Apr 26, 2020 at 12:48:18PM +0300, Max Gurtovoy wrote: >> I see no reason not to support a simple Format NVM for our fabrics target >> implementation. But that isn't the point - you don't really need Format >> as you can also control it from configfs in your series. So for the >> initial version I don't think we need Format NVM, but I don't mind >> adding it later. > > so we're ok with passing -p in nvme-cli during connect command ? PI should be enable by default. We can think of a hook disabling it, but please keep it at the end of the series.