[PATCH 1/1] nvme: temporary fix for Apple NVMe controller reset

[guenther@tum.de (Stephan Günther)]

On 2015/November/21 12:20, Christoph Hellwig wrote:
> Hi Stephan,
> 
> > +	/*
> > +	 * Temporary fix for the Apple controller found in the MacBook8,1 and
> > +	 * some MacBook7,1 to avoid controller resets and data loss.
> > +	 */
> > +	if (pdev->vendor == PCI_VENDOR_ID_APPLE && pdev->device == 0x2001)
> > +		dev->q_depth = 2;
> 
> Can you add a dev_warn() here?  A NVMe device with queue depth is going
> to perform bady, and we should tell people that we're applying a crude
> workaround.

I'll resend the patch with an updated commit message and dev_warn().

Interestingly, I cannot really complain about performance with that
patch applied. That little notebook behaves really well. But that queue
depth cannot be the solution of course.

> 
> Otherwise this looks fine to me to get you guys going, but I really hope
> we'll find the root cause soon.

I wish Apple would show any interest in that matter...