From nobody Fri Nov 14 18:17:41 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=foxmail.com ARC-Seal: i=1; a=rsa-sha256; t=1589639111; cv=none; d=zohomail.com; s=zohoarc; b=RfFyvLKA2aKtXxI3HVJ3tPffjJ+wvoD+kG2Xd97juX0Wn+8nytQhqfR3LSKFAgy4AA1KUCuBtkvD66uLZM9x0mY9LY9kcN60aLOEGBqIc70ZsfqAUVl9wrHgvvkD3qT3V1k9y14Ubu0kJhqlWUa87qDGa+l1I1fFfh396FRP6b4= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1589639111; h=Content-Transfer-Encoding:Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To; bh=dqNWq+Cq319NegWg6B2F9qiSIJaev6Gp6VZfGEzFlqE=; b=Tg5PW4vrie6tN6aQRdlZtyQ17LQ/VvAFM9UYrH1BqngijxzR/rzasQgKlvni1+HsyzPXiSAHyGESPP3tzRPXxjfcwD/o/3buggzZFCB9biPbw+cKLmJbSbCiXX8cLR5YYlZYSQoV6rbiUSf4lB6+ebd7LROd4324IKBiAMYfTRw= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1589639111433996.573302993203; Sat, 16 May 2020 07:25:11 -0700 (PDT) Received: from localhost ([::1]:48324 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jZxkT-00078T-W0 for importer@patchew.org; Sat, 16 May 2020 10:25:10 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:57460) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jZxSS-0007lF-Mh for qemu-devel@nongnu.org; Sat, 16 May 2020 10:06:32 -0400 Received: from out203-205-221-149.mail.qq.com ([203.205.221.149]:49254 helo=qq.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jZxSM-00072F-Oh for qemu-devel@nongnu.org; Sat, 16 May 2020 10:06:32 -0400 Received: from localhost.localdomain ([120.200.144.22]) by newxmesmtplogicsvrszc5.qq.com (NewEsmtp) with SMTP id 1879961C; Sat, 16 May 2020 22:06:07 +0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qq.com; s=s201512; t=1589637971; bh=dqNWq+Cq319NegWg6B2F9qiSIJaev6Gp6VZfGEzFlqE=; h=From:To:Cc:Subject:Date; b=knLHA2TiFl4xb+GGQP7Jc6G0LOOb+ubps3lzznILADGTk6sQ6cP73eDNZkTepv1qo 1WE82dC8NvOTrIoL8nmljOGIiOj4S/LJDYquzejPzeHMdA7/X8/1tJqpRrQeMPivvO BwdGQOqzCCnjaqWtynGYOd1p6iWEIZwnPhlbQ1VE= X-QQ-mid: xmsmtpt1589637967tnnuy7lma X-QQ-XMAILINFO: Nrw6fLiikHXY6esOCuMUdSmsPcObMYAh4r/8NbhietWtzeYuuaQBseW0SjCQRu nBQHNZ7qmkaST5GyRrW6hJtjO6Pw91E9Zf7+TEhv6KRMt3MR8iJ6we7/Tl9Wo7EJqMNGnN51uu4C AZMYxiOfq1TV4sTSuaiiPSdkdvWThiB9XXcZTxspReKAqm3rFIxMyQUZCYBDUBhYp8XCmkv/NMlr 9csql8+FYu91KzKnBJEEPCoXwasvglHi1LYCvnWP64M0CDm+H9RaGs71w1WAi2tgnbsGo6qxWr40 iU8OIq9uY1gpbnlduu03sIFSjw0xcLy95B1itk6N13FM0GG9Hl6pUY6sbYjVKZiLPSQ5w8HyQnro 1/JrUsz82loX6yBIm6tsq0zyiT1o2ds8HCuxLKq8TRyDapQUiR8ua1tmYKQyqUmUpVwr/Asi2wdc +b+ilFqGzLPRrPMQj5pkSuhH9V+mDA6KQgzykkdWQn9+N9K8FhUpLfts1uHjZcju4zHqhkY+0PG2 2hHK7lxM3wUqSLdcCW/igBjEWnmjjDG8omO5Am8fQtuB2CGtoP2pjCpdGrz3UBgI8jKu4MzOszNT vpqfBKb8MxdW79Vdxvo6YEM08AExDun0qwYnj8cjYBrHBYjc1Ngg/QACkwzLPVfrBbNLDvQ/hurJ EY0Q== From: teachk To: qemu-devel@nongnu.org Subject: [PATCH] virtio-crypto: Convert DPRINTF to trace event Date: Sat, 16 May 2020 22:05:40 +0800 Message-Id: <20200516140540.51-1-teachk@foxmail.com> X-Mailer: git-send-email 2.22.0.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=203.205.221.149; envelope-from=teachk@foxmail.com; helo=qq.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/05/16 10:06:13 X-ACL-Warn: Detected OS = Linux 3.1-3.10 X-Spam_score_int: -8 X-Spam_score: -0.9 X-Spam_bar: / X-Spam_report: (-0.9 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RDNS_DYNAMIC=0.982, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001 autolearn=_AUTOLEARN X-Spam_action: no action X-Mailman-Approved-At: Sat, 16 May 2020 10:23:14 -0400 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Hou Weiying , arei.gonglei@huawei.com, mst@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" From: Hou Weiying Signed-off-by: Hou Weiying --- hw/virtio/trace-events | 12 ++++++++++++ hw/virtio/virtio-crypto.c | 26 ++++++++++++++------------ include/hw/virtio/virtio-crypto.h | 11 ----------- 3 files changed, 26 insertions(+), 23 deletions(-) diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events index e83500bee9..f7c20f211a 100644 --- a/hw/virtio/trace-events +++ b/hw/virtio/trace-events @@ -73,3 +73,15 @@ virtio_iommu_get_domain(uint32_t domain_id) "Alloc domai= n=3D%d" virtio_iommu_put_domain(uint32_t domain_id) "Free domain=3D%d" virtio_iommu_translate_out(uint64_t virt_addr, uint64_t phys_addr, uint32_= t sid) "0x%"PRIx64" -> 0x%"PRIx64 " for sid=3D%d" virtio_iommu_report_fault(uint8_t reason, uint32_t flags, uint32_t endpoin= t, uint64_t addr) "FAULT reason=3D%d flags=3D%d endpoint=3D%d address =3D0x= %"PRIx64 + +# virtio-crypto.c +virtio_crypto_cipher_session_helper_cipher_alg_and_direction(uint32_t ciph= er_alg, uint8_t direction) "cipher_alg=3D%" PRIu32 ", info->direction=3D%" = PRIu32 +virtio_crypto_cipher_session_helper_keylen(uint32_t keylen) "keylen=3D%" P= RIu32 +virtio_crypto_create_sym_session_auth_keylen(uint32_t auth_keylen) "auth_k= eylen=3D%" PRIu32 +virtio_crypto_create_sym_session_session_id(int64_t session_id) "create se= ssion_id=3D%" PRIu64 " successfully" +virtio_crypto_sym_op_helper_src_len(uint32_t src_len) "src_len=3D%" PRIu32 +virtio_crypto_sym_op_helper_dst_len(uint32_t dst_len) "dst_len=3D%" PRIu32 +virtio_crypto_sym_op_helper_hash_result_len(uint32_t hash_result_len) "has= h_result_len=3D%" PRIu32 +virtio_crypto_handle_close_session(uint64_t session_id) "close session id = %" PRIu64 +virtio_crypto_sym_op_helper_iv_len(uint32_t iv_len) "iv_len %" PRIu32 +virtio_crypto_sym_op_helper_aad_len(uint32_t aad_len) "aad_len %" PRIu32 diff --git a/hw/virtio/virtio-crypto.c b/hw/virtio/virtio-crypto.c index bd9165c565..676948a4dd 100644 --- a/hw/virtio/virtio-crypto.c +++ b/hw/virtio/virtio-crypto.c @@ -24,6 +24,7 @@ #include "hw/virtio/virtio-access.h" #include "standard-headers/linux/virtio_ids.h" #include "sysemu/cryptodev-vhost.h" +#include "trace.h" =20 #define VIRTIO_CRYPTO_VM_VERSION 1 =20 @@ -49,8 +50,9 @@ virtio_crypto_cipher_session_helper(VirtIODevice *vdev, info->cipher_alg =3D ldl_le_p(&cipher_para->algo); info->key_len =3D ldl_le_p(&cipher_para->keylen); info->direction =3D ldl_le_p(&cipher_para->op); - DPRINTF("cipher_alg=3D%" PRIu32 ", info->direction=3D%" PRIu32 "\n", - info->cipher_alg, info->direction); + trace_virtio_crypto_cipher_session_helper_cipher_alg_and_direction( + info->cipher_alg, info->direction); + =20 if (info->key_len > vcrypto->conf.max_cipher_key_len) { error_report("virtio-crypto length of cipher key is too big: %u", @@ -60,7 +62,7 @@ virtio_crypto_cipher_session_helper(VirtIODevice *vdev, /* Get cipher key */ if (info->key_len > 0) { size_t s; - DPRINTF("keylen=3D%" PRIu32 "\n", info->key_len); + trace_virtio_crypto_cipher_session_helper_keylen(info->key_len); =20 info->cipher_key =3D g_malloc(info->key_len); s =3D iov_to_buf(*iov, num, 0, info->cipher_key, info->key_len); @@ -130,7 +132,8 @@ virtio_crypto_create_sym_session(VirtIOCrypto *vcrypto, } /* get auth key */ if (info.auth_key_len > 0) { - DPRINTF("auth_keylen=3D%" PRIu32 "\n", info.auth_key_len); + trace_virtio_crypto_create_sym_session_auth_keylen( + info.auth_key_len); info.auth_key =3D g_malloc(info.auth_key_len); s =3D iov_to_buf(iov, out_num, 0, info.auth_key, info.auth_key_len); @@ -165,8 +168,7 @@ virtio_crypto_create_sym_session(VirtIOCrypto *vcrypto, vcrypto->cryptodev, &info, queue_index, &local_err); if (session_id >=3D 0) { - DPRINTF("create session_id=3D%" PRIu64 " successfully\n", - session_id); + trace_virtio_crypto_create_sym_session_session_id(session_id); =20 ret =3D session_id; } else { @@ -193,7 +195,7 @@ virtio_crypto_handle_close_session(VirtIOCrypto *vcrypt= o, Error *local_err =3D NULL; =20 session_id =3D ldq_le_p(&close_sess_req->session_id); - DPRINTF("close session, id=3D%" PRIu64 "\n", session_id); + trace_virtio_crypto_handle_close_session(session_id); =20 ret =3D cryptodev_backend_sym_close_session( vcrypto->cryptodev, session_id, queue_id, &local_err); @@ -474,7 +476,7 @@ virtio_crypto_sym_op_helper(VirtIODevice *vdev, op_info->len_to_cipher =3D len_to_cipher; /* Handle the initilization vector */ if (op_info->iv_len > 0) { - DPRINTF("iv_len=3D%" PRIu32 "\n", op_info->iv_len); + trace_virtio_crypto_sym_op_helper_iv_len(op_info->iv_len); op_info->iv =3D op_info->data + curr_size; =20 s =3D iov_to_buf(iov, out_num, 0, op_info->iv, op_info->iv_len); @@ -488,7 +490,7 @@ virtio_crypto_sym_op_helper(VirtIODevice *vdev, =20 /* Handle additional authentication data if exists */ if (op_info->aad_len > 0) { - DPRINTF("aad_len=3D%" PRIu32 "\n", op_info->aad_len); + trace_virtio_crypto_sym_op_helper_aad_len(op_info->aad_len); op_info->aad_data =3D op_info->data + curr_size; =20 s =3D iov_to_buf(iov, out_num, 0, op_info->aad_data, op_info->aad_= len); @@ -503,7 +505,7 @@ virtio_crypto_sym_op_helper(VirtIODevice *vdev, =20 /* Handle the source data */ if (op_info->src_len > 0) { - DPRINTF("src_len=3D%" PRIu32 "\n", op_info->src_len); + trace_virtio_crypto_sym_op_helper_src_len(op_info->src_len); op_info->src =3D op_info->data + curr_size; =20 s =3D iov_to_buf(iov, out_num, 0, op_info->src, op_info->src_len); @@ -520,11 +522,11 @@ virtio_crypto_sym_op_helper(VirtIODevice *vdev, op_info->dst =3D op_info->data + curr_size; curr_size +=3D op_info->dst_len; =20 - DPRINTF("dst_len=3D%" PRIu32 "\n", op_info->dst_len); + trace_virtio_crypto_sym_op_helper_dst_len(op_info->dst_len); =20 /* Handle the hash digest result */ if (hash_result_len > 0) { - DPRINTF("hash_result_len=3D%" PRIu32 "\n", hash_result_len); + trace_virtio_crypto_sym_op_helper_hash_result_len(hash_result_len); op_info->digest_result =3D op_info->data + curr_size; } =20 diff --git a/include/hw/virtio/virtio-crypto.h b/include/hw/virtio/virtio-c= rypto.h index ffe2391ece..f59f91f11e 100644 --- a/include/hw/virtio/virtio-crypto.h +++ b/include/hw/virtio/virtio-crypto.h @@ -19,17 +19,6 @@ #include "sysemu/iothread.h" #include "sysemu/cryptodev.h" =20 - -#define DEBUG_VIRTIO_CRYPTO 0 - -#define DPRINTF(fmt, ...) \ -do { \ - if (DEBUG_VIRTIO_CRYPTO) { \ - fprintf(stderr, "virtio_crypto: " fmt, ##__VA_ARGS__); \ - } \ -} while (0) - - #define TYPE_VIRTIO_CRYPTO "virtio-crypto-device" #define VIRTIO_CRYPTO(obj) \ OBJECT_CHECK(VirtIOCrypto, (obj), TYPE_VIRTIO_CRYPTO) --=20 2.17.1