public inbox for linux-rdma@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] infiniband: qib: Missing linux/slab.h include
@ 2010-08-05  5:55 David Miller
       [not found] ` <20100804.225552.241946831.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
  0 siblings, 1 reply; 4+ messages in thread
From: David Miller @ 2010-08-05  5:55 UTC (permalink / raw)
  To: rolandd-FYB4Gu1CFyUAvxtiuMwx3w; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA


Signed-off-by: David S. Miller <davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
---

Without this I'm getting build errors with Linus's current tree.
Apologies if this is already fixed in the RDMA git tree.

diff --git a/drivers/infiniband/hw/qib/qib.h b/drivers/infiniband/hw/qib/qib.h
index 3593983..312ca9a 100644
--- a/drivers/infiniband/hw/qib/qib.h
+++ b/drivers/infiniband/hw/qib/qib.h
@@ -45,6 +45,7 @@
 #include <linux/mutex.h>
 #include <linux/list.h>
 #include <linux/scatterlist.h>
+#include <linux/slab.h>
 #include <linux/io.h>
 #include <linux/fs.h>
 #include <linux/completion.h>
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* Re: [PATCH] infiniband: qib: Missing linux/slab.h include
       [not found] ` <20100804.225552.241946831.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
@ 2010-08-05 16:29   ` Roland Dreier
       [not found]     ` <adafwyt58ft.fsf-BjVyx320WGW9gfZ95n9DRSW4+XlvGpQz@public.gmane.org>
  0 siblings, 1 reply; 4+ messages in thread
From: Roland Dreier @ 2010-08-05 16:29 UTC (permalink / raw)
  To: David Miller
  Cc: rolandd-FYB4Gu1CFyUAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA

  > Without this I'm getting build errors with Linus's current tree.
  > Apologies if this is already fixed in the RDMA git tree.
  > 
  > diff --git a/drivers/infiniband/hw/qib/qib.h b/drivers/infiniband/hw/qib/qib.h
  > index 3593983..312ca9a 100644
  > --- a/drivers/infiniband/hw/qib/qib.h
  > +++ b/drivers/infiniband/hw/qib/qib.h
  > @@ -45,6 +45,7 @@
  >  #include <linux/mutex.h>
  >  #include <linux/list.h>
  >  #include <linux/scatterlist.h>
  > +#include <linux/slab.h>

No objection to adding this and I will include it for my next pull.  But
out of curiousity what arch are you seeing this on?  I'm not seeing any
problems on x86 and I haven't heard any complaints about -next so I'm
curious what the error you're seeing is.
-- 
Roland Dreier <rolandd-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org> || For corporate legal information go to:
http://www.cisco.com/web/about/doing_business/legal/cri/index.html
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* Re: [PATCH] infiniband: qib: Missing linux/slab.h include
       [not found]     ` <adafwyt58ft.fsf-BjVyx320WGW9gfZ95n9DRSW4+XlvGpQz@public.gmane.org>
@ 2010-08-05 19:09       ` David Miller
       [not found]         ` <20100805.120946.45921714.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
  0 siblings, 1 reply; 4+ messages in thread
From: David Miller @ 2010-08-05 19:09 UTC (permalink / raw)
  To: rdreier-FYB4Gu1CFyUAvxtiuMwx3w
  Cc: rolandd-FYB4Gu1CFyUAvxtiuMwx3w, linux-rdma-u79uwXL29TY76Z2rM5mHXA

From: Roland Dreier <rdreier-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org>
Date: Thu, 05 Aug 2010 09:29:58 -0700

> No objection to adding this and I will include it for my next pull.  But
> out of curiousity what arch are you seeing this on?  I'm not seeing any
> problems on x86 and I haven't heard any complaints about -next so I'm
> curious what the error you're seeing is.

sparc64 allmodconfig
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* Re: [PATCH] infiniband: qib: Missing linux/slab.h include
       [not found]         ` <20100805.120946.45921714.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
@ 2010-08-05 21:19           ` Roland Dreier
  0 siblings, 0 replies; 4+ messages in thread
From: Roland Dreier @ 2010-08-05 21:19 UTC (permalink / raw)
  To: David Miller; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA

 > > No objection to adding this and I will include it for my next pull.  But
 > > out of curiousity what arch are you seeing this on?  I'm not seeing any
 > > problems on x86 and I haven't heard any complaints about -next so I'm
 > > curious what the error you're seeing is.

 > sparc64 allmodconfig

Got it.  Interestingly x86, powerpc and ia64 all include linux/slab.h
via asm/pci.h (for no very obvious reason that I can see from a quick
look at what their asm/pci.h have), which is why I missed this.

anyway, applied your patch and will pass on to Linus shortly.
-- 
Roland Dreier <rolandd-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org> || For corporate legal information go to:
http://www.cisco.com/web/about/doing_business/legal/cri/index.html
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

end of thread, other threads:[~2010-08-05 21:19 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-08-05  5:55 [PATCH] infiniband: qib: Missing linux/slab.h include David Miller
     [not found] ` <20100804.225552.241946831.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
2010-08-05 16:29   ` Roland Dreier
     [not found]     ` <adafwyt58ft.fsf-BjVyx320WGW9gfZ95n9DRSW4+XlvGpQz@public.gmane.org>
2010-08-05 19:09       ` David Miller
     [not found]         ` <20100805.120946.45921714.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
2010-08-05 21:19           ` Roland Dreier

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox