[pve-devel] [PATCH cluster] cleanup format strings for cfs_* messages
Thomas Lamprecht
t.lamprecht at proxmox.com
Thu Jun 30 09:49:47 CEST 2016
ping
On 05/24/2016 03:55 PM, Thomas Lamprecht wrote:
> This does not change semantics on our current target platform
> (x86_64) but is needed for porting it to other platforms.
>
> The GCC on ARM, for example, complains about them.
>
> For all:
> * size_t use "%z*"
> * off_t use "%j*"
> * uint64_t use "PRI*64"
>
> where * may be one of (X,d,u).
>
> Also cast guint64 to uint64_t to allow use of a general, portable
> format which also supports hex output as the GUINT64_FORMAT allows
> decimal output only.
>
> Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
> Cc: mir at datanom.net
> ---
>
> This allows me to compile pmxcfs successfully under armhf
> and naturally the already working x86_64.
>
> data/src/cfs-plug.c | 6 +++---
> data/src/database.c | 25 +++++++++++++------------
> data/src/dcdb.c | 29 +++++++++++++++--------------
> data/src/dfsm.c | 23 ++++++++++++-----------
> data/src/logger.c | 3 ++-
> data/src/memdb.c | 17 +++++++++--------
> data/src/pmxcfs.c | 6 +++---
> data/src/status.c | 6 +++---
> 8 files changed, 60 insertions(+), 55 deletions(-)
>
> diff --git a/data/src/cfs-plug.c b/data/src/cfs-plug.c
> index 9ccbb3d..3eee161 100644
> --- a/data/src/cfs-plug.c
> +++ b/data/src/cfs-plug.c
> @@ -268,7 +268,7 @@ static int cfs_plug_base_read(cfs_plug_t *plug, const char *path, char *buf,
> g_return_val_if_fail(buf != NULL, PARAM_CHECK_ERRNO);
> g_return_val_if_fail(fi != NULL, PARAM_CHECK_ERRNO);
>
> - cfs_debug("enter cfs_plug_base_read %s %lu %ld", path, size, offset);
> + cfs_debug("enter cfs_plug_base_read %s %zu %jd", path, size, offset);
>
> int ret = -EACCES;
>
> @@ -291,7 +291,7 @@ static int cfs_plug_base_write(cfs_plug_t *plug, const char *path, const char *b
> g_return_val_if_fail(buf != NULL, PARAM_CHECK_ERRNO);
> g_return_val_if_fail(fi != NULL, PARAM_CHECK_ERRNO);
>
> - cfs_debug("enter cfs_plug_base_write %s %lu %ld", path, size, offset);
> + cfs_debug("enter cfs_plug_base_write %s %zu %jd", path, size, offset);
>
> int ret = -EACCES;
>
> @@ -309,7 +309,7 @@ static int cfs_plug_base_truncate(cfs_plug_t *plug, const char *path, off_t size
> g_return_val_if_fail(plug->ops == &cfs_ops, PARAM_CHECK_ERRNO);
> g_return_val_if_fail(path != NULL, PARAM_CHECK_ERRNO);
>
> - cfs_debug("enter cfs_plug_base_truncate %s %ld", path, size);
> + cfs_debug("enter cfs_plug_base_truncate %s %jd", path, size);
>
> int ret = -EACCES;
>
> diff --git a/data/src/database.c b/data/src/database.c
> index 391f3e6..e6f4ee8 100644
> --- a/data/src/database.c
> +++ b/data/src/database.c
> @@ -25,6 +25,7 @@
> #endif /* HAVE_CONFIG_H */
>
> #include <stdlib.h>
> +#include <inttypes.h>
> #include <stdio.h>
> #include <string.h>
> #include <unistd.h>
> @@ -153,7 +154,7 @@ static int backend_write_inode(
> {
> int rc;
>
> - cfs_debug("enter backend_write_inode %016zX", inode);
> + cfs_debug("enter backend_write_inode %016" PRIX64, inode);
>
> if ((rc = sqlite3_bind_int64(stmt, 1, inode)) != SQLITE_OK) {
> cfs_critical("sqlite3_bind failed: %s\n", sqlite3_errmsg(db));
> @@ -271,7 +272,7 @@ int bdb_backend_write(
> goto rollback;
>
> if (sqlite3_changes(bdb->db) != 1) {
> - cfs_critical("no such inode %016zX", inode);
> + cfs_critical("no such inode %016" PRIX64, inode);
> goto rollback;
> }
> }
> @@ -332,7 +333,7 @@ static gboolean bdb_backend_load_index(
> const char *name = (const char *)sqlite3_column_text(stmt, 6);
> int namelen = sqlite3_column_bytes(stmt, 6);
> if (name == NULL || namelen == 0) {
> - cfs_critical("inode has no name (inode = %016zX)", inode);
> + cfs_critical("inode has no name (inode = %016" PRIX64 ")", inode);
> goto fail;
> }
> te = g_malloc0(sizeof(memdb_tree_entry_t) + namelen + 1);
> @@ -355,20 +356,20 @@ static gboolean bdb_backend_load_index(
> te->data.value = g_memdup(value, size);
> } else if (te->type == DT_DIR) {
> if (size) {
> - cfs_critical("directory inode contains data (inode = %016zX)",
> + cfs_critical("directory inode contains data (inode = %016" PRIX64 ")",
> te->inode);
> g_free(te);
> goto fail;
> }
> te->data.entries = NULL;
> } else {
> - cfs_critical("inode has unknown type (inode = %016zX, type = %d)",
> + cfs_critical("inode has unknown type (inode = %016" PRIX64 ", type = %d)",
> te->inode, te->type);
> g_free(te);
> goto fail;
> }
>
> - cfs_debug("name %s (inode = %016zX, parent = %016zX)",
> + cfs_debug("name %s (inode = %016" PRIX64 ", parent = %016" PRIX64 ")",
> te->name, te->inode, te->parent);
>
> if (te->inode == 0) {
> @@ -396,7 +397,7 @@ static gboolean bdb_backend_load_index(
>
> } else if (!(pte->type == DT_DIR || pte->type == 0)) {
> cfs_critical("parent is not a directory "
> - "(inode = %016zX, parent = %016zX, name = '%s')",
> + "(inode = %016" PRIX64 ", parent = %016" PRIX64 ", name = '%s')",
> te->inode, te->parent, te->name);
> memdb_tree_entry_free(te);
> goto fail;
> @@ -408,7 +409,7 @@ static gboolean bdb_backend_load_index(
> if ((tmpte = g_hash_table_lookup(index, &te->inode))) {
> if (tmpte->type != 0) {
> cfs_critical("found strange placeholder for "
> - "(inode = %016zX, parent = %016zX, name = '%s', type = '%d')",
> + "(inode = %016" PRIX64 ", parent = %016" PRIX64 ", name = '%s', type = '%d')",
> te->inode, te->parent, te->name, tmpte->type);
> memdb_tree_entry_free(te);
> goto fail;
> @@ -423,7 +424,7 @@ static gboolean bdb_backend_load_index(
>
> if (g_hash_table_lookup(pte->data.entries, te->name)) {
> cfs_critical("found entry with duplicate name "
> - "(inode = %016zX, parent = %016zX, name = '%s')",
> + "(inode = %016" PRIX64 ", parent = %016" PRIX64 ", name = '%s')",
> te->inode, te->parent, te->name);
> goto fail;
> }
> @@ -444,7 +445,7 @@ static gboolean bdb_backend_load_index(
> while (g_hash_table_iter_next (&iter, &key, &value)) {
> memdb_tree_entry_t *te = (memdb_tree_entry_t *)value;
> if (te->type == 0) {
> - cfs_critical("missing directory inode (inode = %016zX)", te->inode);
> + cfs_critical("missing directory inode (inode = %016" PRIX64 ")", te->inode);
> goto fail;
> }
> }
> @@ -505,7 +506,7 @@ gboolean bdb_backend_commit_update(
> if (bdb_backend_delete_inode(bdb, slave_inode) != SQLITE_OK)
> goto abort;
>
> - cfs_debug("deleted inode %016zX", slave_inode);
> + cfs_debug("deleted inode %016" PRIX64, slave_inode);
> }
> j++;
> }
> @@ -519,7 +520,7 @@ gboolean bdb_backend_commit_update(
> if (bdb_backend_delete_inode(bdb, slave_inode) != SQLITE_OK)
> goto abort;
>
> - cfs_debug("deleted inode %016zX", slave_inode);
> + cfs_debug("deleted inode %016" PRIX64, slave_inode);
>
> j++;
> }
> diff --git a/data/src/dcdb.c b/data/src/dcdb.c
> index b83caa4..df7b37a 100644
> --- a/data/src/dcdb.c
> +++ b/data/src/dcdb.c
> @@ -26,6 +26,7 @@
>
> #include <stdlib.h>
> #include <stdio.h>
> +#include <inttypes.h>
> #include <string.h>
> #include <unistd.h>
> #include <glib.h>
> @@ -88,7 +89,7 @@ dcdb_parse_unlock_request(
> g_return_val_if_fail(csum != NULL, FALSE);
>
> if (msg_len < 33) {
> - cfs_critical("received short unlock message (%lu < 33)", msg_len);
> + cfs_critical("received short unlock message (%zu < 33)", msg_len);
> return FALSE;
> }
>
> @@ -178,7 +179,7 @@ dcdb_parse_fuse_message(
> g_return_val_if_fail(flags != NULL, FALSE);
>
> if (msg_len < 20) {
> - cfs_critical("received short fuse message (%lu < 20)", msg_len);
> + cfs_critical("received short fuse message (%zu < 20)", msg_len);
> return FALSE;
> }
>
> @@ -399,7 +400,7 @@ dcdb_sync_corosync_conf(
> if (!atomic_write_file(HOST_CLUSTER_CONF_FN, data, len, 0644, 0))
> goto ret;
>
> - cfs_message("wrote new corosync config '%s' (version = %zd)",
> + cfs_message("wrote new corosync config '%s' (version = %" G_GUINT64_FORMAT ")",
> HOST_CLUSTER_CONF_FN, new_version);
>
> if (notify_corosync && old_version) {
> @@ -434,7 +435,7 @@ dcdb_get_state(
>
> g_return_val_if_fail(memdb->root != NULL, FALSE);
>
> - cfs_debug("enter %s %016zX %08X", __func__, memdb->root->version, memdb->root->mtime);
> + cfs_debug("enter %s %016" PRIX64 " %08X", __func__, (uint64_t) memdb->root->version, memdb->root->mtime);
>
> g_mutex_lock (&memdb->mutex);
> memdb_index_t *idx = memdb_encode_index(memdb->index, memdb->root);
> @@ -524,12 +525,12 @@ dcdb_create_and_send_updates(
> if (g_hash_table_lookup(updates, &inode))
> continue;
>
> - cfs_debug("found different inode %d %016zX", i, inode);
> + cfs_debug("found different inode %d %016" PRIX64, i, (uint64_t) inode);
>
> memdb_tree_entry_t *te, *cpy;
>
> if (!(te = g_hash_table_lookup(memdb->index, &inode))) {
> - cfs_critical("can get inode data for inode %016zX", inode);
> + cfs_critical("can get inode data for inode %016" PRIX64, (uint64_t) inode);
> goto ret;
> }
>
> @@ -555,9 +556,9 @@ dcdb_create_and_send_updates(
>
> if (!dcdb_send_update_inode(dfsm, te)) {
> /* tolerate error here */
> - cfs_critical("sending update inode failed %016zX", te->inode);
> + cfs_critical("sending update inode failed %016" PRIX64, (uint64_t) te->inode);
> } else {
> - cfs_debug("sent update inode %016zX", te->inode);
> + cfs_debug("sent update inode %016" PRIX64, (uint64_t) te->inode);
> }
>
> memdb_tree_entry_free(te);
> @@ -683,8 +684,8 @@ dcdb_process_update(
> if (!(te = dcdb_parse_update_inode(msg, msg_len)))
> return -1;
>
> - cfs_debug("received inode update %016zX from node %d",
> - te->inode, nodeid);
> + cfs_debug("received inode update %016" PRIX64 " from node %d",
> + (uint64_t) te->inode, nodeid);
>
> dcdb_sync_info_t *localsi = (dcdb_sync_info_t *)syncinfo->data;
>
> @@ -764,13 +765,13 @@ dcdb_checksum(
>
> g_return_val_if_fail(memdb != NULL, FALSE);
>
> - cfs_debug("enter %s %016zX %08X", __func__, memdb->root->version, memdb->root->mtime);
> + cfs_debug("enter %s %016" PRIX64 " %08X", __func__, memdb->root->version, memdb->root->mtime);
>
> g_mutex_lock (&memdb->mutex);
> gboolean res = memdb_compute_checksum(memdb->index, memdb->root, csum, csum_len);
> g_mutex_unlock (&memdb->mutex);
>
> - cfs_debug("leave %s %016zX (%d)", __func__, *(uint64_t *)csum, res);
> + cfs_debug("leave %s %016" PRIX64 " (%d)", __func__, *( (uint64_t *) csum), res);
>
> return res;
> }
> @@ -802,7 +803,7 @@ dcdb_deliver(
> if (!DCDB_VALID_MESSAGE_TYPE(msg_type))
> goto unknown;
>
> - cfs_debug("process message %d (length = %ld)", msg_type, msg_len);
> + cfs_debug("process message %u (length = %zd)", msg_type, msg_len);
>
> if (!cfs_is_quorate()) {
> cfs_critical("received write while not quorate - trigger resync");
> @@ -920,7 +921,7 @@ ret:
> return res;
>
> unknown:
> - cfs_critical("received unknown message type (msg_type == %d)", msg_type);
> + cfs_critical("received unknown message type (msg_type == %u)", msg_type);
> leave:
> res = -1;
> goto ret;
> diff --git a/data/src/dfsm.c b/data/src/dfsm.c
> index fd35174..cdf473e 100644
> --- a/data/src/dfsm.c
> +++ b/data/src/dfsm.c
> @@ -30,6 +30,7 @@
> #endif /* HAVE_CONFIG_H */
>
> #include <sys/types.h>
> +#include <inttypes.h>
> #include <unistd.h>
> #include <string.h>
> #include <stdlib.h>
> @@ -667,7 +668,7 @@ dfsm_cpg_deliver_callback(
> }
>
> if (msg_len < sizeof(dfsm_message_header_t)) {
> - cfs_dom_critical(dfsm->log_domain, "received short message (%ld bytes)", msg_len);
> + cfs_dom_critical(dfsm->log_domain, "received short message (%zd bytes)", msg_len);
> goto leave;
> }
>
> @@ -689,13 +690,13 @@ dfsm_cpg_deliver_callback(
> dfsm_message_normal_header_t *header = (dfsm_message_normal_header_t *)msg;
>
> if (msg_len < sizeof(dfsm_message_normal_header_t)) {
> - cfs_dom_critical(dfsm->log_domain, "received short message (type = %d, subtype = %d, %ld bytes)",
> + cfs_dom_critical(dfsm->log_domain, "received short message (type = %d, subtype = %d, %zd bytes)",
> base_header->type, base_header->subtype, msg_len);
> goto leave;
> }
>
> if (mode != DFSM_MODE_SYNCED) {
> - cfs_dom_debug(dfsm->log_domain, "queue message %zu (subtype = %d, length = %ld)",
> + cfs_dom_debug(dfsm->log_domain, "queue message %" PRIu64 " (subtype = %d, length = %zd)",
> header->count, base_header->subtype, msg_len);
>
> if (!dfsm_queue_add_message(dfsm, nodeid, pid, header->count, msg, msg_len))
> @@ -725,7 +726,7 @@ dfsm_cpg_deliver_callback(
> dfsm_message_state_header_t *header = (dfsm_message_state_header_t *)msg;
>
> if (msg_len < sizeof(dfsm_message_state_header_t)) {
> - cfs_dom_critical(dfsm->log_domain, "received short state message (type = %d, subtype = %d, %ld bytes)",
> + cfs_dom_critical(dfsm->log_domain, "received short state message (type = %d, subtype = %d, %zd bytes)",
> base_header->type, base_header->subtype, msg_len);
> goto leave;
> }
> @@ -755,14 +756,14 @@ dfsm_cpg_deliver_callback(
> } else if (base_header->type == DFSM_MESSAGE_VERIFY_REQUEST) {
>
> if (msg_len != sizeof(dfsm->csum_counter)) {
> - cfs_dom_critical(dfsm->log_domain, "cpg received verify request with wrong length (%ld bytes) form node %d/%d", msg_len, nodeid, pid);
> + cfs_dom_critical(dfsm->log_domain, "cpg received verify request with wrong length (%zd bytes) form node %d/%d", msg_len, nodeid, pid);
> goto leave;
> }
>
> uint64_t csum_id = *((uint64_t *)msg);
> msg += 8; msg_len -= 8;
>
> - cfs_dom_debug(dfsm->log_domain, "got verify request from node %d %016zX", nodeid, csum_id);
> + cfs_dom_debug(dfsm->log_domain, "got verify request from node %d %016" PRIX64, nodeid, csum_id);
>
> if (dfsm->dfsm_callbacks->dfsm_checksum_fn) {
> if (!dfsm->dfsm_callbacks->dfsm_checksum_fn(
> @@ -789,7 +790,7 @@ dfsm_cpg_deliver_callback(
> if (dfsm->dfsm_callbacks->dfsm_checksum_fn) {
>
> if (msg_len != (sizeof(dfsm->csum_id) + sizeof(dfsm->csum))) {
> - cfs_dom_critical(dfsm->log_domain, "cpg received verify message with wrong length (%ld bytes)", msg_len);
> + cfs_dom_critical(dfsm->log_domain, "cpg received verify message with wrong length (%zd bytes)", msg_len);
> goto leave;
> }
>
> @@ -799,7 +800,7 @@ dfsm_cpg_deliver_callback(
> if (dfsm->csum_id == csum_id &&
> (memcmp(&dfsm->csum_epoch, &header->epoch, sizeof(dfsm_sync_epoch_t)) == 0)) {
> if (memcmp(msg, dfsm->csum, sizeof(dfsm->csum)) != 0) {
> - cfs_dom_critical(dfsm->log_domain, "wrong checksum %016zX != %016zX - restarting",
> + cfs_dom_critical(dfsm->log_domain, "wrong checksum %016" PRIX64 " != %016" PRIX64 " - restarting",
> *(uint64_t *)msg, *(uint64_t *)dfsm->csum);
> goto leave;
> } else {
> @@ -957,7 +958,7 @@ dfsm_cpg_deliver_callback(
> cfs_dom_debug(dfsm->log_domain, "ignore verify message %d while not synced", base_header->type);
>
> } else {
> - cfs_dom_critical(dfsm->log_domain, "received unknown state message type (type = %d, %ld bytes)",
> + cfs_dom_critical(dfsm->log_domain, "received unknown state message type (type = %d, %zd bytes)",
> base_header->type, msg_len);
> goto leave;
> }
> @@ -1309,7 +1310,7 @@ dfsm_verify_request(dfsm_t *dfsm)
> struct iovec iov[len];
>
> if (dfsm->csum_counter != dfsm->csum_id) {
> - g_message("delay verify request %016zX", dfsm->csum_counter + 1);
> + g_message("delay verify request %016" PRIX64, dfsm->csum_counter + 1);
> return CS_OK;
> };
>
> @@ -1317,7 +1318,7 @@ dfsm_verify_request(dfsm_t *dfsm)
> iov[0].iov_base = (char *)&dfsm->csum_counter;
> iov[0].iov_len = sizeof(dfsm->csum_counter);
>
> - cfs_debug("send verify request %016zX", dfsm->csum_counter);
> + cfs_debug("send verify request %016" PRIX64, dfsm->csum_counter);
>
> cs_error_t result;
> result = dfsm_send_state_message_full(dfsm, DFSM_MESSAGE_VERIFY_REQUEST, iov, len);
> diff --git a/data/src/logger.c b/data/src/logger.c
> index c0c61fb..c3d682b 100644
> --- a/data/src/logger.c
> +++ b/data/src/logger.c
> @@ -27,6 +27,7 @@
>
> #include <unistd.h>
> #include <stdio.h>
> +#include <inttypes.h>
> #include <stdint.h>
> #include <glib.h>
> #include <string.h>
> @@ -121,7 +122,7 @@ clog_dump_entry(clog_entry_t *cur, uint32_t cpos)
> char tbuf[256];
> strftime(tbuf, sizeof(tbuf), "%F %T", localtime(<));
> printf("cpos %05d %08x %s", cpos, cur->uid, tbuf);
> - printf(" %s{%016zX} %s[%s{%016zX}]: %s\n", node, cur->node_digest, tag, ident, cur->ident_digest, msg);
> + printf(" %s{%016" PRIX64 "} %s[%s{%016" PRIX64 "}]: %s\n", node, cur->node_digest, tag, ident, cur->ident_digest, msg);
>
> }
>
> diff --git a/data/src/memdb.c b/data/src/memdb.c
> index 560cd4b..1c95e95 100644
> --- a/data/src/memdb.c
> +++ b/data/src/memdb.c
> @@ -23,6 +23,7 @@
> #endif /* HAVE_CONFIG_H */
>
> #include <stdio.h>
> +#include <inttypes.h>
> #include <stdlib.h>
> #include <sys/types.h>
> #include <sys/stat.h>
> @@ -530,7 +531,7 @@ memdb_open(const char *dbfilename)
>
> memdb_update_locks(memdb);
>
> - cfs_debug("memdb open '%s' successful (version = %016zX)",
> + cfs_debug("memdb open '%s' successful (version = %016" PRIX64 ")",
> dbfilename, memdb->root->version);
>
> return memdb;
> @@ -630,7 +631,7 @@ int memdb_mkdir(
> g_hash_table_replace(parent->data.entries, te->name, te);
> g_hash_table_replace(memdb->index, &te->inode, te);
>
> - cfs_debug("memdb_mkdir %s %s %016zX", dirname, base, memdb->root->version);
> + cfs_debug("memdb_mkdir %s %s %016" PRIX64, dirname, base, memdb->root->version);
>
> if (bdb_backend_write(memdb->bdb, te->inode, te->parent, te->version,
> te->writer, te->mtime, 0, DT_DIR, te->name, NULL, 0)) {
> @@ -832,7 +833,7 @@ memdb_pwrite(
>
> record_memdb_change(path);
>
> - cfs_debug("memdb_pwrite %s %s %016zX %016zX", dirname, te->name, te->inode, te->version);
> + cfs_debug("memdb_pwrite %s %s %016" PRIX64 " %016" PRIX64, dirname, te->name, te->inode, te->version);
>
> if (bdb_backend_write(memdb->bdb, te->inode, te->parent, te->version,
> te->writer, te->mtime, te->size, te->type, te->name,
> @@ -935,7 +936,7 @@ memdb_mtime(
>
> record_memdb_change(path);
>
> - cfs_debug("memdb_mtime %s %s %016zX %016zX", dirname, te->name, te->inode, te->version);
> + cfs_debug("memdb_mtime %s %s %016" PRIX64 " %016" PRIX64, dirname, te->name, te->inode, te->version);
>
> if (bdb_backend_write(memdb->bdb, te->inode, te->parent, te->version,
> te->writer, te->mtime, te->size, te->type, te->name,
> @@ -1365,7 +1366,7 @@ tree_entry_debug(memdb_tree_entry_t *te)
>
> // same as tree_entry_print(), but use cfs_debug() instead of g_print()
>
> - cfs_debug("%016zX %c %016zX %016zX %08X %08X %08X %s\n",
> + cfs_debug("%016" PRIX64 " %c %016" PRIX64 " %016" PRIX64 " %08X %08X %08X %s\n",
> te->inode, te->type == DT_DIR ? 'D' : 'R', te->parent, te->version,
> te->writer, te->mtime, te->size, te->name);
> }
> @@ -1375,7 +1376,7 @@ tree_entry_print(memdb_tree_entry_t *te)
> {
> g_return_if_fail(te != NULL);
>
> - g_print("%016zX %c %016zX %016zX %08X %08X %08X %s\n",
> + g_print("%016" PRIX64 " %c %016" PRIX64 " %016" PRIX64 " %08X %08X %08X %s\n",
> te->inode, te->type == DT_DIR ? 'D' : 'R', te->parent, te->version,
> te->writer, te->mtime, te->size, te->name);
> }
> @@ -1413,11 +1414,11 @@ memdb_dump_index (memdb_index_t *idx)
> {
> g_return_if_fail(idx != NULL);
>
> - g_print ("INDEX DUMP %016zX\n", idx->version);
> + g_print ("INDEX DUMP %016" PRIX64 "\n", idx->version);
>
> int i;
> for (i = 0; i < idx->size; i++) {
> - g_print ("%016zX %016zX%016zX%016zX%016zX\n", idx->entries[i].inode,
> + g_print ("%016" PRIX64 " %016" PRIX64 "%016" PRIX64 "%016" PRIX64 "%016" PRIX64 "\n", idx->entries[i].inode,
> *((guint64 *)idx->entries[i].digest),
> *((guint64 *)(idx->entries[i].digest + 8)),
> *((guint64 *)(idx->entries[i].digest + 16)),
> diff --git a/data/src/pmxcfs.c b/data/src/pmxcfs.c
> index 26cbc30..e1b6fed 100644
> --- a/data/src/pmxcfs.c
> +++ b/data/src/pmxcfs.c
> @@ -295,7 +295,7 @@ static int cfs_fuse_read(const char *path, char *buf, size_t size, off_t offset,
> {
> (void) fi;
>
> - cfs_debug("enter cfs_fuse_read %s %lu %ld", path, size, offset);
> + cfs_debug("enter cfs_fuse_read %s %zu %jd", path, size, offset);
>
> int ret = -EACCES;
>
> @@ -320,7 +320,7 @@ static int cfs_fuse_write(const char *path, const char *buf, size_t size,
> {
> (void) fi;
>
> - cfs_debug("enter cfs_fuse_write %s %lu %ld", path, size, offset);
> + cfs_debug("enter cfs_fuse_write %s %zu %jd", path, size, offset);
>
> int ret = -EACCES;
>
> @@ -343,7 +343,7 @@ static int cfs_fuse_write(const char *path, const char *buf, size_t size,
>
> static int cfs_fuse_truncate(const char *path, off_t size)
> {
> - cfs_debug("enter cfs_fuse_truncate %s %ld", path, size);
> + cfs_debug("enter cfs_fuse_truncate %s %jd", path, size);
>
> int ret = -EACCES;
>
> diff --git a/data/src/status.c b/data/src/status.c
> index bcd9145..3896fcb 100644
> --- a/data/src/status.c
> +++ b/data/src/status.c
> @@ -1204,7 +1204,7 @@ kvstore_parse_log_message(
> g_return_val_if_fail(msg != NULL, NULL);
>
> if (msg_len < sizeof(clog_entry_t)) {
> - cfs_critical("received short log message (%lu < %lu)", msg_len, sizeof(clog_entry_t));
> + cfs_critical("received short log message (%zu < %zu)", msg_len, sizeof(clog_entry_t));
> return NULL;
> }
>
> @@ -1214,7 +1214,7 @@ kvstore_parse_log_message(
> entry->ident_len + entry->tag_len + entry->msg_len;
>
> if (msg_len != size) {
> - cfs_critical("received log message with wrong size (%lu != %u)", msg_len, size);
> + cfs_critical("received log message with wrong size (%zu != %u)", msg_len, size);
> return NULL;
> }
>
> @@ -1260,7 +1260,7 @@ kvstore_parse_update_message(
> g_return_val_if_fail(len != NULL, FALSE);
>
> if (msg_len < 256) {
> - cfs_critical("received short kvstore message (%lu < 256)", msg_len);
> + cfs_critical("received short kvstore message (%zu < 256)", msg_len);
> return FALSE;
> }
>
More information about the pve-devel
mailing list