> > On Tue, May 15, 2018 at 12:35:34PM -0400, Doug Ledford wrote: > > On Mon, 2018-05-14 at 09:51 -0500, Steve Wise wrote: > > > > > > On 5/13/2018 8:24 AM, Leon Romanovsky wrote: > > > > On Mon, May 07, 2018 at 08:53:16AM -0700, Steve Wise wrote: > > > > > This enhancement allows printing rdma device-specific state, if > provided > > > > > by the kernel. This is done in a generic manner, so rdma tool doesn't > > > > > > > > Double space between "." and "This". > > > > > > > > > need to know about the details of every type of rdma device. > > > > > > > > > > Driver attributes for a rdma resource are in the form of <key, > > > > > [print_type], value> tuples, where the key is a string and the value can > > > > > be any supported driver attribute. The print_type attribute, if > present, > > > > > > > > ditto > > > > > > I'll fix these. > > > > Fix it if you want, but don't do it because Leon told you to. A double > > space after period is perfectly acceptable. > > It is very controversial thing [1], > > "Most style guides indicate that single sentence spacing is proper for > final or published work today, and most publishers require manuscripts > to be submitted as they will appear in publication—single > sentence spaced." > > [1] https://en.wikipedia.org/wiki/Sentence_spacing We're not writing a manuscript. 😉 Regardless, I made the changes and they are in v2 of the patch series, which I think is probably ready to merge. Steve. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
