cluster-devel.redhat.com archive mirror
 help / color / mirror / Atom feed
* [Cluster-devel] [PATCH] dlm, sctp: Do not allocate a fd for peeloff
       [not found] ` <4F578CE0.6030106@hp.com>
@ 2012-03-08  8:24   ` David Miller
       [not found]     ` <1331222159-28980-1-git-send-email-bpoirier@suse.de>
  0 siblings, 1 reply; 3+ messages in thread
From: David Miller @ 2012-03-08  8:24 UTC (permalink / raw)
  To: cluster-devel.redhat.com

From: Vladislav Yasevich <vladislav.yasevich@hp.com>
Date: Wed, 07 Mar 2012 11:29:20 -0500

> On 03/07/2012 10:41 AM, Benjamin Poirier wrote:
>> avoids allocating a fd that a) propagates to every kernel thread and
>> usermodehelper b) is not properly released.
>> 
>> References: http://article.gmane.org/gmane.linux.network.drbd/22529
>> Signed-off-by: Benjamin Poirier <bpoirier@suse.de>
> 
> It might make sense to change sctp_do_peeloff to take the association id as
> the first argument and not do the mapping from id to association yourself.
> It's a bit ugly to expose internal sctp structures outside of SCTP.

Agreed.



^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Cluster-devel] [PATCH v2 1/2] sctp: Export sctp_do_peeloff
       [not found]       ` <4F59118C.9090605@hp.com>
@ 2012-03-08 21:52         ` David Miller
  0 siblings, 0 replies; 3+ messages in thread
From: David Miller @ 2012-03-08 21:52 UTC (permalink / raw)
  To: cluster-devel.redhat.com

From: Vladislav Yasevich <vladislav.yasevich@hp.com>
Date: Thu, 08 Mar 2012 15:07:40 -0500

> On 03/08/2012 10:55 AM, Benjamin Poirier wrote:
>> lookup sctp_association within sctp_do_peeloff() to enable its use outside of
>> the sctp code with minimal knowledge of the former.
>> 
>> Signed-off-by: Benjamin Poirier <bpoirier@suse.de>
>> 
> 
> Acked-by: Vlad Yasevich <vladislav.yasevich@hp.com>

Applied to net-next.



^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Cluster-devel] [PATCH v2 2/2] dlm: Do not allocate a fd for peeloff
       [not found]         ` <4F5911B1.6050600@hp.com>
@ 2012-03-08 21:52           ` David Miller
  0 siblings, 0 replies; 3+ messages in thread
From: David Miller @ 2012-03-08 21:52 UTC (permalink / raw)
  To: cluster-devel.redhat.com

From: Vladislav Yasevich <vladislav.yasevich@hp.com>
Date: Thu, 08 Mar 2012 15:08:17 -0500

> On 03/08/2012 10:55 AM, Benjamin Poirier wrote:
>> avoids allocating a fd that a) propagates to every kernel thread and
>> usermodehelper b) is not properly released.
>> 
>> References: http://article.gmane.org/gmane.linux.network.drbd/22529
>> Signed-off-by: Benjamin Poirier <bpoirier@suse.de>
> 
> Looks much better.

Also applied to net-next, thanks.



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-03-08 21:52 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1331134862-14812-1-git-send-email-bpoirier@suse.de>
     [not found] ` <4F578CE0.6030106@hp.com>
2012-03-08  8:24   ` [Cluster-devel] [PATCH] dlm, sctp: Do not allocate a fd for peeloff David Miller
     [not found]     ` <1331222159-28980-1-git-send-email-bpoirier@suse.de>
     [not found]       ` <4F59118C.9090605@hp.com>
2012-03-08 21:52         ` [Cluster-devel] [PATCH v2 1/2] sctp: Export sctp_do_peeloff David Miller
     [not found]       ` <1331222159-28980-2-git-send-email-bpoirier@suse.de>
     [not found]         ` <4F5911B1.6050600@hp.com>
2012-03-08 21:52           ` [Cluster-devel] [PATCH v2 2/2] dlm: Do not allocate a fd for peeloff David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).