summaryrefslogtreecommitdiff
path: root/src/osd/OSD.cc
diff options
context:
space:
mode:
Diffstat (limited to 'src/osd/OSD.cc')
-rw-r--r--src/osd/OSD.cc77
1 files changed, 29 insertions, 48 deletions
diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc
index 5b260dd93a6..42437fc8ad5 100644
--- a/src/osd/OSD.cc
+++ b/src/osd/OSD.cc
@@ -98,7 +98,7 @@
#define dout_prefix _prefix(*_dout, whoami, osdmap)
static ostream& _prefix(ostream& out, int whoami, OSDMap *osdmap) {
- return out << dbeginl << "osd" << whoami << " " << (osdmap ? osdmap->get_epoch():0) << " ";
+ return out << "osd" << whoami << " " << (osdmap ? osdmap->get_epoch():0) << " ";
}
const coll_t coll_t::META_COLL("meta");
@@ -342,7 +342,7 @@ OSD::OSD(int id, Messenger *internal_messenger, Messenger *external_messenger, M
logger(NULL), logger_started(false),
store(NULL),
map_in_progress(false),
- logclient(client_messenger, &mc->monmap, mc),
+ clog(client_messenger, &mc->monmap, mc, LogClient::NO_FLAGS),
whoami(id),
dev_path(dev), journal_path(jdev),
dispatch_running(false),
@@ -518,7 +518,7 @@ int OSD::init()
// i'm ready!
client_messenger->add_dispatcher_head(this);
- client_messenger->add_dispatcher_head(&logclient);
+ client_messenger->add_dispatcher_head(&clog);
cluster_messenger->add_dispatcher_head(this);
heartbeat_messenger->add_dispatcher_head(&heartbeat_dispatcher);
@@ -787,7 +787,7 @@ int OSD::read_superblock()
}
if (whoami != superblock.whoami) {
- derr(0) << "read_superblock superblock says osd" << superblock.whoami
+ dout(0) << "read_superblock superblock says osd" << superblock.whoami
<< ", but i (think i) am osd" << whoami << dendl;
return -1;
}
@@ -1515,8 +1515,6 @@ void OSD::tick()
logger->set(l_osd_buf, buffer_total_alloc.read());
- _dout_check_log();
-
if (got_sigterm) {
dout(0) << "got SIGTERM, shutting down" << dendl;
cluster_messenger->send_message(new MGenericMessage(CEPH_MSG_SHUTDOWN),
@@ -1562,7 +1560,7 @@ void OSD::tick()
map_lock.put_read();
- logclient.send_log();
+ clog.send_log();
timer.add_event_after(1.0, new C_Tick(this));
@@ -1835,59 +1833,50 @@ void OSD::handle_command(MMonCommand *m)
stringstream ss;
uint64_t rate = (double)count / (end - start);
- ss << "bench: wrote " << prettybyte_t(count) << " in blocks of " << prettybyte_t(bsize)
- << " in " << (end-start)
- << " sec at " << prettybyte_t(rate) << "/sec";
- logclient.log(LOG_INFO, ss);
+ clog.info() << "bench: wrote " << prettybyte_t(count)
+ << " in blocks of " << prettybyte_t(bsize) << " in "
+ << (end-start) << " sec at " << prettybyte_t(rate) << "/sec\n";
} else if (m->cmd.size() == 2 && m->cmd[0] == "logger" && m->cmd[1] == "reset") {
logger_reset_all();
} else if (m->cmd.size() == 2 && m->cmd[0] == "logger" && m->cmd[1] == "reopen") {
logger_reopen_all();
} else if (m->cmd.size() == 1 && m->cmd[0] == "heapdump") {
- stringstream ss;
if (g_conf.tcmalloc_have) {
if (!g_conf.profiler_running()) {
- ss << "can't dump heap: profiler not running";
+ clog.info() << "can't dump heap: profiler not running\n";
} else {
- ss << g_conf.name << "dumping heap profile now";
+ clog.info() << g_conf.name << "dumping heap profile now\n";
g_conf.profiler_dump("admin request");
}
} else {
- ss << g_conf.name << " does not have tcmalloc, can't use profiler";
+ clog.info() << g_conf.name << " does not have tcmalloc, "
+ "can't use profiler\n";
}
- logclient.log(LOG_INFO, ss);
} else if (m->cmd.size() == 1 && m->cmd[0] == "enable_profiler_options") {
char val[sizeof(int)*8+1];
snprintf(val, sizeof(val), "%i", g_conf.profiler_allocation_interval);
setenv("HEAP_PROFILE_ALLOCATION_INTERVAL", val, g_conf.profiler_allocation_interval);
snprintf(val, sizeof(val), "%i", g_conf.profiler_highwater_interval);
setenv("HEAP_PROFILE_INUSE_INTERVAL", val, g_conf.profiler_highwater_interval);
- stringstream ss;
- ss << g_conf.name << " set heap variables from current config";
- logclient.log(LOG_INFO, ss);
+ clog.info() << g_conf.name << " set heap variables from current config";
} else if (m->cmd.size() == 1 && m->cmd[0] == "start_profiler") {
char location[PATH_MAX];
snprintf(location, sizeof(location),
"%s/%s", g_conf.log_dir, g_conf.name);
g_conf.profiler_start(location);
- stringstream ss;
- ss << g_conf.name << " started profiler with output " << location;
- logclient.log(LOG_INFO, ss);
+ clog.info() << g_conf.name << " started profiler with output "
+ << location << "\n";
} else if (m->cmd.size() == 1 && m->cmd[0] == "stop_profiler") {
g_conf.profiler_stop();
- stringstream ss;
- ss << g_conf.name << " stopped profiler";
- logclient.log(LOG_INFO, ss);
+ clog.info() << g_conf.name << " stopped profiler\n";
}
else if (m->cmd.size() > 1 && m->cmd[0] == "debug") {
if (m->cmd.size() == 3 && m->cmd[1] == "dump_missing") {
const string &file_name(m->cmd[2]);
std::ofstream fout(file_name.c_str());
if (!fout.is_open()) {
- stringstream ss;
- ss << "failed to open file '" << file_name << "'";
- logclient.log(LOG_INFO, ss);
+ clog.info() << "failed to open file '" << file_name << "'\n";
goto done;
}
@@ -1927,10 +1916,8 @@ void OSD::handle_command(MMonCommand *m)
}
else if (m->cmd.size() == 3 && m->cmd[1] == "kick_recovery_wq") {
g_conf.osd_recovery_delay_start = atoi(m->cmd[2].c_str());
- stringstream ss;
- ss << "kicking recovery queue. set osd_recovery_delay_start to "
- << g_conf.osd_recovery_delay_start;
- logclient.log(LOG_INFO, ss);
+ clog.info() << "kicking recovery queue. set osd_recovery_delay_start "
+ << "to " << g_conf.osd_recovery_delay_start << "\n";
defer_recovery_until = g_clock.now();
defer_recovery_until += g_conf.osd_recovery_delay_start;
@@ -2745,9 +2732,8 @@ void OSD::handle_osd_map(MOSDMap *m)
do_shutdown = true; // don't call shutdown() while we have everything paused
} else if (!osdmap->is_up(whoami) ||
osdmap->get_addr(whoami) != client_messenger->get_myaddr()) {
- stringstream ss;
- ss << "map e" << osdmap->get_epoch() << " wrongly marked me down";
- logclient.log(LOG_WARN, ss);
+ clog.warn() << "map e" << osdmap->get_epoch()
+ << " wrongly marked me down\n";
state = STATE_BOOTING;
up_epoch = 0;
@@ -3589,9 +3575,8 @@ void OSD::handle_pg_create(MOSDPGCreate *m)
}
if (up != acting) {
dout(10) << "mkpg " << pgid << " up " << up << " != acting " << acting << dendl;
- stringstream ss;
- ss << "mkpg " << pgid << " up " << up << " != acting " << acting;
- logclient.log(LOG_ERROR, ss);
+ clog.error() << "mkpg " << pgid << " up " << up << " != acting "
+ << acting << "\n";
continue;
}
@@ -3992,10 +3977,8 @@ void OSD::_process_pg_info(epoch_t epoch, int from,
// did a snap just get purged?
if (info.purged_snaps.size() < pg->info.purged_snaps.size()) {
- stringstream ss;
- ss << "pg " << pg->info.pgid << " replica got purged_snaps " << info.purged_snaps
- << " had " << pg->info.purged_snaps;
- logclient.log(LOG_WARN, ss);
+ clog.warn() << "pg " << pg->info.pgid << " replica got purged_snaps "
+ << info.purged_snaps << " had " << pg->info.purged_snaps << "\n";
pg->info.purged_snaps = info.purged_snaps;
} else {
interval_set<snapid_t> p = info.purged_snaps;
@@ -4751,11 +4734,9 @@ void OSD::handle_misdirected_op(PG *pg, MOSDOp *op)
op->put();
} else {
dout(7) << *pg << " misdirected op in " << op->get_map_epoch() << dendl;
- stringstream ss;
- ss << op->get_source_inst() << " misdirected " << op->get_reqid()
- << " " << pg->info.pgid << " to osd" << whoami
- << " not " << pg->acting;
- logclient.log(LOG_WARN, ss);
+ clog.warn() << op->get_source_inst() << " misdirected "
+ << op->get_reqid() << " " << pg->info.pgid << " to osd" << whoami
+ << " not " << pg->acting << "\n";
reply_op_error(op, -ENXIO);
}
}
@@ -4845,7 +4826,7 @@ void OSD::handle_op(MOSDOp *op)
if (!op->may_write()) {
stat_rd_ops++;
if (op->get_source().is_osd()) {
- //derr(-10) << "shed in " << stat_rd_ops_shed_in << " / " << stat_rd_ops << dendl;
+ //dout(-10) << "shed in " << stat_rd_ops_shed_in << " / " << stat_rd_ops << dendl;
stat_rd_ops_shed_in++;
}
}