Re: [PATCH 2/3] ceph: fix bounds check macros ceph_decode_need and ceph_encode_need

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/18/2012 12:53 PM, Sage Weil wrote:
Would something like ceph_has_room() make more sense?

Yes, and standing by itself it reads a lot better too.

The "need_ok" came from the fact that it was only ever
called by ceph_need_*() macros.

Without objection I'll change it to ceph_has_room()
before I commit it for testing.

					-Alex
--
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


[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux