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

2018-10-18 Thread Frank Lee
>On Thu, 18 Oct 2018
>Ingo Molnar  wrote:
>* Yangtao Li  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 
On Fri, Oct 19, 2018 at 12:10 AM Ingo Molnar  wrote:
>
>
> * Yangtao Li  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


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

2018-10-18 Thread Frank Lee
>On Thu, 18 Oct 2018
>Ingo Molnar  wrote:
>* Yangtao Li  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 
On Fri, Oct 19, 2018 at 12:10 AM Ingo Molnar  wrote:
>
>
> * Yangtao Li  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


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

2018-10-18 Thread Ingo Molnar


* Yangtao Li  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


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

2018-10-18 Thread Ingo Molnar


* Yangtao Li  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


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

2018-10-18 Thread Frank Lee
This place looks like a small mistake. :-)

-- Yangtao Li

>On Thu, 18 Oct 2018
>Steven Rostedt (VMware)  wrote:
>I don't know if the trivial tree is dead or not, but this would be a
>prime candidate.
>
>Acked-by: Steven Rostedt (VMware) 
>
>-- Steve
>
>
>On Thu, 18 Oct 2018 10:21:33 -0400
>Yangtao Li  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
>>   */

On Thu, Oct 18, 2018 at 10:21 PM Yangtao Li  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
>  */
> --
> 2.17.0
>


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

2018-10-18 Thread Frank Lee
This place looks like a small mistake. :-)

-- Yangtao Li

>On Thu, 18 Oct 2018
>Steven Rostedt (VMware)  wrote:
>I don't know if the trivial tree is dead or not, but this would be a
>prime candidate.
>
>Acked-by: Steven Rostedt (VMware) 
>
>-- Steve
>
>
>On Thu, 18 Oct 2018 10:21:33 -0400
>Yangtao Li  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
>>   */

On Thu, Oct 18, 2018 at 10:21 PM Yangtao Li  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
>  */
> --
> 2.17.0
>


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

2018-10-18 Thread Steven Rostedt


I don't know if the trivial tree is dead or not, but this would be a
prime candidate.

Acked-by: Steven Rostedt (VMware) 

-- Steve


On Thu, 18 Oct 2018 10:21:33 -0400
Yangtao Li  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
>*/



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

2018-10-18 Thread Steven Rostedt


I don't know if the trivial tree is dead or not, but this would be a
prime candidate.

Acked-by: Steven Rostedt (VMware) 

-- Steve


On Thu, 18 Oct 2018 10:21:33 -0400
Yangtao Li  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
>*/



[PATCH] softirq: modify comments about PF_MEMALLOC in __do_softirq

2018-10-18 Thread Yangtao Li
---
 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
 */
-- 
2.17.0



[PATCH] softirq: modify comments about PF_MEMALLOC in __do_softirq

2018-10-18 Thread Yangtao Li
---
 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
 */
-- 
2.17.0