mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "emin ak" <eminak71@gmail.com>
To: "Ingo Molnar" <mingo@elte.hu>
Cc: "Kumar Gala" <galak@kernel.crashing.org>,
	"Nick Piggin" <nickpiggin@yahoo.com.au>,
	linux-kernel@vger.kernel.org
Subject: Re: 2.6.16-rt10 crash on ppc
Date: Thu, 30 Mar 2006 13:25:35 +0300	[thread overview]
Message-ID: <2cf1ee820603300225x3c5a6f98qe8bdda1023f0d74f@mail.gmail.com> (raw)
In-Reply-To: <20060330072339.GB3941@elte.hu>

Soryy fot the late answer. I have change the driver to use
SA_INTERRUPT, OOM messages disappeared but the ethernet
throughput decreased significantly as Kumar said before(90MBit for
1518Byte packet, and throughput without the patch is
about 900Bits) and the system can't manage load
balancing between to interfaces,the system acts like
the priorty of tsec0 (eth0) is higher then tsec1 because of
this, under heavy load on tsec0 blocks packet receiving
on tsec1 (eth1). And also I have tried Nick solution, increased
/proc/sys/vm/min_free_kbytes to 10MB, the result did'nt changed, OOM
messages was repeated again.
I'll try Ingo patch immediately and and report you the results.
Thanks.
Emin


2006/3/30, Ingo Molnar <mingo@elte.hu>:
>
> * Kumar Gala <galak@kernel.crashing.org> wrote:
>
> > The issue me actually be a driver interrupt locking bug.  The driver
> > supports three distinct interrupts for TX, RX, Error.  I asked Emin to
> > try changing the driver to use SA_INTERRUPT in the request_irq() to
> > see what happens.  I believe that when he did that it worked but hurts
> > performance.
>
> this is the -rt kernel, with PREEMPT_RT enabled, so SA_INTERRUPT should
> make no difference. All interrupts are executed by their interrupt
> thread, and are fully preemptible:
>
> fastcall int handle_IRQ_event(unsigned int irq, struct pt_regs *regs,
>                                struct irqaction *action)
> {
>        int ret, retval = 0, status = 0;
>
>        /*
>         * Unconditionally enable interrupts for threaded
>         * IRQ handlers:
>         */
>        if (!hardirq_count() || !(action->flags & SA_INTERRUPT))
>                local_irq_enable();
>
> i.e. SA_INTERRUPT should have no effect.
>
>        Ingo
>

  reply	other threads:[~2006-03-30 10:25 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-27 14:56 emin ak
2006-03-27 17:11 ` emin ak
2006-03-27 20:31 ` emin ak
2006-03-28  1:21   ` Nick Piggin
2006-03-29 15:08     ` Ingo Molnar
2006-03-30  2:55       ` Nick Piggin
2006-03-30  7:13         ` Ingo Molnar
2006-03-30  7:18           ` Kumar Gala
2006-03-30  7:23             ` Ingo Molnar
2006-03-30 10:25               ` emin ak [this message]
2006-03-30 11:34                 ` Ingo Molnar
2006-03-30 14:55                   ` emin ak

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=2cf1ee820603300225x3c5a6f98qe8bdda1023f0d74f@mail.gmail.com \
    --to=eminak71@gmail.com \
    --cc=galak@kernel.crashing.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=nickpiggin@yahoo.com.au \
    /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

all inboxes | Powered by JetHome®