http://review.gluster.org/#/c/11362/ has been merged. ----- Original Message ----- > From: "Atin Mukherjee" <amukherj@xxxxxxxxxx> > To: "Raghavendra Gowdappa" <rgowdapp@xxxxxxxxxx> > Cc: "Niels de Vos" <ndevos@xxxxxxxxxx>, "Vijaikumar M" <vmallika@xxxxxxxxxx>, "Raghavendra G" > <raghavendra@xxxxxxxxxxx>, "Gluster Devel" <gluster-devel@xxxxxxxxxxx> > Sent: Wednesday, June 24, 2015 10:55:02 AM > Subject: Re: /tests/bugs/quota/bug-1153964.t is consistently failing > > > > 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
