> -----Original Message----- > From: Aurélien Aptel <aaptel@xxxxxxxx> > Sent: Tuesday, November 26, 2019 11:54 AM > To: Tom Talpey <ttalpey@xxxxxxxxxxxxx>; Pavel Shilovsky > <piastryyy@xxxxxxxxx> > Cc: linux-cifs <linux-cifs@xxxxxxxxxxxxxxx>; Steve French > <smfrench@xxxxxxxxx> > Subject: RE: [EXTERNAL] Re: [PATCH v4 1/6] cifs: sort interface list by speed > > Hi, > > Tom Talpey <ttalpey@xxxxxxxxxxxxx> writes: > > Sorting by speed is definitely appropriate, but sorting the other > > multichannel attributes is just as important. For example, if the > > RDMA attribute is set, the address should actually be excluded > > for non-RDMA connections (a second, non-RDMA entry is included, > > if the interface supports both). And, the RSS attribute indicates a > > "better" destination than non-RSS for a given speed, because it is > > more efficient at local traffic management. > > Note that the way the list is used has been altered in a later patch Hmm, that patch does help by taking RSS into account, but it does not take RDMA into consideration. The logic for that may be more complex, as it would need to determine whether an RDMA mount was desired, and an adapter available. And it looks like the code will try the RDMA addresses from other transport protocols. I also think it is more than a little too "heroic". Retrying seems unnecessary, doesn't the code refresh the list from time to time and reattempt connections? That's what the Windows client does, anyway. I would suggest avoiding any kind of retry except on the very first connection, and even then, many admins will want a fast-fail. Three TCP retries can take a long time! Tom. > here: > > https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Flore.ker > nel.org%2Flinux-cifs%2F20191120161559.30295-2- > aaptel%40suse.com%2FT%2F%23u&data=02%7C01%7Cttalpey%40micros > oft.com%7Cf76a99bd50ea4fbf4f7908d772914946%7C72f988bf86f141af91ab2 > d7cd011db47%7C1%7C0%7C637103840632982128&sdata=OJxqwDoV8C > CYNo69cgR9nwmc1xGR22s8J9Y9GDiV74M%3D&reserved=0 > > Cheers, > -- > Aurélien Aptel / SUSE Labs Samba Team > GPG: 1839 CB5F 9F5B FB9B AA97 8C99 03C8 A49B 521B D5D3 > SUSE Software Solutions Germany GmbH, Maxfeldstr. 5, 90409 Nürnberg, DE > GF: Felix Imendörffer, Mary Higgins, Sri Rasiah HRB 247165 (AG München)
