New NFS-RDMA Patchset

New NFS-RDMA Patchset

[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


-------------------------------------------------------------------------
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  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs

Re: New NFS-RDMA Patchset

[Trond Myklebust]

On Tue, 2007-06-12 at 10:27 -0500, Tom Tucker wrote:
> 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  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs

Re: New NFS-RDMA Patchset

[Christoph Hellwig]

On Tue, Jun 12, 2007 at 11:38:48PM -0400, Trond Myklebust wrote:
> On Tue, 2007-06-12 at 10:27 -0500, Tom Tucker wrote:
> > 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 :-)

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  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs