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


Groups > linux.kernel > #1418947 > unrolled thread

Re: [PATCHv3 0/2] target: make location of /var/targets configurable

Started byLee Duncan <lduncan@suse.com>
First post2016-06-10 02:00 +0200
Last post2016-06-10 06:50 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCHv3 0/2] target: make location of /var/targets configurable Lee Duncan <lduncan@suse.com> - 2016-06-10 02:00 +0200
    Re: [PATCHv3 0/2] target: make location of /var/targets configurable "Nicholas A. Bellinger" <nab@linux-iscsi.org> - 2016-06-10 06:50 +0200

#1418947 — Re: [PATCHv3 0/2] target: make location of /var/targets configurable

FromLee Duncan <lduncan@suse.com>
Date2016-06-10 02:00 +0200
SubjectRe: [PATCHv3 0/2] target: make location of /var/targets configurable
Message-ID<rIi1I-5bB-15@gated-at.bofh.it>
Ping?

We really need to move the target database out of /var/target

On 04/14/2016 06:18 PM, Lee Duncan wrote:
> These patches make the location of "/var/target" configurable,
> though it still defauls to "/var/target".
> 
> This "target database directory" can only be changed
> after the target_core_mod loads but before any
> fabric drivers are loaded, and must be the pathname
> of an existing directory.
> 
> This configuration is accomplished via the configfs
> top-level target attribute "dbroot", i.e. dumping
> out "/sys/kernel/config/target/dbroot" will normally
> return "/var/target". Writing to this attribute
> changes the loation where the kernel looks for the
> target database.
> 
> The first patch creates this configurable value for
> the "dbroot", and the second patch modifies users
> of this directory to use this new attribute.
> 
> Changes from v2:
>  * Add locking around access to target driver list
> 
> Changes from v1:
>  * Only allow changing target DB root before it
>    can be used by others
>  * Validate that new DB root is a valid directory
> 
> Lee Duncan (2):
>   target: make target db location configurable
>   target: use new "dbroot" target attribute
> 
>  drivers/target/target_core_alua.c     |  6 ++--
>  drivers/target/target_core_configfs.c | 62 +++++++++++++++++++++++++++++++++++
>  drivers/target/target_core_internal.h |  6 ++++
>  drivers/target/target_core_pr.c       |  2 +-
>  4 files changed, 72 insertions(+), 4 deletions(-)
> 

-- 
Lee Duncan
SUSE Labs

[toc] | [next] | [standalone]


#1419016

From"Nicholas A. Bellinger" <nab@linux-iscsi.org>
Date2016-06-10 06:50 +0200
Message-ID<rImym-8pE-3@gated-at.bofh.it>
In reply to#1418947
On Thu, 2016-06-09 at 16:51 -0700, Lee Duncan wrote:
> Ping?
> 
> We really need to move the target database out of /var/target
> 

This series has already merged up for v4.7-rc1.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web