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

["Dixit, Ashutosh" <ashutosh.dixit@intel.com>]

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