[igt-dev] Merge patches that are failing to land in the ML and patchworks

[igt-dev] Merge patches that are failing to land in the ML and patchworks

[Umesh Nerlige Ramappa]

Hi Petri,

Reg: [igt-dev] [PATCH i-g-t v6 00/36] Add DG2 OA test

I posted this series a while ago and I have some large XML files in 
these patches that are not landing in the ML because mail servers reject 
it. Hence, it does not make it to patchworks and I am not able to run 
pre-merge CI on the kernel patches.

To resolve this, I am planning to just merge the XML files and relevant 
scripts as a separate series of 10 patches shared here:

https://gitlab.freedesktop.org/unerlige/igt-gpu-tools/-/commits/master

Once that is merged, I will post the remaining patches to the ML so that 
pre-merge can run. I did a sanity check on these patches with kbl, tgl, 
adlp (with guc) and I don't see any issues.

Requesting an Ack for this.

Thanks,
Umesh

Re: [igt-dev] Merge patches that are failing to land in the ML and patchworks

[Dixit, Ashutosh]

On Fri, 21 Oct 2022 08:35:18 -0700, Umesh Nerlige Ramappa wrote:
>
> Hi Petri,
>
> Reg: [igt-dev] [PATCH i-g-t v6 00/36] Add DG2 OA test
>
> I posted this series a while ago and I have some large XML files in these
> patches that are not landing in the ML because mail servers reject
> it. Hence, it does not make it to patchworks and I am not able to run
> pre-merge CI on the kernel patches.
>
> To resolve this, I am planning to just merge the XML files and relevant
> scripts as a separate series of 10 patches shared here:
>
> https://gitlab.freedesktop.org/unerlige/igt-gpu-tools/-/commits/master
>
> Once that is merged, I will post the remaining patches to the ML so that
> pre-merge can run. I did a sanity check on these patches with kbl, tgl,
> adlp (with guc) and I don't see any issues.
>
> Requesting an Ack for this.

Because the patches and scripts are only OA related, this is:

Acked-by: Ashutosh Dixit <ashutosh.dixit@intel.com>

Lionel,

Could you please ack again on the mailing list if possible (Lionel has
already acked privately previously).

Thanks.
--
Ashutosh

Re: [igt-dev] Merge patches that are failing to land in the ML and patchworks

[Petri Latvala]

On Fri, Oct 21, 2022 at 08:35:18AM -0700, Umesh Nerlige Ramappa wrote:
> Hi Petri,
> 
> Reg: [igt-dev] [PATCH i-g-t v6 00/36] Add DG2 OA test
> 
> I posted this series a while ago and I have some large XML files in these
> patches that are not landing in the ML because mail servers reject it.
> Hence, it does not make it to patchworks and I am not able to run pre-merge
> CI on the kernel patches.
> 
> To resolve this, I am planning to just merge the XML files and relevant
> scripts as a separate series of 10 patches shared here:
> 
> https://gitlab.freedesktop.org/unerlige/igt-gpu-tools/-/commits/master
> 
> Once that is merged, I will post the remaining patches to the ML so that
> pre-merge can run. I did a sanity check on these patches with kbl, tgl, adlp
> (with guc) and I don't see any issues.
> 
> Requesting an Ack for this.

Acked-by: Petri Latvala <petri.latvala@intel.com>

Re: [igt-dev] Merge patches that are failing to land in the ML and patchworks

[Dixit, Ashutosh]

On Fri, 21 Oct 2022 08:35:18 -0700, Umesh Nerlige Ramappa wrote:
>

Hi Umesh/Lionel,

> Hi Petri,
>
> Reg: [igt-dev] [PATCH i-g-t v6 00/36] Add DG2 OA test
>
> I posted this series a while ago and I have some large XML files in these
> patches that are not landing in the ML because mail servers reject
> it. Hence, it does not make it to patchworks and I am not able to run
> pre-merge CI on the kernel patches.
>
> To resolve this, I am planning to just merge the XML files and relevant
> scripts as a separate series of 10 patches shared here:
>
> https://gitlab.freedesktop.org/unerlige/igt-gpu-tools/-/commits/master
>
> Once that is merged, I will post the remaining patches to the ML so that
> pre-merge can run. I did a sanity check on these patches with kbl, tgl,
> adlp (with guc) and I don't see any issues.
>
> Requesting an Ack for this.

I am getting the compile warning below. I tried removing the duplicated
definition from "lib/i915/perf-config: extend the device info" but then
that commit doesn't compile. Could you guys please fix this up and let me
know.

Thanks.
--
Ashutosh

[4/475] Compiling C object tests/core_hotunplug.p/core_hotunplug.c.o
In file included from ../lib/igt.h:29,
                 from ../tests/core_hotunplug.c:37:
../lib/igt_aux.h:334: warning: "DIV_ROUND_UP" redefined
  334 | #define DIV_ROUND_UP(n, d) (((n) + (d) - 1) / (d))
      |
In file included from ../tests/core_hotunplug.c:36:
../lib/i915/perf.h:35: note: this is the location of the previous definition
   35 | #define DIV_ROUND_UP(a, b)  (((a) + (b) - 1) / (b))
      |
[5/475] Compiling C object tests/perf.p/i915_perf.c.o
In file included from ../lib/igt.h:29,
                 from ../tests/i915/perf.c:43:
../lib/igt_aux.h:334: warning: "DIV_ROUND_UP" redefined
  334 | #define DIV_ROUND_UP(n, d) (((n) + (d) - 1) / (d))
      |
In file included from ../tests/i915/perf.c:42:
../lib/i915/perf.h:35: note: this is the location of the previous definition
   35 | #define DIV_ROUND_UP(a, b)  (((a) + (b) - 1) / (b))
      |
[84/84] Linking target tests/perf

Re: [igt-dev] Merge patches that are failing to land in the ML and patchworks

[Umesh Nerlige Ramappa]

On Mon, Oct 24, 2022 at 08:34:00AM -0700, Dixit, Ashutosh wrote:
>On Fri, 21 Oct 2022 08:35:18 -0700, Umesh Nerlige Ramappa wrote:
>>
>
>Hi Umesh/Lionel,
>
>> Hi Petri,
>>
>> Reg: [igt-dev] [PATCH i-g-t v6 00/36] Add DG2 OA test
>>
>> I posted this series a while ago and I have some large XML files in these
>> patches that are not landing in the ML because mail servers reject
>> it. Hence, it does not make it to patchworks and I am not able to run
>> pre-merge CI on the kernel patches.
>>
>> To resolve this, I am planning to just merge the XML files and relevant
>> scripts as a separate series of 10 patches shared here:
>>
>> https://gitlab.freedesktop.org/unerlige/igt-gpu-tools/-/commits/master
>>
>> Once that is merged, I will post the remaining patches to the ML so that
>> pre-merge can run. I did a sanity check on these patches with kbl, tgl,
>> adlp (with guc) and I don't see any issues.
>>
>> Requesting an Ack for this.
>
>I am getting the compile warning below. I tried removing the duplicated
>definition from "lib/i915/perf-config: extend the device info" but then
>that commit doesn't compile. Could you guys please fix this up and let me
>know.

Hi Ashutosh,

sorry, forgot to fix this earlier. Synced with Lionel and pushed an 
update to the commit here:

https://gitlab.freedesktop.org/unerlige/igt-gpu-tools/-/commit/cfb7fbc0ecd7de181c2cb57a2bf18618ad3d4fd5

Hi Lionel,

The perf version is already bumped once in this patch and since this is 
not yet merged, I am thinking another bump is not needed. Please let me 
know if any concerns.

Thanks,
Umesh

>
>Thanks.
>--
>Ashutosh
>
>[4/475] Compiling C object tests/core_hotunplug.p/core_hotunplug.c.o
>In file included from ../lib/igt.h:29,
>                 from ../tests/core_hotunplug.c:37:
>../lib/igt_aux.h:334: warning: "DIV_ROUND_UP" redefined
>  334 | #define DIV_ROUND_UP(n, d) (((n) + (d) - 1) / (d))
>      |
>In file included from ../tests/core_hotunplug.c:36:
>../lib/i915/perf.h:35: note: this is the location of the previous definition
>   35 | #define DIV_ROUND_UP(a, b)  (((a) + (b) - 1) / (b))
>      |
>[5/475] Compiling C object tests/perf.p/i915_perf.c.o
>In file included from ../lib/igt.h:29,
>                 from ../tests/i915/perf.c:43:
>../lib/igt_aux.h:334: warning: "DIV_ROUND_UP" redefined
>  334 | #define DIV_ROUND_UP(n, d) (((n) + (d) - 1) / (d))
>      |
>In file included from ../tests/i915/perf.c:42:
>../lib/i915/perf.h:35: note: this is the location of the previous definition
>   35 | #define DIV_ROUND_UP(a, b)  (((a) + (b) - 1) / (b))
>      |
>[84/84] Linking target tests/perf

Re: [igt-dev] Merge patches that are failing to land in the ML and patchworks

[Dixit, Ashutosh]

On Mon, 24 Oct 2022 12:48:49 -0700, Umesh Nerlige Ramappa wrote:
>

Thanks Umesh, this is now merged.

Ashutosh

> On Mon, Oct 24, 2022 at 08:34:00AM -0700, Dixit, Ashutosh wrote:
> > On Fri, 21 Oct 2022 08:35:18 -0700, Umesh Nerlige Ramappa wrote:
> >>
> >
> > Hi Umesh/Lionel,
> >
> >> Hi Petri,
> >>
> >> Reg: [igt-dev] [PATCH i-g-t v6 00/36] Add DG2 OA test
> >>
> >> I posted this series a while ago and I have some large XML files in these
> >> patches that are not landing in the ML because mail servers reject
> >> it. Hence, it does not make it to patchworks and I am not able to run
> >> pre-merge CI on the kernel patches.
> >>
> >> To resolve this, I am planning to just merge the XML files and relevant
> >> scripts as a separate series of 10 patches shared here:
> >>
> >> https://gitlab.freedesktop.org/unerlige/igt-gpu-tools/-/commits/master
> >>
> >> Once that is merged, I will post the remaining patches to the ML so that
> >> pre-merge can run. I did a sanity check on these patches with kbl, tgl,
> >> adlp (with guc) and I don't see any issues.
> >>
> >> Requesting an Ack for this.
> >
> > I am getting the compile warning below. I tried removing the duplicated
> > definition from "lib/i915/perf-config: extend the device info" but then
> > that commit doesn't compile. Could you guys please fix this up and let me
> > know.
>
> Hi Ashutosh,
>
> sorry, forgot to fix this earlier. Synced with Lionel and pushed an update
> to the commit here:
>
> https://gitlab.freedesktop.org/unerlige/igt-gpu-tools/-/commit/cfb7fbc0ecd7de181c2cb57a2bf18618ad3d4fd5
>
> Hi Lionel,
>
> The perf version is already bumped once in this patch and since this is not
> yet merged, I am thinking another bump is not needed. Please let me know if
> any concerns.
>
> Thanks,
> Umesh
>
> >
> > Thanks.
> > --
> > Ashutosh
> >
> > [4/475] Compiling C object tests/core_hotunplug.p/core_hotunplug.c.o
> > In file included from ../lib/igt.h:29,
> >                 from ../tests/core_hotunplug.c:37:
> > ../lib/igt_aux.h:334: warning: "DIV_ROUND_UP" redefined
> >  334 | #define DIV_ROUND_UP(n, d) (((n) + (d) - 1) / (d))
> >      |
> > In file included from ../tests/core_hotunplug.c:36:
> > ../lib/i915/perf.h:35: note: this is the location of the previous definition
> >   35 | #define DIV_ROUND_UP(a, b)  (((a) + (b) - 1) / (b))
> >      |
> > [5/475] Compiling C object tests/perf.p/i915_perf.c.o
> > In file included from ../lib/igt.h:29,
> >                 from ../tests/i915/perf.c:43:
> > ../lib/igt_aux.h:334: warning: "DIV_ROUND_UP" redefined
> >  334 | #define DIV_ROUND_UP(n, d) (((n) + (d) - 1) / (d))
> >      |
> > In file included from ../tests/i915/perf.c:42:
> > ../lib/i915/perf.h:35: note: this is the location of the previous definition
> >   35 | #define DIV_ROUND_UP(a, b)  (((a) + (b) - 1) / (b))
> >      |
> > [84/84] Linking target tests/perf