Re: [PATCH] softirq: modify comments about PF_MEMALLOC in __do_softirq

From: Frank Lee
Date: Thu Oct 18 2018 - 12:20:59 EST


>On Thu, 18 Oct 2018
>Ingo Molnar <mingo@xxxxxxxxxx> wrote:
>* Yangtao Li <tiny.windzz@xxxxxxxxx> wrote:
>
>> ---
>> kernel/softirq.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/kernel/softirq.c b/kernel/softirq.c
>> index 6f584861d329..6193e1d1b30d 100644
>> --- a/kernel/softirq.c
>> +++ b/kernel/softirq.c
>> @@ -257,7 +257,7 @@ asmlinkage __visible void __softirq_entry __do_softirq(void)
>> int softirq_bit;
>>
>> /*
>> - * Mask out PF_MEMALLOC s current task context is borrowed for the
>> + * Mask out PF_MEMALLOC as current task context is borrowed for the
>> * softirq. A softirq handled such as network RX might set PF_MEMALLOC
>> * again if the socket is related to swap
>> */
>
>Can I add your Signed-off-by?
>
>Thanks,
>
> Ingo

yeahïThanks

-- Yangtao Li

Signed-off-by: Yangtao Li <tiny.windzz@xxxxxxxxx>
On Fri, Oct 19, 2018 at 12:10 AM Ingo Molnar <mingo@xxxxxxxxxx> wrote:
>
>
> * Yangtao Li <tiny.windzz@xxxxxxxxx> wrote:
>
> > ---
> > kernel/softirq.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/kernel/softirq.c b/kernel/softirq.c
> > index 6f584861d329..6193e1d1b30d 100644
> > --- a/kernel/softirq.c
> > +++ b/kernel/softirq.c
> > @@ -257,7 +257,7 @@ asmlinkage __visible void __softirq_entry __do_softirq(void)
> > int softirq_bit;
> >
> > /*
> > - * Mask out PF_MEMALLOC s current task context is borrowed for the
> > + * Mask out PF_MEMALLOC as current task context is borrowed for the
> > * softirq. A softirq handled such as network RX might set PF_MEMALLOC
> > * again if the socket is related to swap
> > */
>
> Can I add your Signed-off-by?
>
> Thanks,
>
> Ingo