lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:	Wed, 30 Apr 2014 17:46:05 +0100
From:	Andrew Bennieston <andrew.bennieston@...rix.com>
To:	Wei Liu <wei.liu2@...rix.com>
CC:	<xen-devel@...ts.xenproject.org>, <davem@...emloft.net>,
	<ian.campbell@...rix.com>, <paul.durrant@...rix.com>,
	<netdev@...r.kernel.org>, <david.vrabel@...rix.com>,
	<zoltan.kiss@...rix.com>
Subject: Re: [PATCH V7 net-next 4/7] xen-netback: Correctly clean up after
 queue initialisation error.

On 30/04/14 15:49, Wei Liu wrote:
> On Tue, Apr 29, 2014 at 02:27:12PM +0100, Andrew J. Bennieston wrote:
>> From: "Andrew J. Bennieston" <andrew.bennieston@...rix.com>
>>
>> Multiple queues were introduced in the previous patch, but if the
>> initialisation of a queue failed, earlier queues would not be cleaned
>> up. This patch adds cleanup logic for this scenario, making use of code
>> from the regular teardown sequence.
>>
>
> Shouldn't this patch be folded into that "previous patch" which
> introduced this problem? I see you did this for netfront patch already.
>
> Wei.
>
The netfront patch included this from the beginning; I simply forgot to 
implement it in netback. I added it as a separate patch to make it 
easier to review, but I can certainly merge it in with the other one 
(and indeed should do so to maintain correct behaviour e.g. during a 
bisect).

I think I'd rather receive comments on this separately, then if 
necessary either repost the series with this folded into the relevant 
patch for final merge, or post V8 with it folded after comments have 
been addressed. Especially since this patch series already has some very 
long patches...

Andrew.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Powered by blists - more mailing lists