Skip to contentSkip to conversation

New NFS-RDMA Patchset

DISCUSSION Tom Tucker 3 participants 3 messages Jun 12, 2007 – Jun 13, 2007

Subsystems (2)
lore
Share & tools
lore

Tom Tucker

author
#0 lore raw

I have a new patchset that addresses almost all of the comments from the last review. The patchset includes the transport switch and the rdma transport driver. The patchset is rather large and has 33 patches. I'd like some guidance as to whether I tar it up or just send them out.

Thanks,
Tom

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs

#1 lore raw
Quoted 7 lines · Tom Tucker

I have a new patchset that addresses almost all of the comments from the last review. The patchset includes the transport switch and the rdma transport driver. The patchset is rather large and has 33 patches. I'd like some guidance as to whether I tar it up or just send them out.

Thanks,
Tom

If you are worried about the number of patches then there are 3 possible solutions:

1. Provide a link to a website which lists the patches in an easily reviewable format.
2. Put them up in a git tree, and send us a pointer
3. Just send us a pointer to the gitweb site for the git tree you set up in 2. above :-)

Cheers
Trond

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs

#2 lore raw
Quoted 9 lines · Trond Myklebust
Quoted 7 lines · Tom Tucker

I have a new patchset that addresses almost all of the comments from the last review. The patchset includes the transport switch and the rdma transport driver. The patchset is rather large and has 33 patches. I'd like some guidance as to whether I tar it up or just send them out.

Thanks,
Tom

Quoted 8 lines

If you are worried about the number of patches then there are 3 possible solutions:

1. Provide a link to a website which lists the patches in an easily reviewable format.
2. Put them up in a git tree, and send us a pointer
3. Just send us a pointer to the gitweb site for the git tree you set up in 2. above :-)

Gitweb is extremly painfull to review, the only thing even worse is a git tree without gitweb :)

the only reasonable formats for a big pile of patches to review are a tarball of patches or a patchseries sent to a mailinglist.

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs