All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adrian Bridgett <adrian@smop.co.uk>
To: Andrew Morton <akpm@osdl.org>,
	linux-kernel@vger.kernel.org,
	"David S. Miller" <davem@davemloft.net>, Andi Kleen <ak@muc.de>
Subject: Re: 2.6.16-mm1 leaks in dvb playback (found)
Date: Thu, 30 Mar 2006 23:58:30 +0100	[thread overview]
Message-ID: <20060330225830.GA24009@smop.co.uk> (raw)
In-Reply-To: <20060330004518.GA23404@smop.co.uk>

What I thought was just one patch was actually two and it was the
other patch causing the problem - "Do not lose accepted socket when
-ENFILE/-EMFILE".

Most of the patch seems to be just a restructuring - I guess that
leaves the sys_accept changes that are leaking the memory. 

I've no idea how the code is supposed to work, so large rocks of salt
required :-)  The code now does a sock_alloc_fd, and the error cases
now do a "put_filp" and "put_unused_fd" if the alloc succeeded.
However in the normal case, nothing gets freed (I guess that's the
memory leak).  OTOH the description of the patch is:

 "Try to allocate the struct file and an unused file
 descriptor before we try to pull a newly accepted
 socket out of the protocol layer."

http://www.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=39d8c1b6fbaeb8d6adec4a8c08365cc9eaca6ae4

Cheers,

Adrian

  reply	other threads:[~2006-03-30 22:58 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-26 21:15 2.6.16-mm1 leaks in dvb playback Adrian Bridgett
2006-03-28  1:23 ` Andrew Morton
2006-03-28  7:02   ` adrian
2006-03-28  7:16     ` Andrew Morton
2006-03-29 23:37       ` adrian
2006-03-30  0:06         ` Andrew Morton
2006-03-30  0:45           ` Adrian Bridgett
2006-03-30 22:58             ` Adrian Bridgett [this message]
2006-03-30 23:11               ` 2.6.16-mm1 leaks in dvb playback (found) Adrian Bridgett
2006-03-30 23:28                 ` David S. Miller
2006-03-31  1:22                   ` Andi Kleen
2006-03-31  7:28                     ` Adrian Bridgett
2006-03-31  7:48                       ` David S. Miller
2006-03-31  9:54                         ` Adrian Bridgett
2006-03-31 10:07                           ` David S. Miller
2006-03-31 18:47                             ` Adrian Bridgett
2006-03-28  7:23     ` 2.6.16-mm1 leaks in dvb playback adrian

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060330225830.GA24009@smop.co.uk \
    --to=adrian@smop.co.uk \
    --cc=ak@muc.de \
    --cc=akpm@osdl.org \
    --cc=davem@davemloft.net \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.