All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anton Vorontsov <avorontsov@ru.mvista.com>
To: Sebastien Dugue <sebastien.dugue@bull.net>
Cc: tklein@de.ibm.com, tinytim@us.ibm.com,
	Linux-rt <linux-rt-users@vger.kernel.org>,
	jean-pierre.dion@bull.net, themann@de.ibm.com,
	netdev@vger.kernel.org,
	linux-kernel <linux-kernel@vger.kernel.org>,
	Thomas Klein <osstklei@de.ibm.com>,
	linux-ppc <linuxppc-dev@ozlabs.org>,
	raisch@de.ibm.com, gilles.carry@ext.bull.net
Subject: Re: [PATCH HACK] powerpc: quick hack to get a functional eHEA with hardirq preemption
Date: Tue, 16 Sep 2008 15:59:47 +0400	[thread overview]
Message-ID: <20080916115947.GA6995@oksana.dev.rtsoft.ru> (raw)
In-Reply-To: <20080915151332.6a3a7c80@bull.net>

On Mon, Sep 15, 2008 at 03:13:32PM +0200, Sebastien Dugue wrote:
[...]
> > we are a bit worried about putting this into the mainstream part of non real
> > time linux.
> 
>   Heck, I sure do not want this to be applied mainstream nor into any tree.
> The sole purpose of this patch was to trigger some reaction from the people who
> know the hardware and try to understand where the problem lies.
> 
> > There interrupts work perfectly fine, and it was a bit of a
> > challenge to get there for all cases / configurations / machines.
> 
>   Agreed, but the fact that it fails with hardirq preemption leads me to
> believe (without any more knowledge about the harware) that there might be
> something amiss with this driver (or the code concerning the XICS)
> nevertheless.
> 
> > 
> > Could you try to enable these changes only for RT-Linux via a real-time
> > kconfig switch?
> 
>   Nope, this is just a quick hack that allows me to have a functional eHEA under
> the rt kernel. I want to understand what the problem is:
> 
>   - Is the eHEA really delivering level interrupts to the XICS?
> 
>   - Is the XICS loosing interrupts when they are masked?

There is a known bug in the -rt kernels, the bug causes handlers
to lose edge interrupts.

See this patch:

http://lkml.org/lkml/2008/6/30/372

>   - ...?

-- 
Anton Vorontsov
email: cbouatmailru@gmail.com
irc://irc.freenode.net/bd2

WARNING: multiple messages have this Message-ID (diff)
From: Anton Vorontsov <avorontsov@ru.mvista.com>
To: Sebastien Dugue <sebastien.dugue@bull.net>
Cc: Thomas Klein <osstklei@de.ibm.com>,
	tklein@de.ibm.com, tinytim@us.ibm.com,
	Linux-rt <linux-rt-users@vger.kernel.org>,
	themann@de.ibm.com, netdev@vger.kernel.org,
	linux-kernel <linux-kernel@vger.kernel.org>,
	jean-pierre.dion@bull.net, linux-ppc <linuxppc-dev@ozlabs.org>,
	raisch@de.ibm.com, gilles.carry@ext.bull.net
Subject: Re: [PATCH HACK] powerpc: quick hack to get a functional eHEA with hardirq preemption
Date: Tue, 16 Sep 2008 15:59:47 +0400	[thread overview]
Message-ID: <20080916115947.GA6995@oksana.dev.rtsoft.ru> (raw)
In-Reply-To: <20080915151332.6a3a7c80@bull.net>

On Mon, Sep 15, 2008 at 03:13:32PM +0200, Sebastien Dugue wrote:
[...]
> > we are a bit worried about putting this into the mainstream part of non real
> > time linux.
> 
>   Heck, I sure do not want this to be applied mainstream nor into any tree.
> The sole purpose of this patch was to trigger some reaction from the people who
> know the hardware and try to understand where the problem lies.
> 
> > There interrupts work perfectly fine, and it was a bit of a
> > challenge to get there for all cases / configurations / machines.
> 
>   Agreed, but the fact that it fails with hardirq preemption leads me to
> believe (without any more knowledge about the harware) that there might be
> something amiss with this driver (or the code concerning the XICS)
> nevertheless.
> 
> > 
> > Could you try to enable these changes only for RT-Linux via a real-time
> > kconfig switch?
> 
>   Nope, this is just a quick hack that allows me to have a functional eHEA under
> the rt kernel. I want to understand what the problem is:
> 
>   - Is the eHEA really delivering level interrupts to the XICS?
> 
>   - Is the XICS loosing interrupts when they are masked?

There is a known bug in the -rt kernels, the bug causes handlers
to lose edge interrupts.

See this patch:

http://lkml.org/lkml/2008/6/30/372

>   - ...?

-- 
Anton Vorontsov
email: cbouatmailru@gmail.com
irc://irc.freenode.net/bd2

  reply	other threads:[~2008-09-16 11:59 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-15  8:04 [PATCH HACK] powerpc: quick hack to get a functional eHEA with hardirq preemption Sebastien Dugue
2008-09-15  8:04 ` Sebastien Dugue
2008-09-15 12:17 ` Jan-Bernd Themann
2008-09-15 12:17   ` Jan-Bernd Themann
2008-09-15 12:35 ` Thomas Klein
2008-09-15 12:35   ` Thomas Klein
2008-09-15 13:13   ` Sebastien Dugue
2008-09-15 13:13     ` Sebastien Dugue
2008-09-16 11:59     ` Anton Vorontsov [this message]
2008-09-16 11:59       ` Anton Vorontsov
2008-09-16 12:22       ` Sebastien Dugue
2008-09-16 12:22         ` Sebastien Dugue
2008-09-18  7:53 ` Christoph Raisch
2008-09-18  7:53   ` Christoph Raisch
2008-09-18  9:27   ` Sebastien Dugue
2008-09-18  9:27     ` Sebastien Dugue
2008-09-18 10:42     ` [PATCH HACK] powerpc: quick hack to get a functional eHEA with hardirq preemption, eHCA is close Christoph Raisch
2008-09-18 10:42       ` Christoph Raisch
2008-09-18 12:31       ` Sebastien Dugue
2008-09-18 12:31         ` Sebastien Dugue
2008-09-23 15:43         ` Jan-Bernd Themann
2008-09-23 15:43           ` Jan-Bernd Themann
2008-09-24  9:58           ` [PATCH HACK] powerpc: quick hack to get a functional eHEA with hardirq preemption Milton Miller
2008-09-24  9:58             ` Milton Miller
2008-09-24 10:17             ` Benjamin Herrenschmidt
2008-09-24 10:17               ` Benjamin Herrenschmidt
2008-09-24 11:02               ` Milton Miller
2008-09-24 11:02                 ` Milton Miller
2008-09-24 21:14                 ` Benjamin Herrenschmidt
2008-09-24 21:14                   ` Benjamin Herrenschmidt
2008-09-25  7:31                   ` Sebastien Dugue
2008-09-25  7:31                     ` Sebastien Dugue
2008-09-24 12:35               ` Sebastien Dugue
2008-09-24 12:35                 ` Sebastien Dugue
2008-09-24 21:15                 ` Benjamin Herrenschmidt
2008-09-24 21:15                   ` Benjamin Herrenschmidt
2008-09-25  7:18                   ` Sebastien Dugue
2008-09-25  7:18                     ` Sebastien Dugue
2008-09-25  7:22                     ` Benjamin Herrenschmidt
2008-09-25  7:22                       ` Benjamin Herrenschmidt
2008-09-25  7:42                       ` Sebastien Dugue
2008-09-25  7:42                         ` Sebastien Dugue
2008-09-25  8:36                         ` Benjamin Herrenschmidt
2008-09-25  8:36                           ` Benjamin Herrenschmidt
2008-09-25  8:39                           ` Sebastien Dugue
2008-09-25  8:39                             ` Sebastien Dugue
2008-09-24 12:30             ` Sebastien Dugue
2008-09-24 12:30               ` Sebastien Dugue
2008-09-24 16:42               ` Milton Miller
2008-09-24 16:42                 ` Milton Miller
2008-09-24 21:16                 ` Benjamin Herrenschmidt
2008-09-24 21:16                   ` Benjamin Herrenschmidt
2008-09-25  3:56                   ` Milton Miller
2008-09-25  3:56                     ` Milton Miller
2008-09-25  8:45                 ` Sebastien Dugue
2008-09-25  8:45                   ` Sebastien Dugue
  -- strict thread matches above, loose matches on Subject: below --
2008-09-25 23:40 Milton Miller
2008-09-25 23:40 ` Milton Miller
2008-09-26  9:16 ` Sebastien Dugue
2008-09-26  9:16   ` Sebastien Dugue

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=20080916115947.GA6995@oksana.dev.rtsoft.ru \
    --to=avorontsov@ru.mvista.com \
    --cc=gilles.carry@ext.bull.net \
    --cc=jean-pierre.dion@bull.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rt-users@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=netdev@vger.kernel.org \
    --cc=osstklei@de.ibm.com \
    --cc=raisch@de.ibm.com \
    --cc=sebastien.dugue@bull.net \
    --cc=themann@de.ibm.com \
    --cc=tinytim@us.ibm.com \
    --cc=tklein@de.ibm.com \
    /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.