2011-05-23 21:11:20

by Nicholas A. Bellinger

Subject: [PATCH-v2] MAINTAINERS: Add drivers/target/ entry

From: Nicholas Bellinger <[email protected]>

Signed-off-by: Nicholas A. Bellinger <[email protected]>
---
MAINTAINERS | 11 +++++++++++
1 files changed, 11 insertions(+), 0 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 20be668..16e9861 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -6032,6 +6032,17 @@ F: Documentation/filesystems/sysv-fs.txt
F: fs/sysv/
F: include/linux/sysv_fs.h

+TARGET SUBSYSTEM
+M: Nicholas A. Bellinger <[email protected]>
+L: [email protected]
+L: http://groups.google.com/group/linux-iscsi-target-dev
+W: http://www.linux-iscsi.org
+T: git git://git.kernel.org/pub/scm/linux/kernel/git/nab/lio-core-2.6.git master
+S: Supported
+F: drivers/target/
+F: include/target/
+F: Documentation/target/
+
TASKSTATS STATISTICS INTERFACE
M: Balbir Singh <[email protected]>
S: Maintained
--
1.7.5.2

2011-05-23 21:14:09

by Randy Dunlap

Subject: Re: [PATCH-v2] MAINTAINERS: Add drivers/target/ entry

On 05/23/11 14:10, Nicholas A. Bellinger wrote:
> From: Nicholas Bellinger <[email protected]>
>
> Signed-off-by: Nicholas A. Bellinger <[email protected]>

Acked-by: Randy Dunlap <[email protected]>

Thanks.

> ---
> MAINTAINERS | 11 +++++++++++
> 1 files changed, 11 insertions(+), 0 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 20be668..16e9861 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -6032,6 +6032,17 @@ F: Documentation/filesystems/sysv-fs.txt
> F: fs/sysv/
> F: include/linux/sysv_fs.h
>
> +TARGET SUBSYSTEM
> +M: Nicholas A. Bellinger <[email protected]>
> +L: [email protected]
> +L: http://groups.google.com/group/linux-iscsi-target-dev
> +W: http://www.linux-iscsi.org
> +T: git git://git.kernel.org/pub/scm/linux/kernel/git/nab/lio-core-2.6.git master
> +S: Supported
> +F: drivers/target/
> +F: include/target/
> +F: Documentation/target/
> +
> TASKSTATS STATISTICS INTERFACE
> M: Balbir Singh <[email protected]>
> S: Maintained


--
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

2011-05-23 23:36:34

by Joe Perches

Subject: Re: [PATCH-v2] MAINTAINERS: Add drivers/target/ entry

On Mon, 2011-05-23 at 14:10 -0700, Nicholas A. Bellinger wrote:
> From: Nicholas Bellinger <[email protected]>
> +TARGET SUBSYSTEM
> +M: Nicholas A. Bellinger <[email protected]>
> +L: [email protected]
> +L: http://groups.google.com/group/linux-iscsi-target-dev
> +W: http://www.linux-iscsi.org
> +T: git git://git.kernel.org/pub/scm/linux/kernel/git/nab/lio-core-2.6.git master
> +S: Supported
> +F: drivers/target/
> +F: include/target/
> +F: Documentation/target/

Shouldn't the file layout be more like

drivers/lio/target?

2011-05-24 06:19:28

by Nicholas A. Bellinger

Subject: Re: [PATCH-v2] MAINTAINERS: Add drivers/target/ entry

On Mon, 2011-05-23 at 16:36 -0700, Joe Perches wrote:
> On Mon, 2011-05-23 at 14:10 -0700, Nicholas A. Bellinger wrote:
> > From: Nicholas Bellinger <[email protected]>
> > +TARGET SUBSYSTEM
> > +M: Nicholas A. Bellinger <[email protected]>
> > +L: [email protected]
> > +L: http://groups.google.com/group/linux-iscsi-target-dev
> > +W: http://www.linux-iscsi.org
> > +T: git git://git.kernel.org/pub/scm/linux/kernel/git/nab/lio-core-2.6.git master
> > +S: Supported
> > +F: drivers/target/
> > +F: include/target/
> > +F: Documentation/target/
>
> Shouldn't the file layout be more like
>
> drivers/lio/target?
>

Hi Joe,

Not sure what you mean here.. Currently drivers/target/ contains the
fabric independent target core, and /drivers/target/$FABRIC/ contains
individual fabric module drivers.

I originally decided to use drivers/target/ to avoid any confusion wrt
to the 'LIO' term, which to some folks is interchangable wrt to the
'iscsi-target' fabric module.

--nab

2011-05-24 06:43:43

by Joe Perches

Subject: Re: [PATCH-v2] MAINTAINERS: Add drivers/target/ entry

On Mon, 2011-05-23 at 23:11 -0700, Nicholas A. Bellinger wrote:
> On Mon, 2011-05-23 at 16:36 -0700, Joe Perches wrote:
> > On Mon, 2011-05-23 at 14:10 -0700, Nicholas A. Bellinger wrote:
> > > From: Nicholas Bellinger <[email protected]>
> > > +TARGET SUBSYSTEM
> > > +M: Nicholas A. Bellinger <[email protected]>
> > > +L: [email protected]
> > > +L: http://groups.google.com/group/linux-iscsi-target-dev
> > > +W: http://www.linux-iscsi.org
> > > +T: git git://git.kernel.org/pub/scm/linux/kernel/git/nab/lio-core-2.6.git master
> > > +S: Supported
> > > +F: drivers/target/
> > > +F: include/target/
> > > +F: Documentation/target/
> >
> > Shouldn't the file layout be more like
> > drivers/lio/target?
> Hi Joe,

Hi Nicholas.

> Not sure what you mean here.. Currently drivers/target/ contains the
> fabric independent target core, and /drivers/target/$FABRIC/ contains
> individual fabric module drivers.
>
> I originally decided to use drivers/target/ to avoid any confusion wrt
> to the 'LIO' term, which to some folks is interchangable wrt to the
> 'iscsi-target' fabric module.

Target doesn't mean much to me and the wiki calls the
system LIO.

http://linux-iscsi.org/wiki/Main_Page

I think that drivers/target might be better renamed to
drivers/lio/target and drivers/lio/target/$FABRIC could
be below that.

No worries if you think otherwise.

cheers, Joe