Re: [PATCH RFC 2/4] ib/core: handle FDR-10 link encoding

2011-09-20 Thread Marcel Apfelbaum
Hi Sean, Thank you for your review. In my opinion link_encoding is at the same level as active_speed and active_width, they all help to compute the actual rate. Marcel On Mon, Sep 19, 2011 at 8:53 PM, Hefty, Sean sean.he...@intel.com wrote: Index: b/drivers/infiniband/core/sysfs.c

[PATCH RFC 2/4] ib/core: handle FDR-10 link encoding

2011-09-19 Thread Marcel Apfelbaum
Added support for new link encodings beyond the regular 8 to 10 bit encoding. For now only FDR-10 (64 to 66 bit encoding) is implemented. Signed-off-by: Marcel Apfelbaum marc...@dev.mellanox.co.il Reviewed-by: Hal Rosenstock h...@mellanox.com --- drivers/infiniband/core/sysfs.c |    2 +-

[PATCH RFC 2/4] ib/core: handle FDR-10 link encoding

2011-09-19 Thread Marcel Apfelbaum
Added support for new link encodings beyond the regular 8 to 10 bit encoding. For now only FDR-10 (64 to 66 bit encoding) is implemented. Signed-off-by: Marcel Apfelbaum marc...@dev.mellanox.co.il Reviewed-by: Hal Rosenstock h...@mellanox.com --- drivers/infiniband/core/sysfs.c |2 +-

RE: [PATCH RFC 2/4] ib/core: handle FDR-10 link encoding

2011-09-19 Thread Hefty, Sean
Index: b/drivers/infiniband/core/sysfs.c === --- a/drivers/infiniband/core/sysfs.c 2011-09-14 13:49:58.0 +0300 +++ b/drivers/infiniband/core/sysfs.c 2011-09-14 13:50:43.731775900 +0300 @@ -209,7 +209,7 @@ static ssize_t