On 06/24/2015 10:53 AM, Raghavendra Gowdappa wrote: > > > ----- Original Message ----- >> From: "Atin Mukherjee" <amukherj@xxxxxxxxxx> >> To: "Niels de Vos" <ndevos@xxxxxxxxxx>, "Vijaikumar M" <vmallika@xxxxxxxxxx> >> Cc: "Raghavendra G" <raghavendra@xxxxxxxxxxx>, "Gluster Devel" <gluster-devel@xxxxxxxxxxx> >> Sent: Wednesday, June 24, 2015 10:15:12 AM >> Subject: Re: /tests/bugs/quota/bug-1153964.t is consistently failing >> >> When is this patch getting merged, this is blocking other patches to get in. > > Revert of http://review.gluster.org/11311, is waiting for regression runs to pass. There are three patches (duplicates of each other). If anyone of them pass both regression runs, I'll merge them. As far as refcounting mechanism go, it'll take some time to review and merge the patch. Once the revert patch is merged, we are good to go. Please let us know once it's merged as post that all patches need rebase. ~Atin > >> >> ~Atin >> >> On 06/23/2015 06:26 PM, Niels de Vos wrote: >>> On Tue, Jun 23, 2015 at 05:30:39PM +0530, Vijaikumar M wrote: >>>> >>>> >>>> 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. >>> >>> Ok, thanks! >>> Niels >>> >>>> >>>> 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.shankar@xxxxxxxxx >>>>>>> <mailto:yknev.shankar@xxxxxxxxx>> wrote: >>>>>>> >>>>>>> OK. Two reverts of the same patch ;) >>>>>>> >>>>>>> Pick one. >>>>>>> >>>>>>> On Tue, Jun 23, 2015 at 12:51 PM, Raghavendra Gowdappa >>>>>>> <rgowdapp@xxxxxxxxxx <mailto:rgowdapp@xxxxxxxxxx>> 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" <xhernandez@xxxxxxxxxx >>>>>>> <mailto:xhernandez@xxxxxxxxxx>> >>>>>>> >> To: "Gluster Devel" <gluster-devel@xxxxxxxxxxx >>>>>>> <mailto:gluster-devel@xxxxxxxxxxx>> >>>>>>> >> Sent: Tuesday, June 23, 2015 12:44:47 PM >>>>>>> >> Subject: /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@xxxxxxxxxxx <mailto:Gluster-devel@xxxxxxxxxxx> >>>>>>> >> http://www.gluster.org/mailman/listinfo/gluster-devel >>>>>>> >> >>>>>>> > _______________________________________________ >>>>>>> > Gluster-devel mailing list >>>>>>> > Gluster-devel@xxxxxxxxxxx <mailto:Gluster-devel@xxxxxxxxxxx> >>>>>>> > http://www.gluster.org/mailman/listinfo/gluster-devel >>>>>>> _______________________________________________ >>>>>>> Gluster-devel mailing list >>>>>>> Gluster-devel@xxxxxxxxxxx <mailto:Gluster-devel@xxxxxxxxxxx> >>>>>>> http://www.gluster.org/mailman/listinfo/gluster-devel >>>>>>> >>>>>>> >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> Raghavendra G >>>>>>> >>>>>>> >>>>>>> _______________________________________________ >>>>>>> Gluster-devel mailing list >>>>>>> Gluster-devel@xxxxxxxxxxx >>>>>>> http://www.gluster.org/mailman/listinfo/gluster-devel >>>>>> _______________________________________________ >>>>>> Gluster-devel mailing list >>>>>> Gluster-devel@xxxxxxxxxxx >>>>>> http://www.gluster.org/mailman/listinfo/gluster-devel >>>> >>> _______________________________________________ >>> Gluster-devel mailing list >>> Gluster-devel@xxxxxxxxxxx >>> http://www.gluster.org/mailman/listinfo/gluster-devel >>> >> >> -- >> ~Atin >> _______________________________________________ >> Gluster-devel mailing list >> Gluster-devel@xxxxxxxxxxx >> http://www.gluster.org/mailman/listinfo/gluster-devel >> -- ~Atin _______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://www.gluster.org/mailman/listinfo/gluster-devel
