All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gilles Chanteperdrix <gilles.chanteperdrix@xenomai.org>
To: Richard Cochran <richard.cochran@domain.hid>
Cc: adeos-main@gna.org
Subject: Re: [Adeos-main] FW: [PATCH] repost: ARM FCSE
Date: Sun, 21 Sep 2008 17:52:36 +0200	[thread overview]
Message-ID: <48D66DC4.8030706@domain.hid> (raw)
In-Reply-To: <899865CA54E4444DAF2E3639C04C5F48E4DA6A@trillian.at.omicron.at>

Richard Cochran wrote:
> I posted this patch today on linux-arm-kernel, but I repeat it
> here because there does not seem to be too much interest on that
> list for the ARM FCSE.
> 
> I also tried to combine this patch with ipipe for kernel 2.6.20
> running on the Intel IXDP465, but after booting I soon get a BUG.
> 
> Anyhow, perhaps the ARM people might take a look at combining
> ipipe with FCSE...

Ok. Six monthes later, I finally gave a try to your patch on at91rm9200,
which supports FCSE as well.

When booting, I get random segmentation faults (either with or without
the I-pipe), assertion which fails in glibc, and such things.

I looked at the code, and have a few questions:
- in cpu_switch_mm, you comment out the cache flush but keep the TLB
flush, is the TLB flush really needed ?
- in pmd_populate (file include/asm-arm/pgalloc.h), you populate two
pmds, does not this look wrong ? I mean it looks like pmd_populate is
called with the two "kinds" (translated and untranslated) addresses,
which should not happen.
- in exit_mmap (file mm/mmap.c), we should arrange for flush_cache_mm to
flush the whole cache (I mean without consulting mm->cpu_vm_mask) to
avoid keeping stale entries pointing to pages which are going to be
returned to the system. I tried to change this, but I still get the
random failures.
- anyone still has the old FASS patch ? Perhaps by looking at it we
could find what is missing in your patch.


-- 
					    Gilles.


  parent reply	other threads:[~2008-09-21 15:52 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-07 16:19 [Adeos-main] FW: [PATCH] repost: ARM FCSE Richard Cochran
2008-03-07 16:31 ` Gilles Chanteperdrix
2008-09-21 15:52 ` Gilles Chanteperdrix [this message]
2008-09-21 16:30   ` Gilles Chanteperdrix
2008-09-23 18:32   ` Gilles Chanteperdrix
2008-09-24  5:47     ` Richard Cochran
2008-09-24  7:38       ` Gilles Chanteperdrix
2008-09-27 17:53     ` Gilles Chanteperdrix
2008-10-02  8:44       ` Gilles Chanteperdrix
2008-10-02 14:54         ` Richard Cochran
2008-10-02 15:30           ` Gilles Chanteperdrix
2008-10-02 17:36             ` Gilles Chanteperdrix
2008-10-02 20:03           ` Gilles Chanteperdrix
2008-10-03  7:20             ` Richard Cochran
2008-10-02 22:04         ` Bosko Radivojevic
2008-10-02 22:34           ` Gilles Chanteperdrix
2008-10-03  1:57             ` Philippe Gerum

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=48D66DC4.8030706@domain.hid \
    --to=gilles.chanteperdrix@xenomai.org \
    --cc=adeos-main@gna.org \
    --cc=richard.cochran@domain.hid \
    /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.