is function get_net_marked_down right?

[Cheng Wei-Chung <freeze.vicente.cheng@xxxxxxxxx>]

Dear all:

I have seen this function few days.
OSDMap::Incremental::get_net_marked_down

There is something strange I don't know why.

why we increase n when we got state and previous state is UP?
if (p->second & CEPH_OSD_UP) {
      if (previous->is_up(p->first))
        n++;  // marked down
      else
        n--;  // marked up
}

This function aims to help check whether the osd can be mark down or not, right?

For counting the osd that state is down, the above function is
counting the osd that state is up?

I would like to know there is some minor error or something I don't
really understand.

Thanks!!!!
--
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