Re: [PATCH 0/6] drm/vc4: kms: Misc fixes for HVS commits

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Nov 17, 2021 at 09:24:54AM +0100, Maxime Ripard wrote:
> Hi,
> 
> On Wed, Nov 17, 2021 at 03:08:31PM +0800, Jian-Hong Pan wrote:
> > Maxime Ripard <maxime@xxxxxxxxxx> 於 2021年11月15日 週一 下午7:31寫道:
> > >
> > > Hi,
> > >
> > > The conversion to DRM commit helpers (f3c420fe19f8, "drm/vc4: kms: Convert to
> > > atomic helpers") introduced a number of issues in corner cases, most of them
> > > showing themselves in the form of either a vblank timeout or use-after-free
> > > error.
> > >
> > > These patches should fix most of them, some of them still being debugged.
> > >
> > > Maxime
> > >
> > > Maxime Ripard (6):
> > >   drm/vc4: kms: Wait for the commit before increasing our clock rate
> > >   drm/vc4: kms: Fix return code check
> > >   drm/vc4: kms: Add missing drm_crtc_commit_put
> > >   drm/vc4: kms: Clear the HVS FIFO commit pointer once done
> > >   drm/vc4: kms: Don't duplicate pending commit
> > >   drm/vc4: kms: Fix previous HVS commit wait
> > >
> > >  drivers/gpu/drm/vc4/vc4_kms.c | 36 ++++++++++++++---------------------
> > >  1 file changed, 14 insertions(+), 22 deletions(-)
> > >
> > > --
> > > 2.33.1
> > >
> > 
> > Thanks to Maxime's information!
> > 
> > I tried to applied this patch series based on the latest mainline
> > kernel at commit commit 8ab774587903 ("Merge tag 'trace-v5.16-5' of
> > git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace"),
> > which almost equals "tags/v5.16-rc1" and tested it on RPi 4B.
> > However, the system hangs and becomes dead at the kernel message:
> > 
> > [drm] Initialized vc4 0.0.0 20140616 for gpu on minor 0
> > 
> > The full dmesg can be found at
> > https://bugzilla.kernel.org/attachment.cgi?id=299603
> > 
> > If I revert the patch series to the original mainline kernel, system
> > can boot up.
> 
> Can you share a bit more information on the boot setup you have? Do you
> have a display connected? If so, on both output or just a single one?

Nevermind, I found what the issue is. I'll send a v2 shortly.

Maxime

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux