On Thu, Apr 21, 2016 at 12:11 PM, Jaze Lee <jazeltq@xxxxxxxxx> wrote: > hi all > I found when osds report a target osd failure, the monitor encode > CEPH_OSD_UP in > the pend_inc. Why not put CEPH_OSD_DOWN there? > The code is here > https://github.com/ceph/ceph/blob/master/src/mon/OSDMonitor.cc#L1777 > > May be i miss something? Or my understanding is wrong? Can someone > who know that > explain a little more? you might want to check https://github.com/ceph/ceph/blob/master/src/osd/OSDMap.cc#L1348, so the new state is xor(old_state, pending_state), the same applies to CEPH_OSD_EXISTS. > > Thanks a lot.... > > > > > > -- > 谦谦君子 > -- > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Regards Kefu Chai -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html