server: switch to 16-bit crc
Good enough and we can kill the command padding
Signed-off-by: Jens Axboe <axboe@kernel.dk>
diff --git a/server.c b/server.c
index 79c1041..44f5fa0 100644
--- a/server.c
+++ b/server.c
@@ -15,7 +15,7 @@
#include "fio.h"
#include "server.h"
-#include "crc/crc32.h"
+#include "crc/crc16.h"
int fio_net_port = 8765;
@@ -75,15 +75,15 @@
static int verify_convert_cmd(struct fio_net_cmd *cmd)
{
- uint32_t crc;
+ uint16_t crc;
- cmd->cmd_crc32 = le32_to_cpu(cmd->cmd_crc32);
- cmd->pdu_crc32 = le32_to_cpu(cmd->pdu_crc32);
+ cmd->cmd_crc16 = le16_to_cpu(cmd->cmd_crc16);
+ cmd->pdu_crc16 = le16_to_cpu(cmd->pdu_crc16);
- crc = crc32(cmd, sizeof(*cmd) - 2 * sizeof(uint32_t));
- if (crc != cmd->cmd_crc32) {
+ crc = crc16(cmd, FIO_NET_CMD_CRC_SZ);
+ if (crc != cmd->cmd_crc16) {
log_err("fio: server bad crc on command (got %x, wanted %x)\n",
- cmd->cmd_crc32, crc);
+ cmd->cmd_crc16, crc);
return 1;
}
@@ -112,7 +112,7 @@
struct fio_net_cmd *fio_net_cmd_read(int sk)
{
struct fio_net_cmd cmd, *ret = NULL;
- uint32_t crc;
+ uint16_t crc;
if (fio_recv_data(sk, &cmd, sizeof(cmd)))
return NULL;
@@ -135,10 +135,10 @@
}
/* Verify payload crc */
- crc = crc32(ret->payload, ret->pdu_len);
- if (crc != ret->pdu_crc32) {
+ crc = crc16(ret->payload, ret->pdu_len);
+ if (crc != ret->pdu_crc16) {
log_err("fio: server bad crc on payload (got %x, wanted %x)\n",
- ret->pdu_crc32, crc);
+ ret->pdu_crc16, crc);
free(ret);
return NULL;
}
@@ -150,12 +150,11 @@
{
uint32_t pdu_len;
- cmd->cmd_crc32 = cpu_to_le32(crc32(cmd,
- sizeof(*cmd) - 2 * sizeof(uint32_t)));
+ cmd->cmd_crc16 = cpu_to_le16(crc16(cmd, FIO_NET_CMD_CRC_SZ));
pdu_len = le32_to_cpu(cmd->pdu_len);
if (pdu_len)
- cmd->pdu_crc32 = cpu_to_le32(crc32(cmd->payload, pdu_len));
+ cmd->pdu_crc16 = cpu_to_le16(crc16(cmd->payload, pdu_len));
}
static int send_simple_command(int sk, uint16_t opcode, uint64_t serial)
@@ -176,7 +175,11 @@
*/
static int ack_command(int sk, struct fio_net_cmd *cmd)
{
+#if 0
return send_simple_command(sk, FIO_NET_CMD_ACK, cmd->serial);
+#else
+ return 0;
+#endif
}
#if 0
@@ -285,7 +288,7 @@
if (ret < 0) {
if (errno == EINTR)
break;
- perror("poll");
+ log_err("fio: poll: %s\n", strerror(errno));
goto out;
} else if (!ret)
continue;
diff --git a/server.h b/server.h
index d66782d..73e00a5 100644
--- a/server.h
+++ b/server.h
@@ -13,10 +13,9 @@
uint16_t opcode; /* command opcode */
uint32_t flags; /* modifier flags */
uint64_t serial; /* serial number */
- uint32_t pad;
uint32_t pdu_len; /* length of post-cmd layload */
- uint32_t cmd_crc32; /* cmd checksum */
- uint32_t pdu_crc32; /* payload checksum */
+ uint16_t cmd_crc16; /* cmd checksum */
+ uint16_t pdu_crc16; /* payload checksum */
uint8_t payload[0]; /* payload */
};
@@ -32,6 +31,10 @@
FIO_NET_CMD_ACK = 4,
FIO_NET_CMD_NAK = 5,
FIO_NET_CMD_TEXT = 6,
+
+ /* crc does not include the crc fields */
+ FIO_NET_CMD_CRC_SZ = sizeof(struct fio_net_cmd) -
+ 2 * sizeof(uint16_t),
};
extern int fio_server(void);