public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: iamjoonsoo.kim@lge.com (Joonsoo Kim)
To: linux-arm-kernel@lists.infradead.org
Subject: n900 in next-20170901
Date: Wed, 13 Sep 2017 16:55:17 +0900	[thread overview]
Message-ID: <20170913075516.GA25036@js1304-P5Q-DELUXE> (raw)
In-Reply-To: <20170907161650.GP5024@atomide.com>

On Thu, Sep 07, 2017 at 09:16:51AM -0700, Tony Lindgren wrote:
> * Joonsoo Kim <iamjoonsoo.kim@lge.com> [170907 00:30]:
> > On Wed, Sep 06, 2017 at 06:30:57AM -0700, Tony Lindgren wrote:
> > > Hi,
> > > 
> > > * Joonsoo Kim <iamjoonsoo.kim@lge.com> [170905 16:32]:
> > > > I think that I made a mistake for configuration CONFIG_HIGHMEM=y and
> > > > CONFIG_HAVE_MEMBLOCK_NODE_MAP=y. In this case, the MOVABLE_ZONE can
> > > > be *!highmem*. Could you check that your configuration have above
> > > > options?
> > > 
> > > CONFIG_HIGHMEM is set yeah.
> > > 
> > > > And, could you check that following patch works for you?
> > > 
> > > Does not seem to help, tried against next with just 9caf25f996e8
> > > revert and also with 9caf25f996e8.
> > 
> > Oops. I misunderstood your problem. Could you test with
> > CONFIG_DEBUG_VIRTUAL?
> 
> Sure.
> 
> > After commit 9caf25f996e8, user for CMA memory should use to check
> > PageHighmem in order to get proper virtual address of the page. If
> > someone doesn't use it, it is possible to use wrong virtual address
> > and it then causes the use of wrong physical address.
> > CONFIG_DEBUG_VIRTUAL would catch this case.
> 
> OK, no extra output of current next with CONFIG_DEBUG_VIRTUAL=y.
> Booting of n900 hangs with just the same error:
> 
> save_secure_sram() returns 0000ff02
> 
> > If it doesn't help, is there a way to test n900 configuration in QEMU?
> 
> I doubt that QEMU n900 boots in secure mode but instead shows
> the SoC as general purpose SoC. If so, you'd have to patch the
> omap3_save_secure_ram_context() to attempt to save secure RAM
> context in all cases. If that works then debugging with any
> omap3 board like beagleboard in QEMU should work.

Sorry for late response.

I tried to emulate beagle board by using QEMU and now I find the way
and it works. However, it doesn't call omap3_save_secure_ram_context()
due to different omap_type(). And, even if I call it forcibly, the
system dies with prefetch abort regardless of commit 9caf25f996e8.

Could you let me know the better way to test your situation?

Anyway, could you test linux-next with 'CONFIG_HIGHMEM = n'?
I'd like to know if the issue is related to the change that
all CMA memory is managed like as highmem.

Thanks.

  reply	other threads:[~2017-09-13  7:55 UTC|newest]

Thread overview: 59+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-03 20:37 n900 in next-20170901 Pavel Machek
2017-09-05 20:13 ` Tony Lindgren
2017-09-05 20:27   ` Vlastimil Babka
2017-09-05 20:32     ` Tony Lindgren
2017-09-05 23:32   ` Joonsoo Kim
2017-09-06 13:30     ` Tony Lindgren
2017-09-07  7:30       ` Joonsoo Kim
2017-09-07 16:16         ` Tony Lindgren
2017-09-13  7:55           ` Joonsoo Kim [this message]
2017-09-13 16:31             ` Tony Lindgren
2017-09-15  6:56               ` Joonsoo Kim
2017-09-21 17:28                 ` Tony Lindgren
2017-09-25  8:08                   ` Joonsoo Kim
2017-09-25 14:54                     ` Tony Lindgren
2017-10-18  8:29                       ` Joonsoo Kim
2017-10-19 18:30                         ` Tony Lindgren
2017-10-20  1:55                           ` Joonsoo Kim
2017-10-20 17:31                             ` Tony Lindgren
2017-10-23  4:53                               ` Joonsoo Kim
2017-10-25 17:31                                 ` Tony Lindgren
2017-10-26  4:48                                   ` Joonsoo Kim
2017-10-26 14:16                                     ` Tony Lindgren
2017-11-07  5:33                                       ` Joonsoo Kim
2017-11-07 15:48                                         ` Tony Lindgren
2017-11-08  7:46                                           ` Joonsoo Kim
2017-11-08 16:34                                             ` Tony Lindgren
2017-11-09  0:08                                               ` Joonsoo Kim
2017-11-09  0:11                                                 ` Tony Lindgren
2017-11-09  0:36                                                   ` Joonsoo Kim
2017-11-09  3:50                                                     ` Joonsoo Kim
2017-11-09 15:08                                                       ` Tony Lindgren
2017-11-10  0:13                                                         ` Joonsoo Kim
2017-11-10  3:26                                                           ` Tony Lindgren
2017-11-10  6:19                                                             ` Tony Lindgren
2017-11-10  6:23                                                               ` Tony Lindgren
2017-11-10  6:46                                                                 ` Joonsoo Kim
2017-11-10 15:37                                                                   ` Tony Lindgren
2017-11-10  6:37                                                             ` Joonsoo Kim
2017-11-10 15:36                                                               ` Tony Lindgren
2017-11-13 21:15                                                                 ` Tony Lindgren
2017-11-14  6:40                                                                   ` Joonsoo Kim
2017-11-14  6:37                                                                 ` Joonsoo Kim
2017-11-14 17:37                                                                   ` Tony Lindgren
2017-11-14 19:31                                                                     ` Tero Kristo
2017-11-14 19:44                                                                       ` Tony Lindgren
2017-11-14 20:01                                                                         ` Tero Kristo
2017-11-14 20:54                                                                           ` Tony Lindgren
2017-11-15  0:51                                                                     ` Joonsoo Kim
2017-11-15  2:04                                                                       ` Tony Lindgren
2017-11-15  2:48                                                                         ` Joonsoo Kim
2017-11-15  2:53                                                                           ` Tony Lindgren
2017-09-15 13:18           ` Pavel Machek
2017-09-18  2:01             ` Joonsoo Kim
2017-09-18  8:11               ` Linux-next broken for 2 weeks was " Pavel Machek
2017-09-18 22:00                 ` Stephen Rothwell
2017-09-18 22:16                   ` Pavel Machek
2017-09-15 13:28         ` Pali Rohár
2017-09-18  2:07           ` Joonsoo Kim
2017-09-08  9:31   ` Pavel Machek

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=20170913075516.GA25036@js1304-P5Q-DELUXE \
    --to=iamjoonsoo.kim@lge.com \
    --cc=linux-arm-kernel@lists.infradead.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox