[PATCH 2/5] nvme: introduce a controller state machine

Christoph Hellwig hch at lst.de
Tue Apr 19 11:25:45 PDT 2016


On Mon, Apr 18, 2016 at 11:30:34PM +0300, Sagi Grimberg wrote:
>
>>>> -	clear_bit(NVME_CTRL_RESETTING, &dev->flags);
>>>> +	if (!nvme_change_ctrl_state(&dev->ctrl, NVME_CTRL_LIVE)) {
>>>> +		dev_warn(dev->ctrl.device, "failed to mark controller live\n");
>>>> +		goto out;
>>>> +	}
>>>>    	return;
>>>>
>>>>     out:
>>>
>>> You're missing a state change before removing a
>>> dead controller...
>>
>> why?
>
> Just to have the state consistent with what the driver
> is doing. Not a must though...

Let's allocate states on an as needed basis and avoid that new
state for now.



More information about the Linux-nvme mailing list