From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 46F32C41513 for ; Thu, 27 Jul 2023 16:42:20 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1qP3Qv-00064e-4P; Thu, 27 Jul 2023 12:01:45 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qP3Qr-000645-2u for qemu-devel@nongnu.org; Thu, 27 Jul 2023 12:01:41 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qP3Qp-0000wU-FP for qemu-devel@nongnu.org; Thu, 27 Jul 2023 12:01:40 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1690473698; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=YzMaIOfVQmAPP8N/4t53NHRrB5DicjuSWVIguLQOurE=; b=B4lQMdS31yzPbAr4op+UfXHAJRMeMtSpjPWBJLOx/3Lt/4u+Q+VSZCDiYfmsg+ajkSMod1 gypyEaTFUNg/DNzm7HDd0jjg/VeiOkcEXk8qvS0xrlBzAKWAsPmyw+yNYaPMF/IBjap7uW uASOdPPJPWQsorz+6QMHrY8oc+RaLc4= Received: from mimecast-mx02.redhat.com (66.187.233.73 [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-520-eA15f1n3MHWw52NtQHm-YQ-1; Thu, 27 Jul 2023 12:01:34 -0400 X-MC-Unique: eA15f1n3MHWw52NtQHm-YQ-1 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.rdu2.redhat.com [10.11.54.4]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id A136B381AE62; Thu, 27 Jul 2023 16:01:33 +0000 (UTC) Received: from localhost (unknown [10.39.193.84]) by smtp.corp.redhat.com (Postfix) with ESMTP id C6EEB200BA73; Thu, 27 Jul 2023 16:01:32 +0000 (UTC) Date: Thu, 27 Jul 2023 12:01:31 -0400 From: Stefan Hajnoczi To: Jeuk Kim Cc: qemu-devel@nongnu.org, fam@euphon.net, hreitz@redhat.com, k.jensen@samsung.com, kwolf@redhat.com, pbonzini@redhat.com, qemu-block@nongnu.org, berrange@redhat.com, marcandre.lureau@redhat.com, marcel.apfelbaum@gmail.com, mst@redhat.com, philmd@linaro.org, thuth@redhat.com, lvivier@redhat.com, jeuk20.kim@samsung.com Subject: Re: [PATCH v8 0/4] hw/ufs: Add Universal Flash Storage (UFS) support Message-ID: <20230727160131.GB979354@fedora> References: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="TcBOEFnfLOCVRKIG" Content-Disposition: inline In-Reply-To: X-Scanned-By: MIMEDefang 3.1 on 10.11.54.4 Received-SPF: pass client-ip=170.10.133.124; envelope-from=stefanha@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org --TcBOEFnfLOCVRKIG Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Jul 27, 2023 at 05:45:17PM +0900, Jeuk Kim wrote: > Since v7: > In ufs-test.c, make the following changes > - Change TIMEOUT from 5 to 10 (Thomas's review comment) > - Rename the temporary file to "qemu-ufs.XXXX" (Thomas's review comment) > - Use "-blockdev" instead of "-drive" (Stefan's review comment) >=20 > Since v6: > - Add tests/qtest/ufs-test.c to test ufs initialisation and I/O > - Add struct UtpTaskReqDesc to include/block/ufs.h > - Fix ufs_log2() logic > - Fix ufs-lu to use 4K as default block size to match the ufs spec >=20 > Since I created a new file, tests/qtest/ufs-test.c, I added Laurent Vivie= r to the cc list. > Thank you. >=20 > Since v5: > - Fix to print an error message instead of a segmentation fault > when no drive property is specified for a ufs-lu device >=20 > Since v4: > Addressed review comment from Stefan Hajnoczi. The fixes are as > follows. > - Keep u->reg fields in host endian (Removed little-endian helper=20 > functions from MemoryRegionOps) > - Remove unnecessary NULL checks for g_new and g_malloc0 > - Replace DEFINE_PROP_DRIVE_IOTHREAD -> DEFINE_PROP_DRIVE >=20 > In case you were wondering, ufs and ufs-lu have been tested for the > following behaviours. > 1. Checked ufs device recognition in Windows10 environment > 2. Verified ufs device recognition in Ubuntu 22.04 environment > 3. Verified io behaviour via fio in Ubuntu 22.04 environment > 4. Verified query request via ufs-tools in Ubuntu 22.04 environment >=20 > Since v3: > - Replace softmmu_ss -> system_ss in meson >=20 > Since v2: > Addressed review comment from Stefan Hajnoczi. The main fixes are as > follows. > - Use of SPDX licence identifiers > - fixed endianness error > - removed memory leak > - fixed DMA error handling logic >=20 > Since v1: > - use macros of "hw/registerfields.h" (Addressed Philippe's review > comments) >=20 > This patch series adds support for a new PCI-based UFS device. >=20 > The UFS pci device id (PCI_DEVICE_ID_REDHAT_UFS) is not registered > in the Linux kernel yet, so it does not work right away, but I confirmed > that it works with Linux when the UFS pci device id is registered. >=20 > I have also verified that it works with Windows 10. >=20 > Jeuk Kim (4): > hw/ufs: Initial commit for emulated Universal-Flash-Storage > hw/ufs: Support for Query Transfer Requests > hw/ufs: Support for UFS logical unit > tests/qtest: Introduce tests for UFS Thank you! QEMU is currently in freeze for the 8.1 release. I have merged your patches on my block-next branch. I will send the pull request when qemu.git/master opens again after freeze around August 15th or 22nd (see https://wiki.qemu.org/Planning/8.1). You are the maintainer for hw/ufs/ so you can send pull requests in the future. Please continue to send your own patches to qemu-devel to give others in the community an opportunity to review them. If no one reviews your patches, then I think it's fair to include them in your own pull requests after a week. There is a diagram of the development model here: https://wiki.qemu.org/Contribute/DevelopmentProcess Please also read the documentation about maintainers: https://gitlab.com/qemu-project/qemu/-/blob/master/docs/devel/maintainers.r= st If you'd rather not send pull requests yourself, you can CC me on UFS patches that you'd like to have merged and I'll include them in my block tree pull requests. Thanks, Stefan --TcBOEFnfLOCVRKIG Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEEhpWov9P5fNqsNXdanKSrs4Grc8gFAmTClNsACgkQnKSrs4Gr c8joEQf/YVDiJ2IKRme5SoxMhFJmCWE78y4zzZkfUHp2wrV7wf9vsaUqLQZSW/TD pYGMl+HiQlkfitKnIuqVGuu6mc4QzF3eLoem6JoCpowPQHDPvgTSinz/CJqn7kxH +XmW1xbjHF8O5h+PDnQvJmFUrDFfv8t4bFPudTrZnZXeI8R5gt7ZvChcT2gTT4VU E8O9ZjzBTdpbsf8J3nXZWTFqSS9pQz8lDkTzKRplVGCnQ3me3LYr7ps50Bd5q29W YYO/P2G1KSQq9MnBBz04Fwx0jE9IhXYJUH3P9NWI4YvBkYqeczlKY3nogDK7at1s GAYtQZ8lCXtHtdjWM/1xBVqC9c1UhA== =afHh -----END PGP SIGNATURE----- --TcBOEFnfLOCVRKIG--