LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: 王擎 <wangqing@vivo.com>
To: Mike Galbraith <efault@gmx.de>
Cc: Thomas Gleixner <tglx@linutronix.de>,
	Peter Zijlstra <peterz@infradead.org>,
	Ingo Molnar <mingo@kernel.org>,
	Frederic Weisbecker <frederic@kernel.org>,
	Michal Hocko <mhocko@kernel.org>,
	Davidlohr Bueso <dave@stgolabs.net>,
	Will Deacon <will@kernel.org>,
	"Paul E. McKenney" <paulmck@kernel.org>,
	Dirk Behme <dirk.behme@de.bosch.com>,
	linux-kernel@vger.kernel.org
Subject: Re:Re: [PATCH,RESEND] softirq: Introduce SOFTIRQ_FORCED_THREADING
Date: Mon, 23 Aug 2021 14:33:59 +0800 (GMT+08:00)	[thread overview]
Message-ID: <AJEACAC1D27qN9TVG8Nujqqw.3.1629700439268.Hmail.wangqing@vivo.com> (raw)
In-Reply-To: <b9b55cad5b7ca120ebec2acbbbd0a7fc1a986ec7.camel@gmx.de>


>On Mon, 2021-08-23 at 11:33 +0800, Wang Qing wrote:
>> At present, whether the softirq is executed when the interrupt exits
>> is controlled by IRQ_FORCED_THREADING. This is unreasonable. It should
>> be split and allowed to take effect separately.
>
>Decades long practice suddenly became "unreasonable"?  I think not.  

"unreasonable" may be my misnomer, but it is really necessary to separate
softirq from IRQ_FORCED_THREADING, which can be effective separately.

>
>Trying to carve out bits and pieces of RT to merge immediately isn't
>likely to make the ongoing merge effort go anyfaster or smoother.

I am not trying to carve out bits and pieces of RT, but I encountered actual
problems in my project. For example, in Android, we will not enable 
IRQ_FORCED_THREADING, Android is not a high real-time requirements, 
but in some scenariossome, RT processes cannot be scheduled in time
and the frame is dropped due to the execution time of softirq is too long,
also some softirq cannot be executed in time in ksoftirqs, and delays occur, 
such as IO.

Therefore, why not give the user a choice to balance the execution of softirq
while not enable IRQ_FORCED_THREADING, so as to meet the inconsistent 
scenes and needs

Thanks.
Qing
>
>	Just my $.02,
>
>	-Mike
>
>



  reply	other threads:[~2021-08-23  6:34 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-23  3:33 Wang Qing
2021-08-23  4:22 ` Mike Galbraith
2021-08-23  6:33   ` 王擎 [this message]
2021-08-23  7:43     ` Mike Galbraith
2021-08-27 22:27 ` Thomas Gleixner
2021-08-28  2:18   ` 王擎
2021-08-28 14:07     ` Thomas Gleixner

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=AJEACAC1D27qN9TVG8Nujqqw.3.1629700439268.Hmail.wangqing@vivo.com \
    --to=wangqing@vivo.com \
    --cc=dave@stgolabs.net \
    --cc=dirk.behme@de.bosch.com \
    --cc=efault@gmx.de \
    --cc=frederic@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mhocko@kernel.org \
    --cc=mingo@kernel.org \
    --cc=paulmck@kernel.org \
    --cc=peterz@infradead.org \
    --cc=tglx@linutronix.de \
    --cc=will@kernel.org \
    --subject='Re:Re: [PATCH,RESEND] softirq: Introduce SOFTIRQ_FORCED_THREADING' \
    /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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).