diff mbox series

mm: more vma cache removal

Message ID Y0WuE3Riv4iy5Jx8@localhost.localdomain (mailing list archive)
State New
Headers show
Series mm: more vma cache removal | expand

Commit Message

Alexey Dobriyan Oct. 11, 2022, 5:55 p.m. UTC
Fixes: 7964cf8caa4d ("mm: remove vmacache")
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
---

 include/linux/sched.h |    2 --
 1 file changed, 2 deletions(-)

Comments

Liam R. Howlett Oct. 11, 2022, 7:06 p.m. UTC | #1
Acked-by: Liam Howlett <liam.howlett@oracle.com>

* Alexey Dobriyan <adobriyan@gmail.com> [221011 19:55]:
> Fixes: 7964cf8caa4d ("mm: remove vmacache")
> Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
> ---
> 
>  include/linux/sched.h |    2 --
>  1 file changed, 2 deletions(-)
> 
> --- a/include/linux/sched.h
> +++ b/include/linux/sched.h
> @@ -870,8 +870,6 @@ struct task_struct {
>  	struct mm_struct		*mm;
>  	struct mm_struct		*active_mm;
>  
> -	/* Per-thread vma caching: */
> -
>  #ifdef SPLIT_RSS_COUNTING
>  	struct task_rss_stat		rss_stat;
>  #endif
David Hildenbrand Oct. 11, 2022, 7:34 p.m. UTC | #2
On 11.10.22 19:55, Alexey Dobriyan wrote:
> Fixes: 7964cf8caa4d ("mm: remove vmacache")

It's not an actual fix. Such tags can be misleading.

> Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
> ---
> 
>   include/linux/sched.h |    2 --
>   1 file changed, 2 deletions(-)
> 
> --- a/include/linux/sched.h
> +++ b/include/linux/sched.h
> @@ -870,8 +870,6 @@ struct task_struct {
>   	struct mm_struct		*mm;
>   	struct mm_struct		*active_mm;
>   
> -	/* Per-thread vma caching: */
> -
>   #ifdef SPLIT_RSS_COUNTING
>   	struct task_rss_stat		rss_stat;
>   #endif
> 

Reviewed-by: David Hildenbrand <david@redhat.com>
diff mbox series

Patch

--- a/include/linux/sched.h
+++ b/include/linux/sched.h
@@ -870,8 +870,6 @@  struct task_struct {
 	struct mm_struct		*mm;
 	struct mm_struct		*active_mm;
 
-	/* Per-thread vma caching: */
-
 #ifdef SPLIT_RSS_COUNTING
 	struct task_rss_stat		rss_stat;
 #endif