Re: [PATCH] nvme-loop: add configfs dependency

2016-07-12 Thread Jens Axboe
On 07/10/2016 05:14 AM, Christoph Hellwig wrote: On Thu, Jul 07, 2016 at 08:35:17AM -0600, Jens Axboe wrote: Thanks Arnd, applied. Actually I think we should replace the select with the depends. In fact I though I had done that a while ago, but I must have messed it up. Btw - do you plan to

Re: [PATCH] nvme-loop: add configfs dependency

2016-07-12 Thread Jens Axboe
On 07/10/2016 05:14 AM, Christoph Hellwig wrote: On Thu, Jul 07, 2016 at 08:35:17AM -0600, Jens Axboe wrote: Thanks Arnd, applied. Actually I think we should replace the select with the depends. In fact I though I had done that a while ago, but I must have messed it up. Btw - do you plan to

Re: [PATCH] nvme-loop: add configfs dependency

2016-07-12 Thread Jens Axboe
On 07/11/2016 01:29 PM, Arnd Bergmann wrote: On Sunday, July 10, 2016 2:14:17 PM CEST Christoph Hellwig wrote: On Thu, Jul 07, 2016 at 08:35:17AM -0600, Jens Axboe wrote: Thanks Arnd, applied. Actually I think we should replace the select with the depends. In fact I though I had done that a

Re: [PATCH] nvme-loop: add configfs dependency

2016-07-12 Thread Jens Axboe
On 07/11/2016 01:29 PM, Arnd Bergmann wrote: On Sunday, July 10, 2016 2:14:17 PM CEST Christoph Hellwig wrote: On Thu, Jul 07, 2016 at 08:35:17AM -0600, Jens Axboe wrote: Thanks Arnd, applied. Actually I think we should replace the select with the depends. In fact I though I had done that a

Re: [PATCH] nvme-loop: add configfs dependency

2016-07-11 Thread Arnd Bergmann
On Sunday, July 10, 2016 2:14:17 PM CEST Christoph Hellwig wrote: > On Thu, Jul 07, 2016 at 08:35:17AM -0600, Jens Axboe wrote: > > Thanks Arnd, applied. > > Actually I think we should replace the select with the depends. In > fact I though I had done that a while ago, but I must have messed it

Re: [PATCH] nvme-loop: add configfs dependency

2016-07-11 Thread Arnd Bergmann
On Sunday, July 10, 2016 2:14:17 PM CEST Christoph Hellwig wrote: > On Thu, Jul 07, 2016 at 08:35:17AM -0600, Jens Axboe wrote: > > Thanks Arnd, applied. > > Actually I think we should replace the select with the depends. In > fact I though I had done that a while ago, but I must have messed it

Re: [PATCH] nvme-loop: add configfs dependency

2016-07-11 Thread J Freyensee
On Sun, 2016-07-10 at 14:14 +0200, Christoph Hellwig wrote: > On Thu, Jul 07, 2016 at 08:35:17AM -0600, Jens Axboe wrote: > > Thanks Arnd, applied. > > Actually I think we should replace the select with the depends. In > fact I though I had done that a while ago, but I must have messed it > up.

Re: [PATCH] nvme-loop: add configfs dependency

2016-07-11 Thread J Freyensee
On Sun, 2016-07-10 at 14:14 +0200, Christoph Hellwig wrote: > On Thu, Jul 07, 2016 at 08:35:17AM -0600, Jens Axboe wrote: > > Thanks Arnd, applied. > > Actually I think we should replace the select with the depends. In > fact I though I had done that a while ago, but I must have messed it > up.

Re: [PATCH] nvme-loop: add configfs dependency

2016-07-10 Thread Christoph Hellwig
On Thu, Jul 07, 2016 at 08:35:17AM -0600, Jens Axboe wrote: > Thanks Arnd, applied. Actually I think we should replace the select with the depends. In fact I though I had done that a while ago, but I must have messed it up. Btw - do you plan to grab patches directly from the list now or do you

Re: [PATCH] nvme-loop: add configfs dependency

2016-07-10 Thread Christoph Hellwig
On Thu, Jul 07, 2016 at 08:35:17AM -0600, Jens Axboe wrote: > Thanks Arnd, applied. Actually I think we should replace the select with the depends. In fact I though I had done that a while ago, but I must have messed it up. Btw - do you plan to grab patches directly from the list now or do you

Re: [PATCH] nvme-loop: add configfs dependency

2016-07-07 Thread Jens Axboe
On 07/07/2016 03:01 AM, Arnd Bergmann wrote: CONFIG_NVME_TARGET has a correct CONFIG_CONFIGFS_FS dependency, but the newly added NVME_TARGET_LOOP is missing this, resulting in a link failure: drivers/nvme/built-in.o: In function `nvmet_init_configfs': loop.c:(.init.text+0x2a0): undefined

Re: [PATCH] nvme-loop: add configfs dependency

2016-07-07 Thread Jens Axboe
On 07/07/2016 03:01 AM, Arnd Bergmann wrote: CONFIG_NVME_TARGET has a correct CONFIG_CONFIGFS_FS dependency, but the newly added NVME_TARGET_LOOP is missing this, resulting in a link failure: drivers/nvme/built-in.o: In function `nvmet_init_configfs': loop.c:(.init.text+0x2a0): undefined

[PATCH] nvme-loop: add configfs dependency

2016-07-07 Thread Arnd Bergmann
CONFIG_NVME_TARGET has a correct CONFIG_CONFIGFS_FS dependency, but the newly added NVME_TARGET_LOOP is missing this, resulting in a link failure: drivers/nvme/built-in.o: In function `nvmet_init_configfs': loop.c:(.init.text+0x2a0): undefined reference to `config_group_init'

[PATCH] nvme-loop: add configfs dependency

2016-07-07 Thread Arnd Bergmann
CONFIG_NVME_TARGET has a correct CONFIG_CONFIGFS_FS dependency, but the newly added NVME_TARGET_LOOP is missing this, resulting in a link failure: drivers/nvme/built-in.o: In function `nvmet_init_configfs': loop.c:(.init.text+0x2a0): undefined reference to `config_group_init'