Re: [Xen-devel] [PATCH v2 2/2] xen/scsiback: avoid warnings when adding multiple LUNs to a domain

From: David Vrabel <david.vrabel@citrix.com>
To: Juergen Gross <jgross@suse.com>, <linux-kernel@vger.kernel.org>,
	<xen-devel@lists.xen.org>, <konrad.wilk@oracle.com>,
	<boris.ostrovsky@oracle.com>
Cc: <stable@vger.kernel.org>
Subject: Re: [Xen-devel] [PATCH v2 2/2] xen/scsiback: avoid warnings when adding multiple LUNs to a domain
Date: Mon, 8 Feb 2016 16:52:22 +0000	[thread overview]
Message-ID: <56B8C7C6.4010903@citrix.com> (raw)
In-Reply-To: <56B8C765.4000707@suse.com>

On 08/02/16 16:50, Juergen Gross wrote:
> On 08/02/16 17:46, David Vrabel wrote:
>> On 08/02/16 14:30, Juergen Gross wrote:
>>> When adding more than one LUN to a frontend a warning for a failed
>>> assignment is issued in dom0 for each already existing LUN. Avoid this
>>> warning by checking for a LUN already existing when existence is
>>> allowed (scsiback_do_add_lun() called with try == 1).
>>>
>>> As the LUN existence check is needed now for a third time, factor it
>>> out into a function. This in turn leads to a more or less complete
>>> rewrite of scsiback_del_translation_entry() which will now return a
>>> proper error code in case of failure.
>>>
>>> Signed-off-by: Juergen Gross <jgross@suse.com>
>>> Cc: stable@vger.kernel.org
>>
>> Avoiding warnings doesn't seem like a viable candidate for stable.  Is
>> there more to this patch than the description suggests?
> 
> No.
> 
> Should I resend without the Cc: stable?

No, I've dropped it.

I've applied these two to for-linus-4.5, thanks.

David