Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <4185ee25-52af-8271-c4de-3824490fba6a@sjtu.edu.cn>
Date:   Thu, 21 Jul 2022 19:55:31 +0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.11.0
Subject: Re: [PATCH v3 0/4] virtio/virtio_test
Content-Language: en-US
To:     jasowang@redhat.com, mst@redhat.com
Cc:     eperezma@redhat.com, virtualization@lists.linux-foundation.org,
        linux-kernel@vger.kernel.org, sgarzare@redhat.com
References: <20220709022745.21020-1-qtxuning1999@sjtu.edu.cn>
From:   Guo Zhi <qtxuning1999@sjtu.edu.cn>
In-Reply-To: <20220709022745.21020-1-qtxuning1999@sjtu.edu.cn>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 2022/7/9 10:27, Guo Zhi wrote:
> Original virtio_test only use add one descriptor for each io event, thus code of descriptor chain and indirection have not been tested(one descriptor will not use indirect feature even indirect feature has been specified). In fact it would have not been possible for vhost_test to access to the indirect descriptor table, because it's impossible for virtio_ring.c to allocate it.
>
> This series using descriptor chain and enable indirection feature. And through gcov we find the code coverage has been improved(not high for virtio_ring.c because virtio_test only test split virtqueue):
>
> +------------+-------------+-------------+
> |            |virtio_test.c|virtio_ring.c|
> +------------+-------------+-------------+
> | original   |   72.32%    |   24.71%    |
> +------------+-------------+-------------+
> | current    |    75%      |   28.05%    |
> +------------+-------------+-------------+
>
> Guo Zhi (4):
>    virtio_test: kick vhost for a batch of descriptors
>    virtio_test: use random length scatterlists to test descriptor chain
>    virtio_test: enable indirection feature
>    virtio_test: pregenerate random numbers
>
>   tools/virtio/virtio_test.c | 85 ++++++++++++++++++++++++++++++--------
>   1 file changed, 68 insertions(+), 17 deletions(-)
>
Hi, the new version patch of virtio_test is waiting for review:)

Thanks.