[PATCH i-g-t v2 2/2] tests/intel: check drmModeGetConnector() return in i915_pm_rpm

Luca Coelho luca at coelho.fi
Wed Jun 12 13:57:52 UTC 2024


On Fri, 2024-06-07 at 16:39 +0200, Kamil Konieczny wrote:
> Hi Luca,
> On 2024-05-30 at 00:45:12 +0300, Luca Coelho wrote:
> > On Wed, 2024-05-29 at 18:37 +0200, Kamil Konieczny wrote:
> > > Hi Luca,
> > > On 2024-05-28 at 10:58:55 +0300, Luca Coelho wrote:
> > > 
> > > this subject imho is too long:
> > > 
> > > [PATCH i-g-t v2 2/2] tests/intel: check drmModeGetConnector() return in i915_pm_rpm
> > > 
> > > It should have test name at begin, not at end:
> > > 
> > > tests/intel/i915_pm_rpm: Fix bug in init_mode_set_data()
> > > 
> > > or
> > > 
> > > tests/intel/i915_pm_rpm: check drmModeGetConnector() before use
> > > 
> > > or find out something better.
> > 
> > This is the same as in the previous patch.  So, just tell me which one
> > of your options you want me to use and I'll do it.  No need to bike-
> > shed on this here.
> > 
> > 
> > > > The drmModeGetConnector() function can return NULL in some cases, so
> > > > we need to check the return value before accessing it.  This is not
> > > > being checked in init_mode_set_data(), so fix that.
> > > 
> > > Please cite few lines from bug report:
> > > 
> > > Starting subtest: module-reload
> > > Received signal SIGSEGV.
> > > Stack trace: 
> > > #0 [fatal_sig_handler+0x10f]
> > > #1 [killpg+0x40]
> > > #2 [setup_environment+0x33f]
> > 
> > This is one click away, in the link I added below, but I see the
> 
> It is better to give such info here in patch itself,
> especially as this is needed for reproduction.

I agree that sometimes it is.  But this excerpt looks mostly like noise
to me.  The only significant words there are "module-reload" and
"SIGSEGV".

--
Cheers,
Luca.


More information about the igt-dev mailing list