RE: [PATCH v2 1/3] ntb: Add asynchronous devices support to NTB-bus interface

2016-08-19 Thread Allen Hubbe
From: Serge Semin > 3) IDT driver redevelopment will take a lot of time, since I don't have much > free time to > do it. It may be half of year or even more. > > From my side, such an improvement will significantly complicate the NTB > Kernel API. Since > you are the subsystem maintainer it's yo

Re: [PATCH v2 1/3] ntb: Add asynchronous devices support to NTB-bus interface

2016-08-19 Thread Serge Semin
Allen, There is no any comment below, just this one. After a short meditation I realized what you are trying to achieve. Your primary intentions was to unify the NTB interface so it would fit both Inte/AMD and IDT hardware without doing any abstraction. You may understand why I so eager in refus

Re: [PATCH v2 1/3] ntb: Add asynchronous devices support to NTB-bus interface

2016-08-18 Thread Serge Semin
Hello Allen, Sorry for the delay with response and thanks for thoughtful review. On Mon, Aug 08, 2016 at 05:48:42PM -0400, Allen Hubbe wrote: > From: Serge Semin > > Hello Allen. > > > > Thanks for your careful review. Going through this mailing thread I hope > > we'll come up > > with solutio

RE: [PATCH v2 1/3] ntb: Add asynchronous devices support to NTB-bus interface

2016-08-08 Thread Allen Hubbe
From: Serge Semin > Hello Allen. > > Thanks for your careful review. Going through this mailing thread I hope > we'll come up > with solutions, which improve the driver code as well as extend the Linux > kernel support > of new devices like IDT PCIe-swtiches. > > Before getting to the inline co

Re: [PATCH v2 1/3] ntb: Add asynchronous devices support to NTB-bus interface

2016-08-07 Thread Serge Semin
Hello Allen. Thanks for your careful review. Going through this mailing thread I hope we'll come up with solutions, which improve the driver code as well as extend the Linux kernel support of new devices like IDT PCIe-swtiches. Before getting to the inline commentaries I need to give some intro

RE: [PATCH v2 1/3] ntb: Add asynchronous devices support to NTB-bus interface

2016-08-05 Thread Allen Hubbe
From: Serge Semin > Currently supported AMD and Intel Non-transparent PCIe-bridges are synchronous > devices, so translated base address of memory windows can be direcly written > to peer registers. But there are some IDT PCIe-switches which implement > complex interfaces using Lookup Tables of tra