[Intel-gfx] [PATCH i-g-t 1/3] tests/gem_exec_fence: Fix wrong engine checked for store_dword capability

Janusz Krzysztofik janusz.krzysztofik at linux.intel.com
Thu Aug 18 15:27:26 UTC 2022


Hi Mauro,

Thanks for reviewing this series, I've just pushed it.

On Wednesday, 17 August 2022 14:53:48 CEST Mauro Carvalho Chehab wrote:
> Hi Janusz,
> 
> On Fri, 12 Aug 2022 11:53:44 +0200
> Janusz Krzysztofik <janusz.krzysztofik at linux.intel.com> wrote:
> 
> It seems that there is a numeration issue on this series, as the patches
> on it are:
> 
>    [PATCH i-g-t 1/3] tests/gem_exec_fence: Fix wrong engine checked for store_dword capability
>    [PATCH i-g-t v2 2/3] tests/gem_exec_fence: Exclude 0  from use in store batches
>    [PATCH i-g-t v3 3/3] tests/gem_exec_fence: Restore  pre-hang checks in *await-hang scenarios
> 
> Maybe some broken script? It is also missing a cover letter.

That was not a script, I provided version numbers of individual patches 
manually, and not provided any cover letter.  First patch was a small fix, not 
directly related to the two others.  Second patch was a small enhancement, 
also not directly related to the third one.  However, the third one depended 
on the two for clean apply, and that was the only reason for me sending them 
in series.

That said, let me ask, based on your huge upstream experience, what are your 
preferences on patch version tagging if one is going to submit a series with 
new versions of some patches while still including some other that don't need 
to be changed?  Should all be marked as new (and the same) versions?

Thanks,
Janusz




More information about the Intel-gfx mailing list