Ceph » CephFS

This came up from QA team,

$ setfattr -n ceph.quota.max_bytes -v 1TG /mnt/ceph-fuse/test_quota/
$ setfattr -n ceph.quota.max_bytes -v 1GT /mnt/ceph-fuse/test_quota/
$ setfattr -n ceph.quota.max_bytes -v KT /mnt/ceph-fuse/test_quota/
$ getfattr -n ceph.quota.max_bytes /mnt/ceph-fuse/test_quota/
getfattr: Removing leading '/' from absolute path names
file: mnt/ceph-fuse/test_quota/ ceph.quota.max_bytes="1024" 

The function strict_iec_cast() is implemented in such a way that it only checks for first char i.e. when you apply TG it considers it as Ti, GT as Gi and KT as Ki. This looks to be a corner case but the issue is this is the common code (from ceph/src/common/strtol.cc) i.e. this function is used across ceph and and not only limited to CephFS. Currently used at:

./mds/Server.cc:5971:      int64_t q = strict_iec_cast<int64_t>(value, &cast_err);
./mon/OSDMonitor.cc:8320:    n = strict_iec_cast<int64_t>(val, &interr);
./rgw/rgw_admin.cc:3745:      max_size = strict_iec_cast<long long>(val, &err);

Also add testcases in src/test/strtol.cc