Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1216834 > unrolled thread

[PATCH] staging: ipath: fix build failure

Started bySudip Mukherjee <sudipm.mukherjee@gmail.com>
First post2015-09-01 15:10 +0200
Last post2015-09-01 19:00 +0200
Articles 4 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] staging: ipath: fix build failure Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-09-01 15:10 +0200
    Re: [PATCH] staging: ipath: fix build failure Doug Ledford <dledford@redhat.com> - 2015-09-01 17:00 +0200
      Re: [PATCH] staging: ipath: fix build failure Jim Davis <jim.epost@gmail.com> - 2015-09-01 19:00 +0200
        Re: [PATCH] staging: ipath: fix build failure Doug Ledford <dledford@redhat.com> - 2015-09-01 19:00 +0200

#1216834 — [PATCH] staging: ipath: fix build failure

FromSudip Mukherjee <sudipm.mukherjee@gmail.com>
Date2015-09-01 15:10 +0200
Subject[PATCH] staging: ipath: fix build failure
Message-ID<q3Tu4-7qR-35@gated-at.bofh.it>
building of ipath depends on infiniband. And if ipath is selected and
infiniband is not then build fails with:
ERROR: "ib_alloc_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
ERROR: "ib_dealloc_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
ERROR: "ib_dispatch_event" [drivers/staging/ipath/ib_ipath.ko] undefined!
ERROR: "ib_register_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
ERROR: "ib_unregister_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
ERROR: "ib_modify_qp_is_ok" [drivers/staging/ipath/ib_ipath.ko] undefined!
ERROR: "ib_wq" [drivers/staging/ipath/ib_ipath.ko] undefined!

Config at: https://lkml.org/lkml/2015/8/31/328

Reported-by: Jim Davis <jim.epost@gmail.com>
Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
---
 drivers/staging/ipath/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/staging/ipath/Kconfig b/drivers/staging/ipath/Kconfig
index 041ce06..1edc21b 100644
--- a/drivers/staging/ipath/Kconfig
+++ b/drivers/staging/ipath/Kconfig
@@ -1,6 +1,7 @@
 config INFINIBAND_IPATH
 	tristate "QLogic HTX HCA support"
 	depends on 64BIT && NET && HT_IRQ
+	depends on INFINIBAND
 	---help---
 	This is a driver for the deprecated QLogic Hyper-Transport
 	IB host channel adapter (model QHT7140),
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1216899

FromDoug Ledford <dledford@redhat.com>
Date2015-09-01 17:00 +0200
Message-ID<q3Vcu-1fz-17@gated-at.bofh.it>
In reply to#1216834

[Multipart message — attachments visible in raw view] — view raw

On 09/01/2015 09:01 AM, Sudip Mukherjee wrote:
> building of ipath depends on infiniband. And if ipath is selected and
> infiniband is not then build fails with:
> ERROR: "ib_alloc_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
> ERROR: "ib_dealloc_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
> ERROR: "ib_dispatch_event" [drivers/staging/ipath/ib_ipath.ko] undefined!
> ERROR: "ib_register_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
> ERROR: "ib_unregister_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
> ERROR: "ib_modify_qp_is_ok" [drivers/staging/ipath/ib_ipath.ko] undefined!
> ERROR: "ib_wq" [drivers/staging/ipath/ib_ipath.ko] undefined!
> 
> Config at: https://lkml.org/lkml/2015/8/31/328
> 
> Reported-by: Jim Davis <jim.epost@gmail.com>
> Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
> ---
>  drivers/staging/ipath/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/staging/ipath/Kconfig b/drivers/staging/ipath/Kconfig
> index 041ce06..1edc21b 100644
> --- a/drivers/staging/ipath/Kconfig
> +++ b/drivers/staging/ipath/Kconfig
> @@ -1,6 +1,7 @@
>  config INFINIBAND_IPATH
>  	tristate "QLogic HTX HCA support"
>  	depends on 64BIT && NET && HT_IRQ
> +	depends on INFINIBAND
>  	---help---
>  	This is a driver for the deprecated QLogic Hyper-Transport
>  	IB host channel adapter (model QHT7140),
> 

The ipath driver was moved to staging/rdma/ipath and has proper
protection against being built without the InfiniBand subsystem.  Where
are you seeing this tree?  I'm curious because I no longer have this
driver in this location in my tree and I never pushed it anywhere other
than one of my trees.  Is this maybe in linux-next?

-- 
Doug Ledford <dledford@redhat.com>
              GPG KeyID: 0E572FDD


[toc] | [prev] | [next] | [standalone]


#1216967

FromJim Davis <jim.epost@gmail.com>
Date2015-09-01 19:00 +0200
Message-ID<q3X4C-3Wm-13@gated-at.bofh.it>
In reply to#1216899
On Tue, Sep 1, 2015 at 7:52 AM, Doug Ledford <dledford@redhat.com> wrote:
> On 09/01/2015 09:01 AM, Sudip Mukherjee wrote:
>> building of ipath depends on infiniband. And if ipath is selected and
>> infiniband is not then build fails with:
>> ERROR: "ib_alloc_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
>> ERROR: "ib_dealloc_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
>> ERROR: "ib_dispatch_event" [drivers/staging/ipath/ib_ipath.ko] undefined!
>> ERROR: "ib_register_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
>> ERROR: "ib_unregister_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
>> ERROR: "ib_modify_qp_is_ok" [drivers/staging/ipath/ib_ipath.ko] undefined!
>> ERROR: "ib_wq" [drivers/staging/ipath/ib_ipath.ko] undefined!
>>
>> Config at: https://lkml.org/lkml/2015/8/31/328
>>
>> Reported-by: Jim Davis <jim.epost@gmail.com>
>> Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
>> ---
>>  drivers/staging/ipath/Kconfig | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/staging/ipath/Kconfig b/drivers/staging/ipath/Kconfig
>> index 041ce06..1edc21b 100644
>> --- a/drivers/staging/ipath/Kconfig
>> +++ b/drivers/staging/ipath/Kconfig
>> @@ -1,6 +1,7 @@
>>  config INFINIBAND_IPATH
>>       tristate "QLogic HTX HCA support"
>>       depends on 64BIT && NET && HT_IRQ
>> +     depends on INFINIBAND
>>       ---help---
>>       This is a driver for the deprecated QLogic Hyper-Transport
>>       IB host channel adapter (model QHT7140),
>>
>
> The ipath driver was moved to staging/rdma/ipath and has proper
> protection against being built without the InfiniBand subsystem.  Where
> are you seeing this tree?  I'm curious because I no longer have this
> driver in this location in my tree and I never pushed it anywhere other
> than one of my trees.  Is this maybe in linux-next?

Yes; http://marc.info/?l=linux-kernel&m=144104080331664&w=3
-- 
Jim
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1216968

FromDoug Ledford <dledford@redhat.com>
Date2015-09-01 19:00 +0200
Message-ID<q3X4C-3Wm-23@gated-at.bofh.it>
In reply to#1216967

[Multipart message — attachments visible in raw view] — view raw

On 09/01/2015 12:54 PM, Jim Davis wrote:
> On Tue, Sep 1, 2015 at 7:52 AM, Doug Ledford <dledford@redhat.com> wrote:
>> On 09/01/2015 09:01 AM, Sudip Mukherjee wrote:
>>> building of ipath depends on infiniband. And if ipath is selected and
>>> infiniband is not then build fails with:
>>> ERROR: "ib_alloc_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
>>> ERROR: "ib_dealloc_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
>>> ERROR: "ib_dispatch_event" [drivers/staging/ipath/ib_ipath.ko] undefined!
>>> ERROR: "ib_register_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
>>> ERROR: "ib_unregister_device" [drivers/staging/ipath/ib_ipath.ko] undefined!
>>> ERROR: "ib_modify_qp_is_ok" [drivers/staging/ipath/ib_ipath.ko] undefined!
>>> ERROR: "ib_wq" [drivers/staging/ipath/ib_ipath.ko] undefined!
>>>
>>> Config at: https://lkml.org/lkml/2015/8/31/328
>>>
>>> Reported-by: Jim Davis <jim.epost@gmail.com>
>>> Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
>>> ---
>>>  drivers/staging/ipath/Kconfig | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/drivers/staging/ipath/Kconfig b/drivers/staging/ipath/Kconfig
>>> index 041ce06..1edc21b 100644
>>> --- a/drivers/staging/ipath/Kconfig
>>> +++ b/drivers/staging/ipath/Kconfig
>>> @@ -1,6 +1,7 @@
>>>  config INFINIBAND_IPATH
>>>       tristate "QLogic HTX HCA support"
>>>       depends on 64BIT && NET && HT_IRQ
>>> +     depends on INFINIBAND
>>>       ---help---
>>>       This is a driver for the deprecated QLogic Hyper-Transport
>>>       IB host channel adapter (model QHT7140),
>>>
>>
>> The ipath driver was moved to staging/rdma/ipath and has proper
>> protection against being built without the InfiniBand subsystem.  Where
>> are you seeing this tree?  I'm curious because I no longer have this
>> driver in this location in my tree and I never pushed it anywhere other
>> than one of my trees.  Is this maybe in linux-next?
> 
> Yes; http://marc.info/?l=linux-kernel&m=144104080331664&w=3
> 

OK.  The current setup has this in drivers/staging/rdma/Kconfig:

menuconfig STAGING_RDMA
        bool "RDMA staging drivers"
	depends on INFINIBAND
	depends on PCI || BROKEN
	depends on HAS_IOMEM
	depends on NET
	depends on INET
        default n
        ---help---
          This option allows you to select a number of RDMA drivers that
	  fall into one of two categories: deprecated drivers being held
	  here before finally being removed or new drivers that still need
	  some work before being moved to the normal RDMA driver area.

          If you wish to work on these drivers, to help improve them, or
          to report problems you have with them, please use the
	  linux-rdma@vger.kernel.org mailing list.

          If in doubt, say N here.


# Please keep entries in alphabetic order
if STAGING_RDMA

source "drivers/staging/rdma/amso1100/Kconfig"

source "drivers/staging/rdma/hfi1/Kconfig"

source "drivers/staging/rdma/ipath/Kconfig"

endif


As you can see, it has proper depends for all of the drivers, so no need
to add dependencies piecemeal any longer.

-- 
Doug Ledford <dledford@redhat.com>
              GPG KeyID: 0E572FDD


[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web