Re: geli(4) memory leak

2011-04-10 Thread Mikolaj Golub
On Sun, 03 Apr 2011 20:43:45 +0300 Mikolaj Golub wrote to Pawel Jakub Dawidek: MG> On Sat, 2 Apr 2011 12:17:50 +0200 Pawel Jakub Dawidek wrote: PJD>> On Sat, Apr 02, 2011 at 12:04:09AM +0300, Mikolaj Golub wrote: >>> For me your patch look correct. But the same issue is for read :-). Also,

Re: geli(4) memory leak

2011-04-03 Thread Mikolaj Golub
On Mon, 4 Apr 2011 01:51:24 +0200 Victor Balada Diaz wrote: VBD> On Sun, Apr 03, 2011 at 08:43:45PM +0300, Mikolaj Golub wrote: >> >> On Sat, 2 Apr 2011 12:17:50 +0200 Pawel Jakub Dawidek wrote: >> >> PJD> On Sat, Apr 02, 2011 at 12:04:09AM +0300, Mikolaj Golub wrote: >> >> For me your

Re: geli(4) memory leak

2011-04-03 Thread Victor Balada Diaz
On Sun, Apr 03, 2011 at 08:43:45PM +0300, Mikolaj Golub wrote: > > On Sat, 2 Apr 2011 12:17:50 +0200 Pawel Jakub Dawidek wrote: > > PJD> On Sat, Apr 02, 2011 at 12:04:09AM +0300, Mikolaj Golub wrote: > >> For me your patch look correct. But the same issue is for read :-). Also, > to > >> avoi

Re: geli(4) memory leak

2011-04-03 Thread Mikolaj Golub
On Sat, 2 Apr 2011 12:17:50 +0200 Pawel Jakub Dawidek wrote: PJD> On Sat, Apr 02, 2011 at 12:04:09AM +0300, Mikolaj Golub wrote: >> For me your patch look correct. But the same issue is for read :-). Also, to >> avoid the leak I think we can just do g_destroy_bio() before "all sectors" >> che

Re: geli(4) memory leak

2011-04-02 Thread Pawel Jakub Dawidek
On Sat, Apr 02, 2011 at 12:04:09AM +0300, Mikolaj Golub wrote: > For me your patch look correct. But the same issue is for read :-). Also, to > avoid the leak I think we can just do g_destroy_bio() before "all sectors" > check. See the attached patch (had some testing). The patch looks good. Pleas

Re: geli(4) memory leak

2011-04-01 Thread Mikolaj Golub
On Fri, 1 Apr 2011 19:43:54 +0200 Victor Balada Diaz wrote: VBD> On Sat, Mar 26, 2011 at 01:33:48AM +0100, Victor Balada Diaz wrote: >> Hello, >> >> I'm trying to setup a new geli disk and i'm seeing what looks like a memory >> leak. >> After initializing the device i've tried to do the d

Re: geli(4) memory leak

2011-04-01 Thread Victor Balada Diaz
On Sat, Mar 26, 2011 at 01:33:48AM +0100, Victor Balada Diaz wrote: > Hello, > > I'm trying to setup a new geli disk and i'm seeing what looks like a memory > leak. > After initializing the device i've tried to do the dd command from /dev/random > like this one: > > dd if=/dev/random of=/dev/da0

geli(4) memory leak

2011-03-25 Thread Victor Balada Diaz
Hello, I'm trying to setup a new geli disk and i'm seeing what looks like a memory leak. After initializing the device i've tried to do the dd command from /dev/random like this one: dd if=/dev/random of=/dev/da0p1.eli bs=1m And i can see that wired memory is increasing and never released, usi