diff options
author | David Zafman <david.zafman@inktank.com> | 2013-10-02 09:58:24 -0700 |
---|---|---|
committer | David Zafman <david.zafman@inktank.com> | 2013-10-02 09:58:24 -0700 |
commit | fcbb2663bede3f70fa49a0b028bd48eb91a0c818 (patch) | |
tree | 30f4b9d33c00b9683f66a2d538a61bd0affe9baf | |
parent | 9c12c625bd432551222a5d68713ec869db697926 (diff) | |
download | ceph-fcbb2663bede3f70fa49a0b028bd48eb91a0c818.tar.gz |
Remove no longer needed OSD::write_meta() and OSD::read_meta()
Signed-off-by: David Zafman <david.zafman@inktank.com>
-rw-r--r-- | src/osd/OSD.cc | 28 | ||||
-rw-r--r-- | src/osd/OSD.h | 4 |
2 files changed, 8 insertions, 24 deletions
diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc index 2f86bc431ef..916e002012a 100644 --- a/src/osd/OSD.cc +++ b/src/osd/OSD.cc @@ -712,7 +712,7 @@ int OSD::mkfs(CephContext *cct, const std::string &dev, const std::string &jdev, goto umount_store; } - ret = write_meta(dev, "ready", "ready\n", 6); + ret = safe_write_file(dev.c_str(), "ready", "ready\n", 6); if (ret) { derr << "OSD::mkfs: failed to write ready file: error " << ret << dendl; goto umount_store; @@ -768,31 +768,19 @@ int OSD::dump_journal(CephContext *cct, const std::string &dev, const std::strin return err; } -int OSD::write_meta(const std::string &base, const std::string &file, - const char *val, size_t vallen) -{ - return safe_write_file(base.c_str(), file.c_str(), val, vallen); -} - -int OSD::read_meta(const std::string &base, const std::string &file, - char *val, size_t vallen) -{ - return safe_read_file(base.c_str(), file.c_str(), val, vallen); -} - int OSD::write_meta(const std::string &base, uuid_d& cluster_fsid, uuid_d& osd_fsid, int whoami) { char val[80]; snprintf(val, sizeof(val), "%s\n", CEPH_OSD_ONDISK_MAGIC); - write_meta(base, "magic", val, strlen(val)); + safe_write_file(base.c_str(), "magic", val, strlen(val)); snprintf(val, sizeof(val), "%d\n", whoami); - write_meta(base, "whoami", val, strlen(val)); + safe_write_file(base.c_str(), "whoami", val, strlen(val)); cluster_fsid.print(val); strcat(val, "\n"); - write_meta(base, "ceph_fsid", val, strlen(val)); + safe_write_file(base.c_str(), "ceph_fsid", val, strlen(val)); return 0; } @@ -802,24 +790,24 @@ int OSD::peek_meta(const std::string &dev, std::string& magic, { char val[80] = { 0 }; - if (read_meta(dev, "magic", val, sizeof(val)) < 0) + if (safe_read_file(dev.c_str(), "magic", val, sizeof(val)) < 0) return -errno; int l = strlen(val); if (l && val[l-1] == '\n') val[l-1] = 0; magic = val; - if (read_meta(dev, "whoami", val, sizeof(val)) < 0) + if (safe_read_file(dev.c_str(), "whoami", val, sizeof(val)) < 0) return -errno; whoami = atoi(val); - if (read_meta(dev, "ceph_fsid", val, sizeof(val)) < 0) + if (safe_read_file(dev.c_str(), "ceph_fsid", val, sizeof(val)) < 0) return -errno; if (strlen(val) > 36) val[36] = 0; cluster_fsid.parse(val); - if (read_meta(dev, "fsid", val, sizeof(val)) < 0) + if (safe_read_file(dev.c_str(), "fsid", val, sizeof(val)) < 0) osd_fsid = uuid_d(); else { if (strlen(val) > 36) diff --git a/src/osd/OSD.h b/src/osd/OSD.h index 5fe667344a9..9346cee6890 100644 --- a/src/osd/OSD.h +++ b/src/osd/OSD.h @@ -1738,10 +1738,6 @@ protected: } private: - static int write_meta(const std::string &base, const std::string &file, - const char *val, size_t vallen); - static int read_meta(const std::string &base, const std::string &file, - char *val, size_t vallen); static int write_meta(const std::string &base, uuid_d& cluster_fsid, uuid_d& osd_fsid, int whoami); public: |