thebestvibedev/tromcho.net
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Do not use numbered printf args

Browse Source
This commit is contained in:
Vitaliy Filippov
2025-07-26 14:11:15 +03:00
parent 83fb121f36
commit 93a30efd86
2 changed files with 16 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/osd/osd_peering.cpp
+8 -9
View File
@@ -488,15 +488,14 @@ void osd_t::relock_pg(pg_t & pg)
} }
else if (op->reply.hdr.retval != -EPIPE) else if (op->reply.hdr.retval != -EPIPE)
{ {
printf( printf("Failed to %s PG %u/%u on OSD %ju - ", op->req.sec_lock.flags == OSD_SEC_UNLOCK_PG ? "unlock" : "lock",
(op->reply.hdr.retval == -ENOENT pg_id.pool_id, pg_id.pg_num, peer_osd);
? "Failed to %1$s PG %2$u/%3$u on OSD %4$ju - peer didn't load PG info yet\n" if (op->reply.hdr.retval == -ENOENT)
: (op->reply.sec_lock.cur_primary printf("peer didn't load PG info yet\n");
? "Failed to %1$s PG %2$u/%3$u on OSD %4$ju - taken by OSD %6$ju (retval=%5$jd)\n" else if (op->reply.sec_lock.cur_primary)
: "Failed to %1$s PG %2$u/%3$u on OSD %4$ju - retval=%5$jd\n")), printf("taken by OSD %ju (retval=%jd)\n", op->reply.sec_lock.cur_primary, op->reply.hdr.retval);
op->req.sec_lock.flags == OSD_SEC_UNLOCK_PG ? "unlock" : "lock", else
pg_id.pool_id, pg_id.pg_num, peer_osd, op->reply.hdr.retval, op->reply.sec_lock.cur_primary printf("retval=%jd\n", op->reply.hdr.retval);
);
// Retry relocking/unlocking PG after a short time // Retry relocking/unlocking PG after a short time
pg.lock_waiting = true; pg.lock_waiting = true;
tfd->set_timer(pg_lock_retry_interval_ms, false, [this, pg_id](int) tfd->set_timer(pg_lock_retry_interval_ms, false, [this, pg_id](int)
src/osd/osd_primary_subops.cpp
+8 -9
View File
@@ -420,15 +420,14 @@ void osd_t::handle_primary_subop(osd_op_t *subop, osd_op_t *cur_op)
? msgr.clients[subop->peer_fd]->osd_num : 0); ? msgr.clients[subop->peer_fd]->osd_num : 0);
if (opcode == OSD_OP_SEC_READ || opcode == OSD_OP_SEC_WRITE || opcode == OSD_OP_SEC_WRITE_STABLE) if (opcode == OSD_OP_SEC_READ || opcode == OSD_OP_SEC_WRITE || opcode == OSD_OP_SEC_WRITE_STABLE)
{ {
printf( printf("%s subop to %jx:%jx v%ju failed ", osd_op_names[opcode],
subop->peer_fd >= 0 subop->req.sec_rw.oid.inode, subop->req.sec_rw.oid.stripe, subop->req.sec_rw.version);
? (peer_osd > 0 if (subop->peer_fd >= 0 && peer_osd > 0)
? "%1$s subop to %2$jx:%3$jx v%4$ju failed on osd %7$ju: retval = %5$d (expected %6$d)\n" printf("on osd %ju: retval = %d (expected %d)\n", peer_osd, retval, expected);
: "%1$s subop to %2$jx:%3$jx v%4$ju failed on peer %8$d: retval = %5$d (expected %6$d)\n") else if (peer_osd > 0)
: "%1$s subop to %2$jx:%3$jx v%4$ju failed locally: retval = %5$d (expected %6$d)\n", printf("on peer %d: retval = %d (expected %d)\n", subop->peer_fd, retval, expected);
osd_op_names[opcode], subop->req.sec_rw.oid.inode, subop->req.sec_rw.oid.stripe, subop->req.sec_rw.version, else
retval, expected, peer_osd, subop->peer_fd printf("locally: retval = %d (expected %d)\n", retval, expected);
);
} }
else if (opcode == OSD_OP_SEC_DELETE) else if (opcode == OSD_OP_SEC_DELETE)
{ {