All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: linux-sh@vger.kernel.org
Subject: Re: AP4EVB board couldn't boot on latest linus/master
Date: Tue, 06 Mar 2012 10:14:11 +0000	[thread overview]
Message-ID: <1333815.KhJDah6IKR@avalon> (raw)
In-Reply-To: <87booawe57.wl%kuninori.morimoto.gx@renesas.com>

Hi Morimoto-san,

On Monday 05 March 2012 20:24:08 Kuninori Morimoto wrote:
> Hi SH ML
> 
> Cc Laurent, Rafael
> 
> Could you please give me your help ?
> My AP4EVB board couldn't boot on latest linus/master branch.
> 
> I tried debug/bisect for it, and noticed that it has 2 issues.
> It were
> 
> 1) sh_mobile_lcdc/fbcon seems has dead lock issue
> 2) PM runtime issue (?)
> 
> But mackerel board works well on same tree.
> So, I guess ap4evb.c is missing something.
> 
> ======================> lcdc/fbcon dead lock
> ======================> 
> This issue happen from this merge commit
> 
> 	----------------------------------------------
> 	commit 51be08419dc86c72486ac556aa39bc01026a403d
> 	Merge: ec8013b f787f32
> 	Author: Linus Torvalds <torvalds@linux-foundation.org>
> 	Date:   Sat Jan 14 15:11:19 2012 -0800
> 
> 	    Merge branch 'fbdev-next' of git://github.com/schandinat/linux-2.6
> 
> 	    * 'fbdev-next' of git://github.com/schandinat/linux-2.6: (175 
commits)
> 	      module_param: make bool parameters really bool (drivers/video/i810)
> 	      Revert "atmel_lcdfb: Adjust HFP calculation so it matches the
> manual." ...
> 	----------------------------------------------
> 
> The kernel log was
> 
> 	----------------------------------------------
> 	...
> 	io scheduler noop registered (default)
> 	sh_mobile_lcdc_fb sh_mobile_lcdc_fb.1: start latency exceeded, new value
> 17416 s arm_vmregion_alloc: allocation too big (requested 0x7e9000)
> 	sh_mobile_lcdc_fb sh_mobile_lcdc_fb.1: unable to allocate buffer
> 	sh_mobile_lcdc_fb: probe of sh_mobile_lcdc_fb.1 failed with error -12
> 	sh_mobile_lcdc_fb sh_mobile_lcdc_fb.0: start latency exceeded, new value
> 3750 ns (dead lock)
> 	----------------------------------------------
> 
> I'm not sure why, but console_lock() seems dead lock on 2nd lcdc.
> This is the trace
> 
> sh_mobile_lcdc_probe()
>  - register_framebuffer()
>   - do_register_framebuffer()
>    - fb_notifier_call_chain(FB_EVENT_FB_REGISTERED, xx)
>     - fbcon_event_notify()
>      - fbcon_fb_registered()
>       - fbcon_takeover()
>        - take_over_console()
>         - bind_con_driver()
>          - console_lock()    <== dead lock
> 
> I could skip this issue by removing CONFIG_FRAMEBUFFER_CONSOLE from my
> .config
> 
> ap4evb lcdc_info settings are less than mackerel.
> (note, ap4evb_init adds more settings for it)
> Does it missing something ?

Could you please enable CONFIG_LOCKDEP_SUPPORT support in the kernel to catch 
AB-BA deadlocks ?

> ======================> PM runtime (?) dead
> ======================> 
> I got 2nd issue after skipping above issue.
> It seems happen from this commit
> 
> 	----------------------------------------------
> 	commit 0140d8bd47f798d55c3720f7fcade9e50929a5e5
> 	Author: Rafael J. Wysocki <rjw@sisk.pl>
> 	Date:   Thu Dec 1 00:02:17 2011 +0100
> 
> 	    PM / Domains: Automatically update overoptimistic latency information
> 
> 	    Measure the time of execution of the .stop(), .start(), .save_state()
> 	    and .restore_state() PM domain device callbacks and if the result
> 	    ...
> 	----------------------------------------------
> 
> The kernel log is
> 
> 	----------------------------------------------
> 	...
> 	sh_mobile_lcdc_fb sh_mobile_lcdc_fb.0: start latency exceeded, new value
> 3834 ns graphics fb0: registered sh_mobile_lcdc_fb/mainlcd as 544x961
> 16bpp. SuperH SCI(F) driver initialized
> 	sh-sci.0: ttySC0 at MMIO 0xe6c40000 (irq = 80) is a scifa
> 	sh-sci sh-sci.0: start latency exceeded, new value 4666 ns
> 	console [ttySC0] enabled, bootconsole disabled
> 	console [ttySC0] enabled, bootconsole disabled
> 	(dead)
> 	----------------------------------------------
> 
> what is required for ap4evb ?

I had a similar issue on Mackerel, and I've worked around it by disabling 
CONFIG_SH_TIMER_TMU.

-- 
Regards,

Laurent Pinchart


  reply	other threads:[~2012-03-06 10:14 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-06  4:24 AP4EVB board couldn't boot on latest linus/master Kuninori Morimoto
2012-03-06 10:14 ` Laurent Pinchart [this message]
2012-03-06 21:51 ` Rafael J. Wysocki
2012-03-07  0:11 ` Kuninori Morimoto
2012-03-07  6:33 ` Paul Mundt
2012-03-07  8:48 ` Rafael J. Wysocki

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1333815.KhJDah6IKR@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=linux-sh@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.