Re: [PATCH 00/16] tests: Add CPU topology related smbios test cases

[Zhao Liu]

On Fri, Sep 15, 2023 at 04:30:55PM +0200, Igor Mammedov wrote:
> Date: Fri, 15 Sep 2023 16:30:55 +0200
> From: Igor Mammedov <imammedo@redhat.com>
> Subject: Re: [PATCH 00/16] tests: Add CPU topology related smbios test cases
> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.38; x86_64-redhat-linux-gnu)
> 
> On Fri, 15 Sep 2023 09:48:08 -0400
> "Michael S. Tsirkin" <mst@redhat.com> wrote:
> 
> > On Tue, Sep 05, 2023 at 02:56:46PM +0800, Zhao Liu wrote:
> > > Hi Michael,
> > > 
> > > On Fri, Sep 01, 2023 at 12:55:33PM +0300, Michael Tokarev wrote:  
> > > > Date: Fri, 1 Sep 2023 12:55:33 +0300
> > > > From: Michael Tokarev <mjt@tls.msk.ru>
> > > > Subject: Re: [PATCH 00/16] tests: Add CPU topology related smbios test 
> > > > cases
> > > > 
> > > > 25.08.2023 06:36, Zhao Liu wrote:  
> > > > > From: Zhao Liu <zhao1.liu@intel.com>
> > > > > 
> > > > > Hi all,
> > > > > 
> > > > > This patchset is the follow up tests of previous topology fixes in
> > > > > smbios [1].
> > > > > 
> > > > > In this patchset, add these test cases:
> > > > > 
> > > > > 1. Add the case to test 2 newly added topology helpers (patch 1):
> > > > >     * machine_topo_get_cores_per_socket()
> > > > >     * machine_topo_get_threads_per_socket()
> > > > > 
> > > > > 2. Add the cases in bios-tables-test.c to:
> > > > >     * test smbios type4 table count (patch 2-4).
> > > > >     * test smbios type4 core count field (patch 5-7).
> > > > >     * update the test of smbios type4 core count2 field (patch 8-10).
> > > > >     * test smbios type4 thread count (patch 11-13).
> > > > >     * test smbios type4 thread count2 (patch 14-16).
> > > > > 
> > > > > With the above new cases, cover all commits of [1] in test.
> > > > > 
> > > > > [1]: 
> > > > > https://lists.gnu.org/archive/html/qemu-devel/2023-06/msg06225.html  
> > > > 
> > > > Hmm.  I definitely didn't expect such a large patchset..  I asked for 
> > > > just
> > > > a small, maybe manual (from command-line) reproducer of the original 
> > > > issue,
> > > > so I can verify it is fixed before applying the change in question to 
> > > > qemu
> > > > stable trees.   
> > > 
> > > Since these issues are found by coding reading, they are "byproducts" of
> > > other topology-related work I've been woring on, not directly identified
> > > by any anomaly...
> > > 
> > > ...So I'm not quite sure what the consequences of these few misuses of
> > > topology fields are, and how they might manifest themselves.
> > >   
> > > > It is a large work.  Lemme at least give it a try in the
> > > > gitlab CI...  
> > > 
> > > The test cases are the ones that clearly define the relevant topology
> > > issues.
> > > 
> > > 
> > > Thanks,
> > > Zhao  
> > 
> > I think it's ok, as long as Igor's happy.
> 
> series looks fine to me
> (modulo cosmetic issues and better commit messages as noted in per patch 
> review)

Thanks Igor!

-Zhao

> 
> > 
> > > > 
> > > > Philippe, can you pick this up for the misc tree?
> > > > 
> > > > /mjt  
> > 
>