[U-Boot] test.py failure in test_ut[ut_dm_pci_ep_base]

[U-Boot] test.py failure in test_ut[ut_dm_pci_ep_base]

[Stephen Warren]

Running test.py on sandbox now fails test_ut[ut_dm_pci_ep_base] with:

=> ut dm pci_ep_base
Test: dm_test_pci_ep_base: pci_ep.c
[Errno 5] Input/output error

I believe that last line is printed by the test harness; presumably the 
sandbox process crashed, but I haven't checked that.

I imagine this was introduced by commit "test: pci_ep: add basic pci_ep 
tests".

I do see later commit "test: Disable pci_ep test for now" by Tom, but 
for some reason the test is still running after that; my test system 
last built a9a3a37f92b0 and there's still a failure.

[U-Boot] test.py failure in test_ut[ut_dm_pci_ep_base]

[Tom Rini]

On Mon, Jul 15, 2019 at 09:14:03AM -0600, Stephen Warren wrote:
> Running test.py on sandbox now fails test_ut[ut_dm_pci_ep_base] with:
> 
> => ut dm pci_ep_base
> Test: dm_test_pci_ep_base: pci_ep.c
> [Errno 5] Input/output error
> 
> I believe that last line is printed by the test harness; presumably the
> sandbox process crashed, but I haven't checked that.
> 
> I imagine this was introduced by commit "test: pci_ep: add basic pci_ep
> tests".
> 
> I do see later commit "test: Disable pci_ep test for now" by Tom, but for
> some reason the test is still running after that; my test system last built
> a9a3a37f92b0 and there's still a failure.

To be clear, my commit only removes it from sandbox from travis runs and
a similar '-k not pci' would be needed in other hooks.  Ramon is working
on the problem now.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190715/3a6b163b/attachment.sig>

[U-Boot] test.py failure in test_ut[ut_dm_pci_ep_base]

[Ramon Fried]

On Mon, Jul 15, 2019 at 6:25 PM Tom Rini <trini@konsulko.com> wrote:

> On Mon, Jul 15, 2019 at 09:14:03AM -0600, Stephen Warren wrote:
> > Running test.py on sandbox now fails test_ut[ut_dm_pci_ep_base] with:
> >
> > => ut dm pci_ep_base
> > Test: dm_test_pci_ep_base: pci_ep.c
> > [Errno 5] Input/output error
> >
> > I believe that last line is printed by the test harness; presumably the
> > sandbox process crashed, but I haven't checked that.
> >
> > I imagine this was introduced by commit "test: pci_ep: add basic pci_ep
> > tests".
> >
> > I do see later commit "test: Disable pci_ep test for now" by Tom, but for
> > some reason the test is still running after that; my test system last
> built
> > a9a3a37f92b0 and there's still a failure.
>
> To be clear, my commit only removes it from sandbox from travis runs and
> a similar '-k not pci' would be needed in other hooks.  Ramon is working
> on the problem now.
>
Hi,
here's the fix. sorry for that one (my uninitialized memory was zeros :) )
https://patchwork.ozlabs.org/patch/1132255/


> --
> Tom
>

[U-Boot] test.py failure in test_ut[ut_dm_pci_ep_base]

[Stephen Warren]

On 7/15/19 2:07 PM, Ramon Fried wrote:
> 
> 
> On Mon, Jul 15, 2019 at 6:25 PM Tom Rini <trini@konsulko.com 
> <mailto:trini@konsulko.com>> wrote:
> 
>     On Mon, Jul 15, 2019 at 09:14:03AM -0600, Stephen Warren wrote:
>      > Running test.py on sandbox now fails test_ut[ut_dm_pci_ep_base] with:
>      >
>      > => ut dm pci_ep_base
>      > Test: dm_test_pci_ep_base: pci_ep.c
>      > [Errno 5] Input/output error
>      >
>      > I believe that last line is printed by the test harness;
>     presumably the
>      > sandbox process crashed, but I haven't checked that.
>      >
>      > I imagine this was introduced by commit "test: pci_ep: add basic
>     pci_ep
>      > tests".
>      >
>      > I do see later commit "test: Disable pci_ep test for now" by Tom,
>     but for
>      > some reason the test is still running after that; my test system
>     last built
>      > a9a3a37f92b0 and there's still a failure.
> 
>     To be clear, my commit only removes it from sandbox from travis runs and
>     a similar '-k not pci' would be needed in other hooks.  Ramon is working
>     on the problem now.
> 
> Hi,
> here's the fix. sorry for that one (my uninitialized memory was zeros :) )
> https://patchwork.ozlabs.org/patch/1132255/

Thanks. The test passes for me now; I assume that patch was applied.