Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1538146

RE: [PATCH 02/22] NTB: Add peer indexed ports NTB API

Show all headers | View raw

From: Allen Hubbe
> From: Serge Semin
> > Add new port-index NTB API. Additionally lets get rid of Primary and
> > Secondary topologies, since port-number can be effectively used instead.
> 
> Split into two patches please.
> 
> I see no harm to the TOPO changes, though I wonder if they are necessary.
> 

I am leaning more toward recommending that the topo api be left alone.

The topo changes to the ntb api complicate the Intel driver:
 - The changes add second branches where there had been just one before (need to check topo AND port now).
 - The changes also cause some complicated merge conflicts with https://github.com/davejiang/linux.git ntb.

See RE: [PATCH 10/22] NTB Intel: Add port-related NTB API callback methods

If we leave the ntb topo api as it was, then on multiport devices, if the local port is not the primary port, let it be one of potentially many secondary ports.  Or, if there is no distinction between primary/secondary on some hardware, let them all be primary.

This topo api doesn't have much value for existing drivers (that I know of), except for informative purposes.  So, my preference for changing it would be, only if necessary, and to minimize changes otherwise.

Allen

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

RE: [PATCH 02/22] NTB: Add peer indexed ports NTB API "Allen Hubbe" <Allen.Hubbe@dell.com> - 2016-12-04 01:50 +0100
  RE: [PATCH 02/22] NTB: Add peer indexed ports NTB API "Allen Hubbe" <Allen.Hubbe@dell.com> - 2016-12-08 00:00 +0100

csiph-web