linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] radix-tree: replace <linux/spinlock.h> with <linux/spinlock_types.h>
@ 2017-10-13 13:03 Masahiro Yamada
  2017-11-13  4:42 ` Masahiro Yamada
  0 siblings, 1 reply; 3+ messages in thread
From: Masahiro Yamada @ 2017-10-13 13:03 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Masahiro Yamada, Matthew Wilcox, Jan Kara, linux-kernel,
	Johannes Weiner, Chris Mi

This header contains references to spinlock_t and lockdep_is_held(),
both of which are defined in <linux/spinlock_types.h>

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---

 include/linux/radix-tree.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/radix-tree.h b/include/linux/radix-tree.h
index 567ebb5..6df7fa9 100644
--- a/include/linux/radix-tree.h
+++ b/include/linux/radix-tree.h
@@ -27,7 +27,7 @@
 #include <linux/list.h>
 #include <linux/preempt.h>
 #include <linux/rcupdate.h>
-#include <linux/spinlock.h>
+#include <linux/spinlock_types.h>
 #include <linux/types.h>
 
 /*
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] radix-tree: replace <linux/spinlock.h> with <linux/spinlock_types.h>
  2017-10-13 13:03 [PATCH] radix-tree: replace <linux/spinlock.h> with <linux/spinlock_types.h> Masahiro Yamada
@ 2017-11-13  4:42 ` Masahiro Yamada
  2017-11-13 16:09   ` Matthew Wilcox
  0 siblings, 1 reply; 3+ messages in thread
From: Masahiro Yamada @ 2017-11-13  4:42 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Masahiro Yamada, Matthew Wilcox, Jan Kara,
	Linux Kernel Mailing List, Johannes Weiner, Chris Mi

Hi Andrew,


2017-10-13 22:03 GMT+09:00 Masahiro Yamada <yamada.masahiro@socionext.com>:
> This header contains references to spinlock_t and lockdep_is_held(),
> both of which are defined in <linux/spinlock_types.h>
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>


I have not got any comment on this patch.

Could you take a look at it, please?


> ---
>
>  include/linux/radix-tree.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/radix-tree.h b/include/linux/radix-tree.h
> index 567ebb5..6df7fa9 100644
> --- a/include/linux/radix-tree.h
> +++ b/include/linux/radix-tree.h
> @@ -27,7 +27,7 @@
>  #include <linux/list.h>
>  #include <linux/preempt.h>
>  #include <linux/rcupdate.h>
> -#include <linux/spinlock.h>
> +#include <linux/spinlock_types.h>
>  #include <linux/types.h>
>
>  /*
> --
> 2.7.4
>



-- 
Best Regards
Masahiro Yamada

^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: [PATCH] radix-tree: replace <linux/spinlock.h> with <linux/spinlock_types.h>
  2017-11-13  4:42 ` Masahiro Yamada
@ 2017-11-13 16:09   ` Matthew Wilcox
  0 siblings, 0 replies; 3+ messages in thread
From: Matthew Wilcox @ 2017-11-13 16:09 UTC (permalink / raw)
  To: Masahiro Yamada, Andrew Morton
  Cc: Jan Kara, Linux Kernel Mailing List, Johannes Weiner, Chris Mi

It's a waste of time.  I have the XArray replacement well in hand, so this header isn't staying around long-term.

> -----Original Message-----
> From: Masahiro Yamada [mailto:yamada.masahiro@socionext.com]
> Sent: Sunday, November 12, 2017 11:43 PM
> To: Andrew Morton <akpm@linux-foundation.org>
> Cc: Masahiro Yamada <yamada.masahiro@socionext.com>; Matthew Wilcox
> <mawilcox@microsoft.com>; Jan Kara <jack@suse.cz>; Linux Kernel Mailing List
> <linux-kernel@vger.kernel.org>; Johannes Weiner <hannes@cmpxchg.org>;
> Chris Mi <chrism@mellanox.com>
> Subject: Re: [PATCH] radix-tree: replace <linux/spinlock.h> with
> <linux/spinlock_types.h>
> 
> Hi Andrew,
> 
> 
> 2017-10-13 22:03 GMT+09:00 Masahiro Yamada
> <yamada.masahiro@socionext.com>:
> > This header contains references to spinlock_t and lockdep_is_held(),
> > both of which are defined in <linux/spinlock_types.h>
> >
> > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> 
> 
> I have not got any comment on this patch.
> 
> Could you take a look at it, please?
> 
> 
> > ---
> >
> >  include/linux/radix-tree.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/include/linux/radix-tree.h b/include/linux/radix-tree.h
> > index 567ebb5..6df7fa9 100644
> > --- a/include/linux/radix-tree.h
> > +++ b/include/linux/radix-tree.h
> > @@ -27,7 +27,7 @@
> >  #include <linux/list.h>
> >  #include <linux/preempt.h>
> >  #include <linux/rcupdate.h>
> > -#include <linux/spinlock.h>
> > +#include <linux/spinlock_types.h>
> >  #include <linux/types.h>
> >
> >  /*
> > --
> > 2.7.4
> >
> 
> 
> 
> --
> Best Regards
> Masahiro Yamada

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-11-13 16:09 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-13 13:03 [PATCH] radix-tree: replace <linux/spinlock.h> with <linux/spinlock_types.h> Masahiro Yamada
2017-11-13  4:42 ` Masahiro Yamada
2017-11-13 16:09   ` Matthew Wilcox

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).