Re: [PATCH v3 09/10] ntb_test: Add a selftest script for the NTB subsystem

2016-06-15 Thread Logan Gunthorpe
On 15/06/16 04:17 PM, Allen Hubbe wrote: > This test should fail on Intel RP/TB topology (two cpu sharing one ntb). The > link state is the link state of the secondary side pcie bus connected to the > secondary side cpu. The link must be up in order for the secondary side cpu > to discover

Re: [PATCH v3 09/10] ntb_test: Add a selftest script for the NTB subsystem

2016-06-15 Thread Logan Gunthorpe
On 15/06/16 04:17 PM, Allen Hubbe wrote: > This test should fail on Intel RP/TB topology (two cpu sharing one ntb). The > link state is the link state of the secondary side pcie bus connected to the > secondary side cpu. The link must be up in order for the secondary side cpu > to discover

RE: [PATCH v3 09/10] ntb_test: Add a selftest script for the NTB subsystem

2016-06-15 Thread Allen Hubbe
From: Logan Gunthorpe > On 15/06/16 03:49 PM, Allen Hubbe wrote: > >> +function link_test() > >> +{ > >> + LOC=$1 > >> + REM=$2 > >> + EXP=0 > >> + > >> + echo "Running link tests on: $(basename $LOC) / $(basename $REM)" > >> + > >> + write_file "N" "$LOC/link" > >> + write_file "N"

RE: [PATCH v3 09/10] ntb_test: Add a selftest script for the NTB subsystem

2016-06-15 Thread Allen Hubbe
From: Logan Gunthorpe > On 15/06/16 03:49 PM, Allen Hubbe wrote: > >> +function link_test() > >> +{ > >> + LOC=$1 > >> + REM=$2 > >> + EXP=0 > >> + > >> + echo "Running link tests on: $(basename $LOC) / $(basename $REM)" > >> + > >> + write_file "N" "$LOC/link" > >> + write_file "N"

Re: [PATCH v3 09/10] ntb_test: Add a selftest script for the NTB subsystem

2016-06-15 Thread Logan Gunthorpe
On 15/06/16 03:49 PM, Allen Hubbe wrote: >> +function link_test() >> +{ >> +LOC=$1 >> +REM=$2 >> +EXP=0 >> + >> +echo "Running link tests on: $(basename $LOC) / $(basename $REM)" >> + >> +write_file "N" "$LOC/link" >> +write_file "N" "$LOC/link_event" > > If it fails to

Re: [PATCH v3 09/10] ntb_test: Add a selftest script for the NTB subsystem

2016-06-15 Thread Logan Gunthorpe
On 15/06/16 03:49 PM, Allen Hubbe wrote: >> +function link_test() >> +{ >> +LOC=$1 >> +REM=$2 >> +EXP=0 >> + >> +echo "Running link tests on: $(basename $LOC) / $(basename $REM)" >> + >> +write_file "N" "$LOC/link" >> +write_file "N" "$LOC/link_event" > > If it fails to

RE: [PATCH v3 09/10] ntb_test: Add a selftest script for the NTB subsystem

2016-06-15 Thread Allen Hubbe
From: Logan Gunthorpe > This script automates testing doorbells, scratchpads and memory windows > for an NTB device. It can be run locally, with the NTB looped > back to the same host or use SSH to remotely control the second host. > > In the single host case, the script just needs to be passed

RE: [PATCH v3 09/10] ntb_test: Add a selftest script for the NTB subsystem

2016-06-15 Thread Allen Hubbe
From: Logan Gunthorpe > This script automates testing doorbells, scratchpads and memory windows > for an NTB device. It can be run locally, with the NTB looped > back to the same host or use SSH to remotely control the second host. > > In the single host case, the script just needs to be passed