linux-kernel - Re: [igt-dev] [PATCH i-g-t v2 3/4] lib/igt

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <4438162.8F6SAcFxjW@jkrzyszt-mobl1.ger.corp.intel.com>
Date:   Tue, 14 Feb 2023 10:50:30 +0100
From:   Janusz Krzysztofik <janusz.krzysztofik@...ux.intel.com>
To:     Isabella Basso <isabbasso@...eup.net>
Cc:     igt-dev@...ts.freedesktop.org, linux-kselftest@...r.kernel.org,
        Daniel Vetter <daniel@...ll.ch>,
        KUnit Development <kunit-dev@...glegroups.com>,
        Magali Lemes <magalilemes00@...il.com>,
        Maíra Canal <maira.canal@....br>,
        Daniel Latypov <dlatypov@...gle.com>, n@...aprado.net,
        kernel list <linux-kernel@...r.kernel.org>,
        leandro.ribeiro@...labora.com,
        ML dri-devel <dri-devel@...ts.freedesktop.org>,
        David Gow <davidgow@...gle.com>,
        Tales Aparecida <tales.aparecida@...il.com>,
        Shuah Khan <skhan@...uxfoundation.org>,
        André Almeida <andrealmeid@...eup.net>,
        Brendan Higgins <brendanhiggins@...gle.com>,
        Trevor Woerner <twoerner@...il.com>,
        Mauro Carvalho Chehab <mauro.chehab@...ux.intel.com>
Subject: Re: [igt-dev] [PATCH i-g-t v2 3/4] lib/igt_kmod: add compatibility for KUnit

Hi Isabella,

On Friday, 10 February 2023 17:55:34 CET Isabella Basso wrote:
> Hi, Janusz,
> 
> > Am 2023-02-10 um 11:56 AM schrieb Janusz Krzysztofik <janusz.krzysztofik@...ux.intel.com>:
> > 
> > Hi Isabella,
> > 
> > On Monday, 19 September 2022 22:55:44 CET Isabella Basso wrote:
> >> Hi, Janusz,
> >> 
> >>> Am 09/09/2022 um 12:18 PM schrieb Janusz Krzysztofik <janusz.krzysztofik@...ux.intel.com>:
> >>> 
> >>> ...
> >>> 
> >>> Anyway, related to my comment about naming that function a parser, I think the 
> >>> best approach would be for that parser to return a generic set of results from 
> >>> kunit execution, then we could feed that data into an IGT specific handler 
> >>> that would convert them to IGT results (SUCCESS, FAIL, or SKIP) as if returned 
> >>> by a set of IGT dynamic subtests.
> >> 
> >> That sounds like a good idea to me, I might take some extra time before v3 to
> >> do that, though. 
> > 
> > Were you able to make any progress?  Do you need any help?
> 
> I’ve already handled most common cases but I still have to address Chehab’s comments — specifically in what concerns crashes. I’ve been doing most of the development by myself as GSoC has ended so I’m taking my time on this.
> 
> My most up-to-date work is available at [1], so if you have any suggestions as to what might be done I’d love to hear them before sending out v3 to the mailing list.
> 
> [1] - https://gitlab.freedesktop.org/isinyaaa/igt-gpu-tools/-/merge_requests/1

Thanks for pointing me to that merge request, it's great to know there is 
some progress.  I've just had a look and I can see you are now working on some 
comments from Maria and Tales.  I've requested to be notified on new posts to 
that merge request so I hope to review your next update when available.

As soon as you resolve issues and feel comfortable with sharing your work, 
please submit your changes to the igt-dev list to give them a try on our CI 
system.

Thanks,
Janusz

> 
> Cheers,
> --
> Isabella Basso
> 
> > Thanks,
> > Janusz
> > 
> > 
> 
>