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

From	Logan Gunthorpe <logang@deltatee.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v3 09/10] ntb_test: Add a selftest script for the NTB subsystem
Date	2016-06-16 00:40 +0200
Message-ID	<rKrDA-8vO-7@gated-at.bofh.it> (permalink)
References	<rKqHw-7SL-27@gated-at.bofh.it> <rKr0R-80K-27@gated-at.bofh.it> <rKr0R-80K-29@gated-at.bofh.it> <rKr0R-80K-25@gated-at.bofh.it> <rKrkd-8p8-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw


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 the ntb device, so the driver does not allow the link to be disabled in such topology.

Ok, I wasn't aware of this.  But looking closer I think I have a better
solution:

ntb_link_disable should return -EINVAL if the hardware can't support
bringing the link down. If I add the error check to my tool_link_write
(which I neglected to do) then writing an "N" to $LOC/link will fail and
the test could be skipped. I'll make a v4 spin.

Logan

> A simple thing to do here might be:
> 
> write_file "N" "$LOC/link"
> sleep 1
> read_file "$REM/link"
> 
> You already have my Ack.  This minor issue can be fixed later if anyone cares.  I don't think it is a big deal, just worth pointing out that the script will hang here instead of report a failure.  If it is worth fixing later, at that point we might also want to change this script to continue with other tests instead of exit on the first failure.
>

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

Thread

Re: [PATCH v3 09/10] ntb_test: Add a selftest script for the NTB  subsystem Logan Gunthorpe <logang@deltatee.com> - 2016-06-16 00:00 +0200
  RE: [PATCH v3 09/10] ntb_test: Add a selftest script for the NTB subsystem "Allen Hubbe" <Allen.Hubbe@emc.com> - 2016-06-16 00:20 +0200
    Re: [PATCH v3 09/10] ntb_test: Add a selftest script for the NTB  subsystem Logan Gunthorpe <logang@deltatee.com> - 2016-06-16 00:40 +0200

csiph-web