| mikolajw | bool() can be just replaced (before we deal with bignum) with !!() | 01:21 |
|---|---|---|
| mikolajw | lkcl: it appears you haven't finished writing the test | 01:34 |
| lkcl | mikolajw, correct, and i'm not going to [finish it], i have to deal with the MMU. | 03:34 |
| lkcl | it runs the simulator (runs a compile). if i spend time on that unit test it risks missing the NGI POINTER contractual deadline | 03:35 |
| lkcl | which has extremely serious consequences | 03:35 |
| lkcl | i'm quite happy to put up a bugreport and associated budget with it for you (or someone else) to do the PowerDecode2 unit test | 03:37 |
| lkcl | but i need to get the MMU working as quickly as possible then move on to the next critical path task immediately after that | 03:38 |
| mikolajw | I could try to get this going. How much longer is it going to be than test_power_decoder.py? | 04:09 |
| lkcl | well, i took out the decoder stuff, because that's (effectively) already tested | 04:18 |
| lkcl | perhaps i shouldn't have, but hey | 04:19 |
| lkcl | you could... mmm... say... check that RA and RB are correct | 04:19 |
| lkcl | those should be 2 and 6. sorry, RA and RS | 04:21 |
| lkcl | and those should beee.... dut.dec_a.reg_out.reg_a | 04:22 |
| lkcl | btw none of this you will actually see in the Simulation() | 04:22 |
| lkcl | it'll just be checking values you get *from* the Simulation() | 04:23 |
| lkcl | you can also get RA and RB from... | 04:23 |
| lkcl | dut.e.read_reg1.data | 04:24 |
| lkcl | see decoder/test/test_decoder_gas.py, which _should_ be working (but was written 18 months ago by someone else and i didn't know existed) | 04:27 |
| *** kylel1 is now known as kylel | 10:54 | |
| cesar | When running "python ~/src/soc/src/soc/simple/test/test_issuer.py nosvp64 general", I get: | 11:08 |
| cesar | File "/home/cstrauss/src/openpower-isa/src/openpower/test/runner.py", line 212, in process | 11:08 |
| cesar | self.default_mem.clear() | 11:08 |
| cesar | AttributeError: 'TestRunner' object has no attribute 'default_mem' | 11:09 |
| cesar | Also: File "/home/cstrauss/src/soc/src/soc/simple/test/test_runner.py", line 366, in end_test | 11:09 |
| cesar | print("after test %s cr value %x" % (self.test.name, cr)) | 11:09 |
| cesar | AttributeError: 'HDLRunner' object has no attribute 'test' | 11:10 |
| lkcl | cesar, i haven't run test_issuer.py for some time, let me sort that by adding an empty default_mem | 12:48 |
| lkcl | cesar, sorted https://git.libre-soc.org/?p=openpower-isa.git;a=commitdiff;h=5c97adf0a105709cbcf853e69d0e58873ebfb2b1 | 12:51 |
| cesar | Confirmed fixed, thanks. | 12:55 |
| lkcl | cool | 12:56 |
| * lkcl still on the MMU | 12:56 | |
| * cesar is about to add an --inorder flag to test_issuer.py | 12:58 | |
| lkcl | great. btw you saw i split out the common code into TestIssuerBase? | 12:59 |
| lkcl | i meant to redo inorder.py by going through the same process of deletion of svp64 as i did initially | 13:00 |
| cesar | Indeed. | 13:20 |
| lkcl | cesar, an idea occurred to me, would you like to do a short talk at FOSDEM about gtkwave CSS? | 17:38 |
| * lkcl fixed the mmu.bin test 4, moving on to test 5 | 19:30 | |
| lkcl | just... wow. so much detail involved here. | 19:37 |
| * sadoon_albader[m has finally got around setting up the Debian bullseye server for ppc64/ppc | 22:29 | |
| sadoon_albader[m | Repo* | 22:32 |
| lkcl | coool | 23:26 |
| sadoon_albader[m | Tested and working! | 23:50 |
| sadoon_albader[m | Time to reinstall debian on my machines I suppose, I wish there was a way to downgrade instead heh | 23:50 |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!