On Tuesday 23 June 2015 04:28 PM, Niels de Vos wrote:
On Tue, Jun 23, 2015 at 03:45:43PM +0530, Vijaikumar M wrote:
I have submitted below patch which fixes this issue. I am handling memory
clean-up with reference countmechanism.

http://review.gluster.org/#/c/11361
Is there a reason you can not use the (new) refcounting functions that
were introduceed with http://review.gluster.org/11022 ?

I was not aware that ref-counting patch was merged. Sure we will use these function and re-submit my patch.

Thanks,
Vijay


It would be nicer to standardize all refcounting mechanisms on one
implementation. I hope we can replace existing refcounting with this one
too. Introducing more refcounting ways is not going to be helpful.

Thanks,
Niels

Thanks,
Vijay




On Tuesday 23 June 2015 12:58 PM, Raghavendra G wrote:
Multiple replies to same query. Pick one ;).

On Tue, Jun 23, 2015 at 12:55 PM, Venky Shankar <yknev.shan...@gmail.com
<mailto:yknev.shan...@gmail.com>> wrote:

    OK. Two reverts of the same patch ;)

    Pick one.

    On Tue, Jun 23, 2015 at 12:51 PM, Raghavendra Gowdappa
    <rgowd...@redhat.com <mailto:rgowd...@redhat.com>> wrote:
    > Seems like its a memory corruption caused by:
    > http://review.gluster.org/11311
    >
    > I've reverted the patch at:
    > http://review.gluster.org/11360
    >
    > ----- Original Message -----
    >> From: "Xavier Hernandez" <xhernan...@datalab.es
    <mailto:xhernan...@datalab.es>>
    >> To: "Gluster Devel" <gluster-devel@gluster.org
    <mailto:gluster-devel@gluster.org>>
    >> Sent: Tuesday, June 23, 2015 12:44:47 PM
    >> Subject: [Gluster-devel] /tests/bugs/quota/bug-1153964.t is
    consistently failing
    >>
    >> Hi,
    >>
    >> the quota test bug-1153964.t is failing consistently for a totally
    >> unrelated patch. Is this a known issue ?
    >>
    >>
    
http://build.gluster.org/job/rackspace-regression-2GB-triggered/11142/consoleFull
    >>
    
http://build.gluster.org/job/rackspace-regression-2GB-triggered/11165/consoleFull
    >>
    
http://build.gluster.org/job/rackspace-regression-2GB-triggered/11172/consoleFull
    >>
    
http://build.gluster.org/job/rackspace-regression-2GB-triggered/11191/consoleFull
    >>
    >> Xavi
    >> _______________________________________________
    >> Gluster-devel mailing list
    >> Gluster-devel@gluster.org <mailto:Gluster-devel@gluster.org>
    >> http://www.gluster.org/mailman/listinfo/gluster-devel
    >>
    > _______________________________________________
    > Gluster-devel mailing list
    > Gluster-devel@gluster.org <mailto:Gluster-devel@gluster.org>
    > http://www.gluster.org/mailman/listinfo/gluster-devel
    _______________________________________________
    Gluster-devel mailing list
    Gluster-devel@gluster.org <mailto:Gluster-devel@gluster.org>
    http://www.gluster.org/mailman/listinfo/gluster-devel




--
Raghavendra G


_______________________________________________
Gluster-devel mailing list
Gluster-devel@gluster.org
http://www.gluster.org/mailman/listinfo/gluster-devel
_______________________________________________
Gluster-devel mailing list
Gluster-devel@gluster.org
http://www.gluster.org/mailman/listinfo/gluster-devel

_______________________________________________
Gluster-devel mailing list
Gluster-devel@gluster.org
http://www.gluster.org/mailman/listinfo/gluster-devel

Reply via email to