if (c != hdr->crc16) {
log_err("crc16: verify failed at %llu/%lu\n", io_u->offset, io_u->buflen);
- log_err("crc16: wanted %lx, got %x\n", hdr->crc32, c);
+ log_err("crc16: wanted %x, got %x\n", hdr->crc16, c);
return 1;
}
static int verify_io_u_md5(struct verify_header *hdr, struct io_u *io_u)
{
- unsigned char *p = io_u->buf;
- struct md5_ctx md5_ctx;
+ unsigned char *p = io_u->buf + sizeof(*hdr);
+ uint32_t hash[MD5_HASH_WORDS];
+ struct md5_ctx md5_ctx = {
+ .hash = hash,
+ };
- memset(&md5_ctx, 0, sizeof(md5_ctx));
- p += sizeof(*hdr);
md5_update(&md5_ctx, p, hdr->len - sizeof(*hdr));
- if (memcmp(hdr->md5_digest, md5_ctx.hash, sizeof(md5_ctx.hash))) {
+ if (memcmp(hdr->md5_digest, md5_ctx.hash, sizeof(hash))) {
log_err("md5: verify failed at %llu/%lu\n", io_u->offset, io_u->buflen);
hexdump(hdr->md5_digest, sizeof(hdr->md5_digest));
- hexdump(md5_ctx.hash, sizeof(md5_ctx.hash));
+ hexdump(md5_ctx.hash, sizeof(hash));
return 1;
}
return EIO;
}
- if (hdr->verify_type == VERIFY_MD5)
+ switch (hdr->verify_type) {
+ case VERIFY_MD5:
ret = verify_io_u_md5(hdr, io_u);
- else if (hdr->verify_type == VERIFY_CRC32)
+ break;
+ case VERIFY_CRC32:
ret = verify_io_u_crc32(hdr, io_u);
- else if (hdr->verify_type == VERIFY_CRC16)
+ break;
+ case VERIFY_CRC16:
ret = verify_io_u_crc16(hdr, io_u);
- else if (hdr->verify_type == VERIFY_CRC7)
+ break;
+ case VERIFY_CRC7:
ret = verify_io_u_crc7(hdr, io_u);
- else {
+ break;
+ default:
log_err("Bad verify type %u\n", hdr->verify_type);
ret = 1;
}
static void fill_md5(struct verify_header *hdr, void *p, unsigned int len)
{
- struct md5_ctx md5_ctx;
+ struct md5_ctx md5_ctx = {
+ .hash = (uint32_t *) hdr->md5_digest,
+ };
- memset(&md5_ctx, 0, sizeof(md5_ctx));
md5_update(&md5_ctx, p, len);
- memcpy(hdr->md5_digest, md5_ctx.hash, sizeof(md5_ctx.hash));
}
/*
*/
void populate_verify_io_u(struct thread_data *td, struct io_u *io_u)
{
- unsigned char *p = (unsigned char *) io_u->buf;
- struct verify_header hdr;
+ const unsigned int len = io_u->buflen - sizeof(struct verify_header);
+ struct verify_header *hdr;
+ unsigned char *p;
if (td->o.verify == VERIFY_NULL)
return;
- hdr.fio_magic = FIO_HDR_MAGIC;
- hdr.len = io_u->buflen;
- p += sizeof(hdr);
- fill_random_bytes(td, p, io_u->buflen - sizeof(hdr));
-
- if (td->o.verify == VERIFY_MD5) {
- fill_md5(&hdr, p, io_u->buflen - sizeof(hdr));
- hdr.verify_type = VERIFY_MD5;
- } else if (td->o.verify == VERIFY_CRC32) {
- fill_crc32(&hdr, p, io_u->buflen - sizeof(hdr));
- hdr.verify_type = VERIFY_CRC32;
- } else if (td->o.verify == VERIFY_CRC16) {
- fill_crc16(&hdr, p, io_u->buflen - sizeof(hdr));
- hdr.verify_type = VERIFY_CRC16;
- } else if (td->o.verify == VERIFY_CRC7) {
- fill_crc7(&hdr, p, io_u->buflen - sizeof(hdr));
- hdr.verify_type = VERIFY_CRC7;
+ hdr = (struct verify_header *) io_u->buf;
+ hdr->fio_magic = FIO_HDR_MAGIC;
+ hdr->len = io_u->buflen;
+ hdr->verify_type = td->o.verify;
+
+ p = io_u->buf + sizeof(*hdr);
+ fill_random_bytes(td, p, len);
+
+ switch (td->o.verify) {
+ case VERIFY_MD5:
+ fill_md5(hdr, p, len);
+ break;
+ case VERIFY_CRC32:
+ fill_crc32(hdr, p, len);
+ break;
+ case VERIFY_CRC16:
+ fill_crc16(hdr, p, len);
+ break;
+ case VERIFY_CRC7:
+ fill_crc7(hdr, p, len);
+ break;
+ default:
+ log_err("fio: bad verify type: %d\n", td->o.verify);
+ assert(0);
}
-
- memcpy(io_u->buf, &hdr, sizeof(hdr));
}
int get_next_verify(struct thread_data *td, struct io_u *io_u)