Re: [ceph-users] Ceph mds laggy and failed assert in function replay mds/journal.cc

2014-04-28 Thread Yan, Zheng
On Tue, Apr 29, 2014 at 11:24 AM, Jingyuan Luke wrote: > Hi, > > We had applied the patch and recompile ceph as well as updated the > ceph.conf as per suggested, when we re-run ceph-mds we noticed the > following: > > > 2014-04-29 10:45:22.260798 7f90b971d700 0 log [WRN] : replayed op > client.3

Re: [ceph-users] Ceph mds laggy and failed assert in function replay mds/journal.cc

2014-04-28 Thread Jingyuan Luke
Hi, We had applied the patch and recompile ceph as well as updated the ceph.conf as per suggested, when we re-run ceph-mds we noticed the following: 2014-04-29 10:45:22.260798 7f90b971d700 0 log [WRN] : replayed op client.324186:51366457,12681393 no session for client.324186 2014-04-29 10:45:2

Re: xio-rados-firefly branch update

2014-04-28 Thread Matt W. Benjamin
Hi Greg, - "Gregory Farnum" wrote: > > The re-org mostly looks fine. > I notice you're adding a few more "friend" declarations though, and I > don't think those should be necessary — Connection can label stuff as > protected instead of private to help with that. > PipeConnection should prob

Re: xio-rados-firefly branch update

2014-04-28 Thread Josh Durgin
On 04/28/2014 04:40 PM, Gregory Farnum wrote: The buffer changes appear to have some unnecessary Xio class declarations in buffer.h, and it’d be nice if all of that was guarded by HAVE_XIO blocks. buffer.h is exposed by the librados c++ api, so hopefully nothing xio-specific really needs to be

Re: xio-rados-firefly branch update

2014-04-28 Thread Gregory Farnum
A few days later than I wanted, but I got through various pieces of this today. It wasn't a thorough review but more a "shape of things" check, but I have a bunch of notes. On Tue, Apr 22, 2014 at 2:50 PM, Matt W. Benjamin wrote: > Hi Greg, > > Sure. I'm interested in all feedback you come up wi

master vs firefly branch

2014-04-28 Thread Sage Weil
We're separating the two branches now, so please be sure that anything destined for firefly (which should be almost nothing at this point) needs to target the firefly branch. This will let us start integrating post-firefly stuff into master now. Thanks! sage -- To unsubscribe from this list:

[PATCH] locks: ensure that fl_owner is always initialized properly in flock and lease codepaths

2014-04-28 Thread Jeff Layton
Currently, the fl_owner isn't set for flock locks. Some filesystems use byte-range locks to simulate flock locks and there is a common idiom in those that does: fl->fl_owner = (fl_owner_t)filp; fl->fl_start = 0; fl->fl_end = OFFSET_MAX; Since flock locks are generally "owned" by the o

[ANN] ceph-deploy 1.5.0 released!

2014-04-28 Thread Alfredo Deza
Hi All, There is a new release of ceph-deploy, the easy deployment tool for Ceph. This release comes with a few bug fixes and a few features: * implement `osd list` * add a status check on OSDs when deploying * sync local mirrors to remote hosts when installing * support flags and options set in

Re: [PATCH] locks: ensure that fl_owner is always initialized properly in flock and lease codepaths

2014-04-28 Thread Greg Kroah-Hartman
On Mon, Apr 28, 2014 at 01:50:13PM -0400, Jeff Layton wrote: > Currently, the fl_owner isn't set for flock locks. Some filesystems use > byte-range locks to simulate flock locks and there is a common idiom in > those that does: > > fl->fl_owner = (fl_owner_t)filp; > fl->fl_start = 0; >

ceph branch status

2014-04-28 Thread ceph branch robot
-- All Branches -- Alfredo Deza 2013-09-27 10:33:52 -0400 wip-5900 Dan Mick 2013-07-16 23:00:06 -0700 wip-5634 2014-03-24 21:06:21 -0700 wip-sockaddr 2014-03-28 18:45:51 -0700 wip-fix-testcrypto Danny Al-Gaaf 2014-04-23 16:09:28 +0200 wip-da-S

Re: [PATCH] ceph: delete unnecessary condition judgment

2014-04-28 Thread Alex Elder
On 04/28/2014 01:27 AM, Zhang Zhen wrote: If the return number of ceph_osdc_readpages() is not less than zero, it is certainly greater than or equal to zero. So the condition judgment is unnecessary. Looks good. Reviewed-by: Alex Elder Signed-off-by: Zhang Zhen --- fs/ceph/addr.c | 3 +-