Re: [PATCH 0/2] 9p: Fixes for hard-to-hit bugs

2017-09-07 Thread Latchesar Ionkov
Acked-by: Latchesar Ionkov <lu...@ionkov.net> On Wed, Sep 6, 2017 at 8:59 AM, Tuomas Tynkkynen <tuo...@tuxera.com> wrote: > These two patches fix two hard-to-hit (but really annoying) bugs in 9p. > The first one was posted earlier in February (with one R-b), the second > i

Re: [PATCH 0/2] 9p: Fixes for hard-to-hit bugs

2017-09-07 Thread Latchesar Ionkov
Acked-by: Latchesar Ionkov On Wed, Sep 6, 2017 at 8:59 AM, Tuomas Tynkkynen wrote: > These two patches fix two hard-to-hit (but really annoying) bugs in 9p. > The first one was posted earlier in February (with one R-b), the second > is a new one. > > Both of these have had s

Re: [PATCH] fs/9p: Compare qid.path in v9fs_test_inode

2017-03-09 Thread Latchesar Ionkov
Reviewed-by: Latchesar Ionkov <lu...@ionkov.net> On Tue, Feb 21, 2017 at 6:06 PM, Tuomas Tynkkynen <tuo...@tuxera.com> wrote: > Commit fd2421f54423 ("fs/9p: When doing inode lookup compare qid details > and inode mode bits.") transformed v9fs_qid_iget() to

Re: [PATCH] fs/9p: Compare qid.path in v9fs_test_inode

2017-03-09 Thread Latchesar Ionkov
Reviewed-by: Latchesar Ionkov On Tue, Feb 21, 2017 at 6:06 PM, Tuomas Tynkkynen wrote: > Commit fd2421f54423 ("fs/9p: When doing inode lookup compare qid details > and inode mode bits.") transformed v9fs_qid_iget() to use iget5_locked() > instead of iget_locked(). However,

Re: [PATCH 4/5] 9p: introduce async read requests

2016-12-13 Thread Latchesar Ionkov
If the user asked for more than msize-iohdrsz (or rather iounit) bytes, you should do your best to read as much as possible before you return to user space. That means that if a read returns the maximum possible count you HAVE to issue another read until you get a short read. What Al is hinting

Re: [PATCH 4/5] 9p: introduce async read requests

2016-12-13 Thread Latchesar Ionkov
If the user asked for more than msize-iohdrsz (or rather iounit) bytes, you should do your best to read as much as possible before you return to user space. That means that if a read returns the maximum possible count you HAVE to issue another read until you get a short read. What Al is hinting

Re: [PATCH] 9p: basic sysfs support

2007-11-02 Thread Latchesar Ionkov
On 11/2/07, Greg KH <[EMAIL PROTECTED]> wrote: > On Fri, Nov 02, 2007 at 10:53:31AM -0600, Latchesar Ionkov wrote: > > This patch implements the basic sysfs support for 9p. If CONFIG_NET_9P_DEBUG > > is defined, allows reading and modifying the debug level via > > /sysfs/

[PATCH 1/10] 9p: new transport interface

2007-11-02 Thread Latchesar Ionkov
interface making it more suitable for implementing various 9P transports. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- fs/9p/v9fs.c |1 - include/net/9p/9p.h|1 + include/net/9p/client.h| 13 ++- include/net/9p/conn.h | 57 include/

[PATCH] 9p: basic sysfs support

2007-11-02 Thread Latchesar Ionkov
This patch implements the basic sysfs support for 9p. If CONFIG_NET_9P_DEBUG is defined, allows reading and modifying the debug level via /sysfs/fs/9p/debuglevel. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- include/net/9p/9p.h |1 + net/9p/mod.c

[PATCH 2/10] 9p: fcall deserialization changes

2007-11-02 Thread Latchesar Ionkov
Modify p9_deserialize_fcall to read from p9_fcall's internal buffer instead of specifying an external one. Add functions that copy raw data to and from a fcall. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- include/net/9p/9p.h |4 +++- net/9p/conv.c

[PATCH][REFERENCE ONLY] 9p: ramfs 9p server

2007-11-02 Thread Latchesar Ionkov
Sample ramfs file server that uses the in-kernel 9P file server support. This code is for reference only. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- net/9p/Kconfig |8 +- net/9p/Makefile |1 + net/9p/ramfs/ramfs.c

[PATCH 9/10] 9p: tcp listener implementation

2007-11-02 Thread Latchesar Ionkov
This patch adds a TCP listener for the trans_fd transport. The listener allows the in-kernel servers to listen on a TCP port for client connections. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- net/9p/mux.c | 269 +- 1

[PATCH 8/10] 9p: loopback transport

2007-11-02 Thread Latchesar Ionkov
9P loopback transport that can be used between 9P in-kernel servers and v9fs on the same machine. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- net/9p/Kconfig |7 + net/9p/Makefile |4 + net/9p/trans_loop.c | 371

[PATCH 7/10] 9p: sysfs support for in-kenel servers

2007-11-02 Thread Latchesar Ionkov
off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- include/net/9p/srv.h | 24 +++- net/9p/srv.c | 386 +- 2 files changed, 405 insertions(+), 5 deletions(-) diff --git a/include/net/9p/srv.h b/include/net/9p/srv.h index 72d011e..a81ae

[PATCH 6/10] 9p: in-kernel server basic support

2007-11-02 Thread Latchesar Ionkov
This patch implements the basic functionality required for implementing in-kernel 9P file servers. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- include/net/9p/9p.h |2 + include/net/9p/srv.h | 169 +++ net/9p/Kconfig |5 + net/9p/Makefile |4 +

[PATCH 5/10] 9p: marshalling changes for in-kernel server

2007-11-02 Thread Latchesar Ionkov
This patch implements serialization/deserialization for the server-side 9P messages. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- include/net/9p/9p.h | 16 ++ net/9p/conv.c | 457 +-- 2 files changed, 462 insertions(

[PATCH 4/10] 9p: tranport interface changes for in-kernel server

2007-11-02 Thread Latchesar Ionkov
This patch adds in-kernel 9P server support to the transport interface. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- fs/9p/v9fs.c |2 +- include/net/9p/transport.h | 31 --- net/9p/mux.c |6 +++--- 3 files chang

[PATCH 9/10] 9p: tcp listener implementation

2007-11-02 Thread Latchesar Ionkov
This patch adds a TCP listener for the trans_fd transport. The listener allows the in-kernel servers to listen on a TCP port for client connections. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- net/9p/mux.c | 269 +- 1 files

[PATCH 7/10] 9p: sysfs support for in-kenel servers

2007-11-02 Thread Latchesar Ionkov
/fs/9p/srv/ramfs/0/ctl Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- include/net/9p/srv.h | 24 +++- net/9p/srv.c | 386 +- 2 files changed, 405 insertions(+), 5 deletions(-) diff --git a/include/net/9p/srv.h b/include/net/9p/srv.h

[PATCH 6/10] 9p: in-kernel server basic support

2007-11-02 Thread Latchesar Ionkov
This patch implements the basic functionality required for implementing in-kernel 9P file servers. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- include/net/9p/9p.h |2 + include/net/9p/srv.h | 169 +++ net/9p/Kconfig |5 + net/9p/Makefile |4 + net/9p

[PATCH 1/10] 9p: new transport interface

2007-11-02 Thread Latchesar Ionkov
interface making it more suitable for implementing various 9P transports. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- fs/9p/v9fs.c |1 - include/net/9p/9p.h|1 + include/net/9p/client.h| 13 ++- include/net/9p/conn.h | 57 include/net/9p

[PATCH] 9p: basic sysfs support

2007-11-02 Thread Latchesar Ionkov
This patch implements the basic sysfs support for 9p. If CONFIG_NET_9P_DEBUG is defined, allows reading and modifying the debug level via /sysfs/fs/9p/debuglevel. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- include/net/9p/9p.h |1 + net/9p/mod.c| 45

[PATCH 2/10] 9p: fcall deserialization changes

2007-11-02 Thread Latchesar Ionkov
Modify p9_deserialize_fcall to read from p9_fcall's internal buffer instead of specifying an external one. Add functions that copy raw data to and from a fcall. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- include/net/9p/9p.h |4 +++- net/9p/conv.c | 34

[PATCH 8/10] 9p: loopback transport

2007-11-02 Thread Latchesar Ionkov
9P loopback transport that can be used between 9P in-kernel servers and v9fs on the same machine. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- net/9p/Kconfig |7 + net/9p/Makefile |4 + net/9p/trans_loop.c | 371 +++ 3

[PATCH 5/10] 9p: marshalling changes for in-kernel server

2007-11-02 Thread Latchesar Ionkov
This patch implements serialization/deserialization for the server-side 9P messages. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- include/net/9p/9p.h | 16 ++ net/9p/conv.c | 457 +-- 2 files changed, 462 insertions(+), 11

[PATCH 4/10] 9p: tranport interface changes for in-kernel server

2007-11-02 Thread Latchesar Ionkov
This patch adds in-kernel 9P server support to the transport interface. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- fs/9p/v9fs.c |2 +- include/net/9p/transport.h | 31 --- net/9p/mux.c |6 +++--- 3 files changed, 32

[PATCH][REFERENCE ONLY] 9p: ramfs 9p server

2007-11-02 Thread Latchesar Ionkov
Sample ramfs file server that uses the in-kernel 9P file server support. This code is for reference only. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- net/9p/Kconfig |8 +- net/9p/Makefile |1 + net/9p/ramfs/ramfs.c | 986

Re: [PATCH] 9p: basic sysfs support

2007-11-02 Thread Latchesar Ionkov
On 11/2/07, Greg KH [EMAIL PROTECTED] wrote: On Fri, Nov 02, 2007 at 10:53:31AM -0600, Latchesar Ionkov wrote: This patch implements the basic sysfs support for 9p. If CONFIG_NET_9P_DEBUG is defined, allows reading and modifying the debug level via /sysfs/fs/9p/debuglevel. Since

[PATCH] 9p: add missing end-of-options record for trans_fd

2007-10-26 Thread Latchesar Ionkov
The list of options that the fd transport accepts is missing end-of-options marker. This patch adds it. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- commit 70ec0c7936c2516d128fdb1a32d749071b8846ec tree 8de5495f83b94096825f8bfe0767e4fcbaf36ef3

[PATCH] 9p: return NULL when trans not found

2007-10-26 Thread Latchesar Ionkov
v9fs_match_trans function returns arbitrary transport module instead of NULL when the requested transport is not registered. This patch modifies the function to return NULL in that case. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- commit 25ed88ed319e40fb6426e2aaefcc5f136aadd4e

[PATCH] 9p: use copy of the options value instead of original

2007-10-26 Thread Latchesar Ionkov
field that v9fs_parse_options function uses instead of the original value. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- commit 0c0b7fa3d4e80ab3e4e69b8a55661f649d1b41ff tree eeb4ac6ea85387c153e3f7f6cad370e1c5dc8534 parent c9927c2bf4f45bb85e8b502ab3fb79ad6483c244 author Latchesar Ionkov <[

[PATCH] 9p: return NULL when trans not found

2007-10-26 Thread Latchesar Ionkov
v9fs_match_trans function returns arbitrary transport module instead of NULL when the requested transport is not registered. This patch modifies the function to return NULL in that case. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- commit 25ed88ed319e40fb6426e2aaefcc5f136aadd4ee tree

[PATCH] 9p: add missing end-of-options record for trans_fd

2007-10-26 Thread Latchesar Ionkov
The list of options that the fd transport accepts is missing end-of-options marker. This patch adds it. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- commit 70ec0c7936c2516d128fdb1a32d749071b8846ec tree 8de5495f83b94096825f8bfe0767e4fcbaf36ef3 parent

[PATCH] 9p: use copy of the options value instead of original

2007-10-26 Thread Latchesar Ionkov
that v9fs_parse_options function uses instead of the original value. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- commit 0c0b7fa3d4e80ab3e4e69b8a55661f649d1b41ff tree eeb4ac6ea85387c153e3f7f6cad370e1c5dc8534 parent c9927c2bf4f45bb85e8b502ab3fb79ad6483c244 author Latchesar Ionkov [EMAIL PROTECTED

[PATCH] 9p: fix memory leak in v9fs_get_sb

2007-10-23 Thread Latchesar Ionkov
This patch fixes a memory leak in v9fs_get_sb. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- commit 77250c234636881976ebd567f9edc7c36711bd4a tree 35d683472542706a3b78eb51bc29b92f690c314a parent 01e7ae8c13bb06a2ce622ebace33bb7e28ef596c author Latchesar Ionkov <[EMAIL

[PATCH] 9p: fix memory leak in v9fs_get_sb

2007-10-23 Thread Latchesar Ionkov
This patch fixes a memory leak in v9fs_get_sb. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- commit 77250c234636881976ebd567f9edc7c36711bd4a tree 35d683472542706a3b78eb51bc29b92f690c314a parent 01e7ae8c13bb06a2ce622ebace33bb7e28ef596c author Latchesar Ionkov [EMAIL PROTECTED] 1193169149

[PATCH] 9p: v9fs_vfs_rename incorrect clunk order

2007-10-22 Thread Latchesar Ionkov
In v9fs_vfs_rename function labels don't match the fids that are clunked. The correct clunk order is clunking newdirfid first and then olddirfid next. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- commit 0c9269a1384273b7a409a2163fdf91cd3909288

[PATCH] 9p: v9fs_vfs_rename incorrect clunk order

2007-10-22 Thread Latchesar Ionkov
In v9fs_vfs_rename function labels don't match the fids that are clunked. The correct clunk order is clunking newdirfid first and then olddirfid next. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- commit 0c9269a1384273b7a409a2163fdf91cd39092889 tree

[RFC][PATCH 3/6] 9p: 9P server connection code

2007-09-30 Thread Latchesar Ionkov
This patchset provides support for in-kernel 9P2000 servers. This patch add the 9P server connection code. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- net/9p/srv/conn.c | 743 + 1 files changed, 743 insertions(+), 0 del

[RFC][PATCH 2/6] 9p: 9P server interface and common srv code

2007-09-30 Thread Latchesar Ionkov
This patchset provides support for in-kernel 9P2000 servers. This patch adds the header file that defines the 9P server interface as well as the code for the common support functions for the server. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- include/net/9p/srv.h

[RFC][PATCH 5/6] 9p: 9P server TCP socket connection support

2007-09-30 Thread Latchesar Ionkov
This patchset provides support for in-kernel 9P2000 servers. Support for 9P servers listening on TCP sockets. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- net/9p/srv/socksrv.c | 252 ++ 1 files changed, 252 insertions(+), 0 del

[RFC][REFERENCE ONLY] 9p: 9P ramfs sample implementation

2007-09-30 Thread Latchesar Ionkov
Sample ramfs file server that uses the 9P in-kernel server implementation. This code is for reference only, it is not supposed to be merged into the kernel. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- net/9p/srv/Kconfig |6 + net/9p/srv/Makefile |4 + net/9p/srv/r

[RFC][PATCH 1/6] 9p: 9P server low-level 9P messages support

2007-09-30 Thread Latchesar Ionkov
This patchset provides support for in-kernel 9P2000 servers. Implement support for the 9P messages required by the server side. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- include/net/9p/9p.h | 19 ++ net/9p/conv.c

[RFC][PATCH 4/6] 9p: 9P server fid management

2007-09-30 Thread Latchesar Ionkov
This patchset provides support for in-kernel 9P2000 servers. This patch adds the 9P server fid management. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- net/9p/srv/fid.c | 178 ++ 1 files changed, 178 insertions(+), 0 del

[RFC][PATCH 6/6] 9p: 9P server Kconfig and Makefile changes

2007-09-30 Thread Latchesar Ionkov
This patchset provides support for in-kernel 9P2000 servers. 9P server Kconfig and Makefile changes. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- net/9p/Kconfig |1 + net/9p/Makefile |1 + net/9p/srv/Kconfig |9 + net/9p/srv/Makefile |7

[RFC][PATCH 6/6] 9p: 9P server Kconfig and Makefile changes

2007-09-30 Thread Latchesar Ionkov
This patchset provides support for in-kernel 9P2000 servers. 9P server Kconfig and Makefile changes. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- net/9p/Kconfig |1 + net/9p/Makefile |1 + net/9p/srv/Kconfig |9 + net/9p/srv/Makefile |7 +++ 4

[RFC][PATCH 4/6] 9p: 9P server fid management

2007-09-30 Thread Latchesar Ionkov
This patchset provides support for in-kernel 9P2000 servers. This patch adds the 9P server fid management. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- net/9p/srv/fid.c | 178 ++ 1 files changed, 178 insertions(+), 0 deletions

[RFC][REFERENCE ONLY] 9p: 9P ramfs sample implementation

2007-09-30 Thread Latchesar Ionkov
Sample ramfs file server that uses the 9P in-kernel server implementation. This code is for reference only, it is not supposed to be merged into the kernel. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- net/9p/srv/Kconfig |6 + net/9p/srv/Makefile |4 + net/9p/srv/ramfs.c

[RFC][PATCH 1/6] 9p: 9P server low-level 9P messages support

2007-09-30 Thread Latchesar Ionkov
This patchset provides support for in-kernel 9P2000 servers. Implement support for the 9P messages required by the server side. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- include/net/9p/9p.h | 19 ++ net/9p/conv.c | 508

[RFC][PATCH 2/6] 9p: 9P server interface and common srv code

2007-09-30 Thread Latchesar Ionkov
This patchset provides support for in-kernel 9P2000 servers. This patch adds the header file that defines the 9P server interface as well as the code for the common support functions for the server. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- include/net/9p/srv.h | 208

[RFC][PATCH 5/6] 9p: 9P server TCP socket connection support

2007-09-30 Thread Latchesar Ionkov
This patchset provides support for in-kernel 9P2000 servers. Support for 9P servers listening on TCP sockets. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- net/9p/srv/socksrv.c | 252 ++ 1 files changed, 252 insertions(+), 0 deletions

[RFC][PATCH 3/6] 9p: 9P server connection code

2007-09-30 Thread Latchesar Ionkov
This patchset provides support for in-kernel 9P2000 servers. This patch add the 9P server connection code. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- net/9p/srv/conn.c | 743 + 1 files changed, 743 insertions(+), 0 deletions

Re: [PATCH] 9p: rename uid and gid parameters

2007-09-13 Thread Latchesar Ionkov
, Eric Van Hensbergen <[EMAIL PROTECTED]> wrote: > On 9/12/07, Latchesar Ionkov <[EMAIL PROTECTED]> wrote: > > Change the names of 'uid' and 'gid' parameters to the more appropriate > > 'dfltuid' and 'dfltgid'. > > > > ... > > > strcpy(v9se

Re: [PATCH] 9p: rename uid and gid parameters

2007-09-13 Thread Latchesar Ionkov
, Eric Van Hensbergen [EMAIL PROTECTED] wrote: On 9/12/07, Latchesar Ionkov [EMAIL PROTECTED] wrote: Change the names of 'uid' and 'gid' parameters to the more appropriate 'dfltuid' and 'dfltgid'. ... strcpy(v9ses-name, V9FS_DEFUSER); strcpy(v9ses-remotename, V9FS_DEFANAME

[PATCH] 9p: attach-per-user

2007-09-12 Thread Latchesar Ionkov
ations as a single user (access=any) (default for 9P2000) V9fs does a single attach and all operations are done as a single user. If this mode is selected, the v9fs behavior is identical with the current one. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]>

[PATCH] 9p: define session flags

2007-09-12 Thread Latchesar Ionkov
Create more general flags field in the v9fs_session_info struct and move the 'extended' flag as a bit in the flags. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- commit 52f23404fd5bd77b619460e00930087463ec0cd9 tree 41c68f68a211796fb65d9c772120e7b7587dc945

[PATCH] 9p: rename uid and gid parameters

2007-09-12 Thread Latchesar Ionkov
Change the names of 'uid' and 'gid' parameters to the more appropriate 'dfltuid' and 'dfltgid'. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- commit 13bf527796712619df072c0963e3f6c8c00189b8 tree 7211a2899dcfd58c76b901334a8726c7e60115e1 parent 52f23404fd5bd77b619460e00930087463

[PATCH] 9p: define session flags

2007-09-12 Thread Latchesar Ionkov
Create more general flags field in the v9fs_session_info struct and move the 'extended' flag as a bit in the flags. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- commit 52f23404fd5bd77b619460e00930087463ec0cd9 tree 41c68f68a211796fb65d9c772120e7b7587dc945 parent

[PATCH] 9p: rename uid and gid parameters

2007-09-12 Thread Latchesar Ionkov
Change the names of 'uid' and 'gid' parameters to the more appropriate 'dfltuid' and 'dfltgid'. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- commit 13bf527796712619df072c0963e3f6c8c00189b8 tree 7211a2899dcfd58c76b901334a8726c7e60115e1 parent 52f23404fd5bd77b619460e00930087463ec0cd9

[PATCH] 9p: attach-per-user

2007-09-12 Thread Latchesar Ionkov
as a single user (access=any) (default for 9P2000) V9fs does a single attach and all operations are done as a single user. If this mode is selected, the v9fs behavior is identical with the current one. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- commit

[PATCH] 9p: attach-per-user

2007-09-03 Thread Latchesar Ionkov
V9fs does a single attach and all operations are done as a single user. If this mode is selected, the v9fs behavior is identical with the current one. The patch also renames uid and gid options to dfltuid and dfltgid. The new names describe better the values they set. Signed-off-by: Latchesar

[PATCH] 9p: attach-per-user

2007-09-03 Thread Latchesar Ionkov
does a single attach and all operations are done as a single user. If this mode is selected, the v9fs behavior is identical with the current one. The patch also renames uid and gid options to dfltuid and dfltgid. The new names describe better the values they set. Signed-off-by: Latchesar Ionkov

Re: [V9fs-developer] [kvm-devel] [RFC] 9p: add KVM/QEMU pci transport

2007-08-29 Thread Latchesar Ionkov
p of PCI. I think your PCI transport > would make a pretty nice start of a PCI virtio transport though. > > Regards, > > Anthony Liguori > > On Tue, 2007-08-28 at 13:52 -0500, Eric Van Hensbergen wrote: > > From: Latchesar Ionkov <[EMAIL PROTECTED]> > > > &g

Re: [V9fs-developer] [kvm-devel] [RFC] 9p: add KVM/QEMU pci transport

2007-08-29 Thread Latchesar Ionkov
your PCI transport would make a pretty nice start of a PCI virtio transport though. Regards, Anthony Liguori On Tue, 2007-08-28 at 13:52 -0500, Eric Van Hensbergen wrote: From: Latchesar Ionkov [EMAIL PROTECTED] This adds a shared memory transport for a synthetic 9p device

Re: [kvm-devel] [RFC] 9p: add KVM/QEMU pci transport

2007-08-28 Thread Latchesar Ionkov
On 8/28/07, Arnd Bergmann <[EMAIL PROTECTED]> wrote: > On Tuesday 28 August 2007, Eric Van Hensbergen wrote: > > > This adds a shared memory transport for a synthetic 9p device for > > paravirtualized file system support under KVM/QEMU. > > Nice driver. I'm hoping we can do a virtio driver using a

Re: [kvm-devel] [RFC] 9p: add KVM/QEMU pci transport

2007-08-28 Thread Latchesar Ionkov
On 8/28/07, Arnd Bergmann [EMAIL PROTECTED] wrote: On Tuesday 28 August 2007, Eric Van Hensbergen wrote: This adds a shared memory transport for a synthetic 9p device for paravirtualized file system support under KVM/QEMU. Nice driver. I'm hoping we can do a virtio driver using a similar

Re: net/9p/mux.c: use-after-free

2007-07-25 Thread Latchesar Ionkov
Yep, it's a leak. Thanks, Lucho On 7/25/07, Eric Van Hensbergen <[EMAIL PROTECTED]> wrote: On 7/22/07, Adrian Bunk <[EMAIL PROTECTED]> wrote: > The Coverity checker spotted the following use-after-free > in net/9p/mux.c: > > <-- snip --> > > ... > struct p9_conn *p9_conn_create(struct

Re: net/9p/mux.c: use-after-free

2007-07-25 Thread Latchesar Ionkov
Yep, it's a leak. Thanks, Lucho On 7/25/07, Eric Van Hensbergen [EMAIL PROTECTED] wrote: On 7/22/07, Adrian Bunk [EMAIL PROTECTED] wrote: The Coverity checker spotted the following use-after-free in net/9p/mux.c: -- snip -- ... struct p9_conn *p9_conn_create(struct p9_transport

Re: CTL_UNNUMBERED (Re: [PATCH] 9p: Don't use binary sysctl numbers.)

2007-07-23 Thread Latchesar Ionkov
It doesn't really matter (for me) whether it is sysctl or sysfs interface. The sysctl approach seemed easier to implement. If the consensus is to use sysfs, I'll send a patch (for 2.6.24). Sorry for the incorrect implementation, I guess I stole the code from unappropriate place :) Thanks,

Re: CTL_UNNUMBERED (Re: [PATCH] 9p: Don't use binary sysctl numbers.)

2007-07-23 Thread Latchesar Ionkov
It doesn't really matter (for me) whether it is sysctl or sysfs interface. The sysctl approach seemed easier to implement. If the consensus is to use sysfs, I'll send a patch (for 2.6.24). Sorry for the incorrect implementation, I guess I stole the code from unappropriate place :) Thanks,

Re: [V9fs-developer] [GIT PULL] 9p Patches for 2.6.23 merge window

2007-07-16 Thread Latchesar Ionkov
in Eric's repository and made tracking of the changes harder. We will be more careful next time. Thanks for you comments, Lucho On 7/16/07, Linus Torvalds <[EMAIL PROTECTED]> wrote: On Sun, 15 Jul 2007, Latchesar Ionkov wrote: > > I thought that it is not a good idea to keep the

Re: [V9fs-developer] [GIT PULL] 9p Patches for 2.6.23 merge window

2007-07-16 Thread Latchesar Ionkov
in Eric's repository and made tracking of the changes harder. We will be more careful next time. Thanks for you comments, Lucho On 7/16/07, Linus Torvalds [EMAIL PROTECTED] wrote: On Sun, 15 Jul 2007, Latchesar Ionkov wrote: I thought that it is not a good idea to keep the v9fs_ prefix for code

Re: [V9fs-developer] [GIT PULL] 9p Patches for 2.6.23 merge window

2007-07-15 Thread Latchesar Ionkov
I thought that it is not a good idea to keep the v9fs_ prefix for code that is in different places (fs/9p and net/9p). If keeping the old prefix is more acceptable, I can create a new patch without the "v9fs_"->"p9_" renames. Thanks, Lucho On 7/15/07, Linus Torvalds <[EMAIL PROTECTED]>

Re: [V9fs-developer] [GIT PULL] 9p Patches for 2.6.23 merge window

2007-07-15 Thread Latchesar Ionkov
I thought that it is not a good idea to keep the v9fs_ prefix for code that is in different places (fs/9p and net/9p). If keeping the old prefix is more acceptable, I can create a new patch without the v9fs_-p9_ renames. Thanks, Lucho On 7/15/07, Linus Torvalds [EMAIL PROTECTED] wrote: On

[PATCH 2/4] v9fs: move non-vfs related headers to include/net/9p

2007-05-08 Thread Latchesar Ionkov
-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- fs/9p/9p.h | 415 fs/9p/conn.h | 53 -- fs/9p/conv.c |2 +- fs/9p/fcall.c |6 +- fs/9p/fcprint.c|2 +- fs/9p

[PATCH 2/4] v9fs: move non-vfs related headers to include/net/9p

2007-05-08 Thread Latchesar Ionkov
-by: Latchesar Ionkov [EMAIL PROTECTED] --- fs/9p/9p.h | 415 fs/9p/conn.h | 53 -- fs/9p/conv.c |2 +- fs/9p/fcall.c |6 +- fs/9p/fcprint.c|2 +- fs/9p/fid.c

Re: [PATCH] 9p: create separate 9p client interface

2007-04-30 Thread Latchesar Ionkov
On 4/30/07, Jan Engelhardt <[EMAIL PROTECTED]> wrote: On Apr 30 2007 09:32, Latchesar Ionkov wrote: Hi! > --- a/fs/9p/Makefile > +++ b/fs/9p/Makefile > @@ -3,8 +3,8 @@ obj-$(CONFIG_9P_FS) := 9p.o > 9p-objs := \ > trans_fd.o \ > mux.o \ > - fcall.o \ >

Re: [PATCH] 9p: create separate 9p client interface

2007-04-30 Thread Latchesar Ionkov
Hi, On 4/30/07, Christoph Hellwig <[EMAIL PROTECTED]> wrote: On Mon, Apr 30, 2007 at 09:32:41AM -0600, Latchesar Ionkov wrote: > Create a separate 9P client interface that can be used outside the VFS > layer. In addition to VFS, the new interface can be used to export the >

Re: [PATCH] 9p: create separate 9p client interface

2007-04-30 Thread Latchesar Ionkov
Hi, On 4/30/07, Christoph Hellwig [EMAIL PROTECTED] wrote: On Mon, Apr 30, 2007 at 09:32:41AM -0600, Latchesar Ionkov wrote: Create a separate 9P client interface that can be used outside the VFS layer. In addition to VFS, the new interface can be used to export the authentication channel

Re: [PATCH] 9p: create separate 9p client interface

2007-04-30 Thread Latchesar Ionkov
On 4/30/07, Jan Engelhardt [EMAIL PROTECTED] wrote: On Apr 30 2007 09:32, Latchesar Ionkov wrote: Hi! --- a/fs/9p/Makefile +++ b/fs/9p/Makefile @@ -3,8 +3,8 @@ obj-$(CONFIG_9P_FS) := 9p.o 9p-objs := \ trans_fd.o \ mux.o \ - fcall.o \ conv.o \ + clnt.o \ Do you pay

[PATCH] v9fs: don't use primary fid when removing file

2007-04-21 Thread Latchesar Ionkov
). This patch changes v9fs_remove to use a cloned fid, so the primary fid is not locked and freed. Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]> --- commit ca1a80584fc3211dac158492173467d4f87a27ac tree 787de07bd6d24bdcc9907f90d9085dcd774b2ea4 parent 0f851021c0f91e5073fa89f26b5ac68e23

[PATCH] v9fs: don't use primary fid when removing file

2007-04-21 Thread Latchesar Ionkov
). This patch changes v9fs_remove to use a cloned fid, so the primary fid is not locked and freed. Signed-off-by: Latchesar Ionkov [EMAIL PROTECTED] --- commit ca1a80584fc3211dac158492173467d4f87a27ac tree 787de07bd6d24bdcc9907f90d9085dcd774b2ea4 parent 0f851021c0f91e5073fa89f26b5ac68e23df8e11