[igt-dev] [PATCH i-g-t 2/3] tests/i915/gem_softpin: Add safe-start test

Dixit, Ashutosh ashutosh.dixit at intel.com
Thu Jan 6 13:20:02 UTC 2022


On Thu, 06 Jan 2022 01:55:50 -0800, Zbigniew Kempczyński wrote:
>
> On Wed, Jan 05, 2022 at 06:24:14PM -0800, Dixit, Ashutosh wrote:
> > On Tue, 04 Jan 2022 22:49:44 -0800, Zbigniew Kempczyński wrote:
> > >
> > > +	offset1 = gem_detect_min_start_offset_for_region(i915, region1);
> > > +	offset2 = gem_detect_min_start_offset_for_region(i915, region2);
> > > +	alignment = gem_detect_safe_alignment(i915);
> > > +	igt_debug("safe alignment: %llx\n", (long long) alignment);
> > > +	igt_debug("safe start offset: %llx\n",
> > > +		  (long long) gem_detect_safe_start_offset(i915));
> > > +	igt_debug("minimum object1 start offset: %llx\n", (long long) offset1);
> > > +	igt_debug("minimum object2 start offset: %llx\n", (long long) offset2);
> >
> > Maybe also add print for the min_alignment value here?
>
> You want to print minimum alignment for same region?

Yes, just the min alignment for the pair or regions above.

Otherwise please go ahead and merge. The series is:

Reviewed-by: Ashutosh Dixit <ashutosh.dixit at intel.com>


More information about the igt-dev mailing list