From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.linuxfoundation.org ([140.211.169.12]:53021 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750808AbbHNTTg (ORCPT ); Fri, 14 Aug 2015 15:19:36 -0400 Date: Fri, 14 Aug 2015 12:19:35 -0700 From: Greg KH To: Roland Dreier Cc: "Nicholas A. Bellinger" , target-devel , stable@vger.kernel.org, Himanshu Madhani , Alexei Potashnik , Quinn Tran , Swapnil Nagle , Giridhar Malavali , Andrew Vasquez Subject: Re: Stable backports for qla2xxx target mode Message-ID: <20150814191935.GC14017@kroah.com> References: <1439499645.31978.20.camel@haakon3.risingtidesystems.com> <20150814004911.GB23652@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Sender: stable-owner@vger.kernel.org List-ID: On Fri, Aug 14, 2015 at 11:37:03AM -0700, Roland Dreier wrote: > On Thu, Aug 13, 2015 at 5:49 PM, Greg KH wrote: > > What exactly are they fixing? They look like they add a ton of new > > functions to the driver, and other features. What is so broken in the > > driver today that warrants this type of exception to the rules? > > Let me start by saying that I'm not sure these changes really belong > in stable, and I totally understand why you're pushing back. > > However the fact is that upstream qla2xxx is utterly broken in target > mode — trying to use Linux as a real FC target, where we have to deal > with fabric changes, initiators going away, and so on, is hopeless > without these changes. The target will crash, or data will be > corrupted, or similarly severe issues. > > We were definitely derelict in getting our fixes upstream for a while, > but they are there now. So using a new kernel has a chance at > working. I think it's totally legitimate to say that the changes are > too big for -stable, and someone who wants to use qla2xxx target mode > needs a recent kernel or their own backports. But as I said, the bugs > are pretty severe (kernel crash or worse) so I could see pulling the > changes into -stable also. Ok, so given that it seems this code has always been broken in this mode of operation, and the patches are huge, and they aren't even in a normally released kernel, I'm not going to take them into -stable at this point in time. Feel free to resend them after about 6 months or so, after the fallout of these major changes has made their way to the distros and users, and you have it all working correctly, and I'll be glad to revisit them. thanks, greg k-h