| programmerjake | ghostmansd[m]: I couldn't find your account on debian salsa... | 01:04 |
|---|---|---|
| programmerjake | i'll send you an email where you can enable getting ci build failure emails if you want that, you'll only get them if the commit tested has you listed as author or committer. if you don't want them, feel free to ignore the email | 01:07 |
| programmerjake | note the url expires in 3 days | 01:22 |
| Veera[m] | Hi | 04:57 |
| Veera[m] | Currently xauth is not installed in Talos server. May I install it? Is there any risk. Because I want ssh X11 forwarding. | 04:58 |
| Guest16 | Hi, I am setting up my envt in ubuntu 22.04. | 05:09 |
| Guest16 | The charoot contains two users now, one for my normal user and one for root. both have src directories and on has many repos fetched and the other only has hdl_tools. | 05:10 |
| Guest16 | is this fine? the SUDO_USER shows root. | 05:10 |
| sadoon[m] | I just snagged a 20-core power9 chip off ebay for 1300$! | 07:06 |
| sadoon[m] | Great upgrade to my current 8-core which cost me the same 3 years ago | 07:06 |
| sadoon[m] | This should immensely help with building debian and gentoo :D | 07:07 |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@176.59.55.196> has quit IRC | 07:19 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@91.205.170.186> has joined #libre-soc | 07:19 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@91.205.170.186> has quit IRC | 07:42 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@176.59.160.191> has joined #libre-soc | 07:44 | |
| Veera[m] | sadoon: HI, have you considered LFS/CLFS!! | 08:18 |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@176.59.160.191> has quit IRC | 08:48 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@91.205.170.186> has joined #libre-soc | 08:49 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@91.205.170.186> has quit IRC | 08:58 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@91.205.170.186> has joined #libre-soc | 08:59 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@91.205.170.186> has quit IRC | 09:16 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@91.205.170.186> has joined #libre-soc | 09:17 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@91.205.170.186> has quit IRC | 09:24 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@176.59.42.102> has joined #libre-soc | 09:27 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@176.59.42.102> has quit IRC | 12:07 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@176.59.42.102> has joined #libre-soc | 12:07 | |
| openpowerbot | [mattermost] <lkcl> sadoon[m], niiice | 12:25 |
| openpowerbot | [mattermost] <lkcl> Guest16, hi | 12:25 |
| openpowerbot | [mattermost] <lkcl> ok. right. what you've probably done, if SUDO_USER==root, is you've typed "schroot -c {insert_your_chroot}" at a *root* prompt | 12:25 |
| openpowerbot | [mattermost] <lkcl> and then perhaps typed "sudo bash" inside the chroot when you were already root. | 12:25 |
| openpowerbot | [mattermost] <lkcl> you should type the "schroot" command when an *ordinary* user | 12:26 |
| openpowerbot | [mattermost] <lkcl> and *then* type "sudo bash" to get to a root user inside the chroot | 12:26 |
| openpowerbot | [mattermost] <lkcl> lkcl@fizzy:~/src/libresoc/openpower-isa$ schroot -c cleansoc | 12:26 |
| openpowerbot | [mattermost] <lkcl> (cleansoc)lkcl@fizzy:~$ env | grep SUDO_USER | 12:26 |
| openpowerbot | [mattermost] <lkcl> (nothing) | 12:26 |
| openpowerbot | [mattermost] <lkcl> (cleansoc)lkcl@fizzy:~$ sudo bash | 12:26 |
| openpowerbot | [mattermost] <lkcl> (cleansoc)root@fizzy:/home/lkcl# env | grep SUDO_USER | 12:27 |
| openpowerbot | [mattermost] <lkcl> SUDO_USER=lkcl | 12:27 |
| openpowerbot | [mattermost] <lkcl> yeh? | 12:27 |
| openpowerbot | [mattermost] <lkcl> so start from an *ordinary* user, with the schroot command: that's what schroot is specifically designed for, and why it was chosen: | 12:27 |
| openpowerbot | [mattermost] <lkcl> to allow an *ordinary user* to gain [controlled] access to chroots. | 12:27 |
| openpowerbot | [mattermost] <lkcl> Veera[m], you good to do this? https://bugs.libre-soc.org/show_bug.cgi?id=907 | 12:28 |
| lkcl | Guest16, so for example here: | 12:31 |
| lkcl | https://libre-soc.org/HDL_workflow/devscripts/ | 12:31 |
| lkcl | notice how it says "exit" to get *OUT* of the root prompt, back to an ordinary user, *BEFORE* running the schroot command? | 12:32 |
| lkcl | ghostmansd[m], i'm going to be dropping LDSTRM-2P-1S1D-PU mode, programmerjake came up with a scheme that, with a little adapting, can put pack/unpack into the 24-bit prefix modes | 12:37 |
| Guest16 | I guess I made that mistake. I will probably will have to do this from scratch I guess. | 12:45 |
| ghostmansd[m] | lkcl, perfect! | 12:48 |
| openpowerbot | [mattermost] <lkcl> Guest16 no not at all | 12:48 |
| openpowerbot | [mattermost] <lkcl> or, well, if you ended up with everything in /home/root/src then that's down to you | 12:49 |
| Guest16 | Can I just copy the src dir from root to normal user and continue as nothing like this happened? My worry is if this is going to break something in python as installs were done with 'python3 setup.py develop' | 12:51 |
| openpowerbot | [mattermost] <lkcl> yes you can - just remember (laboriously, one by one) to do "python3 setup.py develop" in each directory | 12:52 |
| openpowerbot | [mattermost] <lkcl> (as root) | 12:52 |
| Guest16 | yes, I can do that. Thanks. | 12:53 |
| openpowerbot | [mattermost] <lkcl> it should not do anything other than just rewrite the /usr/local/lib/python3.7/blahblah .pth file | 12:53 |
| openpowerbot | [mattermost] <lkcl> if you want to make sure it doesn't go off randomly downloading crap off the internet, use the --nodeps option. python3 setup.py develop --nodeps | 12:53 |
| Guest16 | great. | 12:53 |
| Guest16 | If I want to implement things on an FPGA (Say Kintex 7), can I generate verilog code starting with migen and then use that verilog code as a starting point for Xilinx Vivado suit? | 12:55 |
| openpowerbot | [mattermost] <lkcl> yes you can - although none of us have done it, i have used nextpnr-xilinx instead | 12:56 |
| openpowerbot | [mattermost] <lkcl> libresoc.v can be compiled up as a drop-in replacement for microwatt.v (with a little hand-editing) | 12:57 |
| openpowerbot | [mattermost] <lkcl> which is what was done here: https://bugs.libre-soc.org/show_bug.cgi?id=855 | 12:58 |
| openpowerbot | [mattermost] <lkcl> please excuse me i need to go get an aircon unit, urgently | 12:58 |
| Guest16 | Sure, thanks for the support. | 12:59 |
| openpowerbot | [mattermost] <lkcl> any time | 12:59 |
| *** lx0 <lx0!~lxo@gateway/tor-sasl/lxo> has quit IRC | 14:03 | |
| *** lx0 <lx0!~lxo@gateway/tor-sasl/lxo> has joined #libre-soc | 14:04 | |
| *** openpowerbot <openpowerbot!~openpower@94-226-188-34.access.telenet.be> has quit IRC | 14:04 | |
| *** ghostmansd <ghostmansd!~ghostmans@91.205.170.186> has joined #libre-soc | 14:07 | |
| ghostmansd | https://salsa.debian.org/Kazan-team/mirrors/openpower-isa/-/jobs/3098206 | 16:01 |
| ghostmansd | programmerjake, lkcl, do these results look well to you? Can I merge the binutils branch? | 16:02 |
| ghostmansd | src/openpower/decoder/isa/test_caller_bitmanip.py OK locally on talos | 16:04 |
| ghostmansd | nope, sorry, this was alu test | 16:06 |
| ghostmansd | in bitmanip, do_case_grev is broken... I'll check on master | 16:06 |
| ghostmansd | looks like the master is broken too | 16:09 |
| ghostmansd | Wow, src/openpower/decoder/isa/test_caller_bitmanip_av.py ends with segmentation fault | 16:10 |
| ghostmansd | ...and fails on master with some bitching about bmask | 16:10 |
| ghostmansd | src/openpower/decoder/isa/test_caller_fmvis.py fails on master with KeyError("fmvis") | 16:11 |
| ghostmansd | folks, I won't commit the code yet, still, I wonder, is it caused by my issues with the environment? If notm how long it was that, despite the fact we have CI, these tests are broken? | 16:13 |
| ghostmansd | sigh, it seems I haven't done the rebase... hang on | 16:22 |
| ghostmansd | OK take 2... src/openpower/decoder/isa/test_caller_logical.py OK | 16:26 |
| ghostmansd | src/openpower/decoder/isa/test_caller_fmvis.py fails with KeyError: 'fmvis' | 16:31 |
| ghostmansd | src/openpower/decoder/isa/test_caller_bitmanip_av.py fails with KeyError: 'bmask' | 16:34 |
| ghostmansd | src/openpower/decoder/isa/test_caller_bitmanip.py fails somewhere in do_case_grev | 16:35 |
| ghostmansd | src/openpower/decoder/isa/test_caller_alu.py OK | 16:36 |
| sadoon[m] | <Veera[m]> "sadoon: HI, have you considered..." <- Not at all | 16:37 |
| sadoon[m] | LFS has no package management whatsoever, you're on your own. | 16:37 |
| sadoon[m] | It would be a waste of time and resources tbh | 16:37 |
| sadoon[m] | We already have all the basic stuff we need anyways, ppc* all have sid variants and gentoo support | 16:38 |
| *** manili[m] <manili[m]!~manilimat@2001:470:69fc:105::2:479c> has quit IRC | 17:00 | |
| ghostmansd | lkcl, a quick question. You mentioned you were going to drop 2P-1S1D-PU form. | 17:40 |
| ghostmansd | https://libre-soc.org/irclog/%23libre-soc.2022-08-13.log.html#t2022-08-13T12:37:38 | 17:41 |
| ghostmansd | It cancels the task 872, am I right? | 17:41 |
| *** ghostmansd <ghostmansd!~ghostmans@91.205.170.186> has quit IRC | 17:57 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@176.59.42.102> has quit IRC | 18:07 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@176.59.42.102> has joined #libre-soc | 18:07 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@176.59.42.102> has quit IRC | 18:12 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@91.205.168.173> has joined #libre-soc | 18:13 | |
| lkcl | ghostmansd[m], pretty impressive when a /usr/bin/python3 segfault occurs, innit? :) | 18:17 |
| lkcl | ghostmansd[m], yes the grev ones i expect to break, as they are no longer valid | 18:18 |
| ghostmansd[m] | I couldn't even realize why. I rebased the repository and then it became 1:1 with master with some tests I checked. | 18:18 |
| * lkcl back from shopping for a mini aircon unit, it's so hot now my laptop's in danger | 18:19 | |
| ghostmansd[m] | However, since CI is all broken with this soc module, it's hard to tell whether I'm safe to merge the code. | 18:19 |
| lkcl | gimme 1 sec to check | 18:20 |
| ghostmansd[m] | That said, no need to merge it now, I'll only need the code generation for binutils. | 18:20 |
| ghostmansd[m] | Since I will have the copy in binutils, this is not urgent. | 18:20 |
| lkcl | i've disabled the grev tests, their opcode space allocation has been used for other things | 18:21 |
| ghostmansd[m] | Fine! And what about fmvis and bmask KeyError? | 18:22 |
| lkcl | https://bugs.libre-soc.org/show_bug.cgi?id=872 - no, it changes to "support for new LD/ST, normal and CRops Pack/Unpack mode" | 18:22 |
| lkcl | give me a sec still catching up | 18:22 |
| ghostmansd[m] | Aha, sure, no rush | 18:22 |
| lkcl | done changed subject/etc of 873 | 18:24 |
| lkcl | s/873/872 | 18:24 |
| lkcl | fmvis/fishmv work perfectly fine here (master branch) | 18:25 |
| lkcl | you remembered to do "pywriter noall av"? (or just "pywriter")? | 18:26 |
| lkcl | likewise no bmask error. | 18:27 |
| * lkcl checked binutils branch, test_caller_bitmanip_av.py passes | 18:29 | |
| * lkcl test_called_fmvis.py passes | 18:29 | |
| lkcl | we expect the grev ones to fail | 18:30 |
| * lkcl going to run some more | 18:30 | |
| lkcl | logical good | 18:31 |
| *** tplaten <tplaten!~isengaara@55d4b887.access.ecotel.net> has joined #libre-soc | 18:31 | |
| lkcl | test_caller_fp.py good | 18:52 |
| lkcl | alu exts transcendentals good | 18:53 |
| lkcl | matrix dct fft... good | 18:56 |
| ghostmansd[m] | >you remembered to do "pywriter noall av"? (or just "pywriter")? | 18:56 |
| ghostmansd[m] | Aaah crap. I always forget about it. | 18:56 |
| ghostmansd[m] | Thanks for tip! | 18:56 |
| lkcl | pywriter (needlessly) does everything, pywriter noall {something} will run just that one "thing" | 18:57 |
| lkcl | so you don't have to recompile the whole lot | 18:57 |
| lkcl | yeahh it's turning to be quite regular, forgetting it is a compiler :) | 18:57 |
| lkcl | ya know what, i think we're good. | 18:58 |
| tplaten | SVP64 is five instructions, if I understand that correctly those are: setvl, svremap, svindex, svshape and svstep | 18:58 |
| lkcl | tplaten, yes. plus 25% of EXT001 v3.1 "Prefix" space | 18:59 |
| tplaten | I already knew setvl before reading the spec again. I vaguely remember some of the other four ones. | 18:59 |
| ghostmansd[m] | cf. minor_22.csv | 18:59 |
| ghostmansd[m] | There are more | 18:59 |
| tplaten | In the past I did a port of the Intel Embree Library to POWER9 making use of VSX. In the future I might do a port of Embree to SVP64. | 19:00 |
| lkcl | toshywoshy, mattermost has a new one - bad gateway | 19:04 |
| * lkcl checking all good after disconnect.... | 19:04 | |
| lkcl | svstep is for Vertical-First Mode but also you can get it to generate the indices for you which would be used in a Vector op | 19:05 |
| lkcl | which is extremely useful. | 19:05 |
| lkcl | let's say you have... mmm... you want to use REMAP but it's too complicated, you don't have enough SVSHAPEs | 19:05 |
| lkcl | what you would do is | 19:05 |
| lkcl | set up the SVSHAPE you want | 19:05 |
| lkcl | run the svstep instruction to get a copy of the indices (whatever they are) and enable "Indexing" Mode | 19:05 |
| lkcl | then set up the *alternative* (other) SVSHAPEs using some other different svremap mode | 19:05 |
| lkcl | or | 19:06 |
| lkcl | you can get the indices using svstep, then, say, multiply them by 4 or 8, or whatever, to use them for LD/ST offsets or something | 19:06 |
| lkcl | https://www.embree.org/ oooOoo | 19:06 |
| lkcl | ghostmansd[m], libera disconnected, i saw "cf minor_22.csv" but not much else? | 19:07 |
| lkcl | i ran a ton of the test_caller_* | 19:07 |
| lkcl | i really should run the main Soc HDL TestCaller though | 19:08 |
| lkcl | gimme about... half an hour | 19:08 |
| lkcl | ok that's underway | 19:09 |
| ghostmansd[m] | Yeah this was basically all I wanted to say to tplaten :-) | 19:09 |
| lkcl | :) | 19:09 |
| ghostmansd[m] | I meant that new insns are there, plus also minor_59.csv, IIRC | 19:09 |
| *** ghostmansd <ghostmansd!~ghostmans@91.205.168.173> has joined #libre-soc | 19:11 | |
| ghostmansd | lkcl, I think I should finally clean up sv_analysis.py from the crap I introduced for binutils :-) | 19:27 |
| ghostmansd | Sorry, I promise I'll do it, just not now | 19:27 |
| ghostmansd | You remember, there was some intermediate stage before I decided to decouple binutils stuff into the standalone script. | 19:28 |
| ghostmansd | I know this makes sv_analysis barely readable, I'm really sorry. Just one of things I couldn't realize at that point. After I'll complete this jihad with prefixes, I'll do it. | 19:30 |
| ghostmansd | (I just had to take a look at some bits in sv_analysis and found I forgot to clean it) | 19:31 |
| Guest16 | hello, if I want to get an idea about how all separate modules glued together to create the SOC, should I look at the class 'TestIssuerBase'? | 19:42 |
| lkcl | Guest16, yes or more TestIssuer. then Core. then soc/fu/*.py | 20:00 |
| lkcl | then probably this https://libre-soc.org/3d_gpu/architecture/ | 20:01 |
| lkcl | these images will also help | 20:02 |
| lkcl | https://ftp.libre-soc.org/course_18oct2021/ | 20:02 |
| Guest16 | Thanks. I will take a look. | 20:03 |
| Guest16 | Also what is the purpose of the Power simulator (mostly generated one). Is it to use as a benchmark to test accuracy of the real implementation? | 20:03 |
| *** tplaten <tplaten!~isengaara@55d4b887.access.ecotel.net> has quit IRC | 20:03 | |
| lkcl | it's auto-generated | 20:03 |
| lkcl | therefore we can confirm the accuracy of the actual specification as a [deliberate] side-effect | 20:04 |
| lkcl | plus save a frickin lot of time | 20:04 |
| lkcl | but it also serves the purpose of understanding what the hell is going on. Power ISA is... (a) comprehensive and | 20:04 |
| lkcl | (b) written in MSB0 order not LSB0 order, in the spec | 20:04 |
| Guest16 | ok, got it. | 20:05 |
| lkcl | so for example in order to make sure the RADIX MMU was working, we wrote one for the Simulator | 20:06 |
| lkcl | https://git.libre-soc.org/?p=openpower-isa.git;a=blob;f=src/openpower/decoder/isa/radixmmu.py;hb=HEAD | 20:06 |
| lkcl | that then allows us to run unit tests side-by-side on the simulated MMU with the HDL | 20:07 |
| lkcl | on a really fast 4.8 ghz processor the simulator does 2,500 instructions per second *and we don't care* | 20:07 |
| lkcl | because when you run the HDL that's somewhere well below *10* instructions per second, so who cares if the simulator is 2,500 2,500,000 or 2,500,000,000 | 20:08 |
| lkcl | ghostmansd, simple/test/test_issuer.py works great with a whole batch of pipeline unit tests, i'm more than happy for the binutils openpower-isa branch to be merged to master | 20:09 |
| ghostmansd | Well, this is mostly due to the fact that currently I only touched enumerations from the common part; I only used insndb for binutils for now. | 20:10 |
| ghostmansd | So it's not like that too much stuff is expected to break (though I managed to break some bits with enum experiments). | 20:11 |
| ghostmansd | But I think merging it is a good idea to kinda mark the transition. | 20:11 |
| ghostmansd | Perhaps I could even decouple it from 838, but I'm too busy with prefices right now to think about it. :-) | 20:12 |
| ghostmansd | As I mentioned, we have a bit different masks if compared to binutils (binutils also consider Rc). So we're cheating a little bit. | 20:13 |
| lkcl | as long as it's cheating within the sv.xxxxx context i don't think anyone will notice :) | 20:14 |
| Guest16 | is there any docs available that describe how to create a processor (say by running issuer_verilog), simulate it using verilator (or any simulator) possibly by loading a program (built with cross compiler)? | 20:14 |
| ghostmansd | Fair remark :-) | 20:14 |
| lkcl | Guest16, kinda. although to be honest you're slightly better off reading microwatt docs then dropping in libresoc.v as a replacement for microwatt.v | 20:17 |
| lkcl | that said | 20:17 |
| lkcl | i have a branch of microwatt that i did exactly that, which tplaten has been playing with | 20:18 |
| lkcl | which can execute helloworld.bin and even joel shenki's linux-5.7 | 20:18 |
| lkcl | 1 sec | 20:18 |
| lkcl | https://libre-soc.org/HDL_workflow/microwatt/ | 20:19 |
| Guest16 | nice, this will be very useful. | 20:20 |
| Guest16 | 18 years back I did a course at the university which was based on this book - https://home.csulb.edu/~mchelian/ACA_I.html (probably 3rd edition). Not sure if this book will still be useful/applicable in understanding superscaller/branch prediction/etc used in this implementation? | 20:23 |
| *** lx0 <lx0!~lxo@gateway/tor-sasl/lxo> has quit IRC | 20:44 | |
| *** lx0 <lx0!~lxo@gateway/tor-sasl/lxo> has joined #libre-soc | 20:45 | |
| *** openpowerbot <openpowerbot!~openpower@94-226-188-34.access.telenet.be> has joined #libre-soc | 20:45 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@91.205.168.173> has quit IRC | 21:24 | |
| *** ghostmansd[m] <ghostmansd[m]!~ghostmans@176.59.56.10> has joined #libre-soc | 21:24 | |
| *** Guest16 <Guest16!~Guest16@2402:d000:8114:342c:fd7d:f033:dccb:b516> has quit IRC | 21:37 | |
| *** ghostmansd <ghostmansd!~ghostmans@91.205.168.173> has quit IRC | 21:39 | |
| lkcl | ah yes. hennessy and patterson. patterson was unbelievably rude to me. also his technical assessment of the 6600 scoreboard system caused Mitch Alsup to write up two extended chapters to "Design of a Computer" | 22:00 |
| lkcl | Mitch very kindly spent 6 *months* on comp.arch, explaining 6600 scoreboards to me, how they may be made Precise and also Multi-Issue. | 22:00 |
| lkcl | branch-prediction predication precise-exceptions may all be achieved by "Shadowing" which is nothing more difficult than having a bit-matrix that prevents write-back of results until you know it's safe to do so | 22:02 |
| lkcl | https://libre-soc.org/3d_gpu/architecture/6600scoreboard/ | 22:02 |
| *** ghostmansd <ghostmansd!~ghostmans@176.59.56.10> has joined #libre-soc | 22:03 | |
| lkcl | the word "Shadow" is used because once a write-hold-back is enabled it must also cause a "shadow" (holding back write) on all *downstream* instructions as well. | 22:03 |
| lkcl | when you know that the instruction is 100% with certainty okay to commit (i.e. there is no possibility of an exception ocurring, no branches are wrong, no predicate mask-bit unread) | 22:05 |
| lkcl | you pull the shadow flag and allow the write | 22:05 |
| lkcl | if on the other hand an exception occurs, the branch-prediction is wrogn, or the predicate mask-bit read and found to be zero | 22:06 |
| ghostmansd | Yeshttps://bugs.libre-soc.org/show_bug.cgi?id=577#c2 | 22:07 |
| lkcl | you pull the *cancel* flag | 22:07 |
| lkcl | regardless of whether the operation has been completed and has the result already, or is waiting for operands, or even has started the operation | 22:07 |
| ghostmansd[m] | Fuck, stupid IRC | 22:08 |
| ghostmansd | lkcl, could it happen that we touched something regarding remapping algorithm? For example, some fields which were marked as immediate... | 22:08 |
| ghostmansd | I'm asking since after the last generation this doesn't work | 22:08 |
| ghostmansd | .set fv0, 32 | 22:08 |
| ghostmansd | .set win, 4 | 22:08 |
| ghostmansd | sv.lfs/els *fv0, 256(win) | 22:08 |
| ghostmansd | https://bugs.libre-soc.org/show_bug.cgi?id=577#c2 | 22:08 |
| ghostmansd | It fails upon remapping the second argument | 22:09 |
| lkcl | ghostmansd, rright, ok, so there would have been two mappings, one for sv.lfssh | 22:09 |
| lkcl | one for sv.lfs | 22:09 |
| lkcl | they would, cough, have overwritten each other | 22:09 |
| lkcl | the correct one is... | 22:10 |
| lkcl | + .in2 = SVP64_IN2_SEL_CONST_SI, | 22:10 |
| lkcl | + .in3 = SVP64_IN3_SEL_NONE, | 22:10 |
| lkcl | yes that's correct. | 22:10 |
| *** ghostmansd <ghostmansd!~ghostmans@176.59.56.10> has quit IRC | 22:10 | |
| lkcl | + .sv_etype = SVP64_ETYPE_EXTRA2, | 22:10 |
| lkcl | + .sv_in1 = SVP64_EXTRA_NONE, | 22:10 |
| lkcl | that's odd. | 22:11 |
| lkcl | ah i think i know what it is - it might be being classified as a Pack/Unpack. | 22:11 |
| *** ghostmansd <ghostmansd!~ghostmans@176.59.56.10> has joined #libre-soc | 22:11 | |
| lkcl | it should definitely not be NONE | 22:12 |
| lkcl | - .sv_etype = SVP64_ETYPE_EXTRA3, | 22:12 |
| lkcl | - .sv_in1 = SVP64_EXTRA_IDX1, | 22:12 |
| lkcl | that'll be correct | 22:12 |
| ghostmansd | So is it that the extra calculation broken? | 22:13 |
| lkcl | yes. | 22:13 |
| lkcl | it may be accidentally being qualified as LDSTRM-2P-1S1D-PU which is being removed | 22:14 |
| ghostmansd | Hm. | 22:16 |
| ghostmansd | https://git.libre-soc.org/?p=openpower-isa.git;a=blob;f=src/openpower/decoder/power_insn.py;h=48300beb2da0e3fb7651bae34296ffeb522c9686;hb=53a8d9118f70333f82e2e049e6763b5f5715b93c#l473 | 22:16 |
| ghostmansd | This is how sv_* is calculated in insndb | 22:16 |
| Veera[m] | <openpowerbot> "[mattermost] <lkcl> Veera, you..." <- I will. Have started! | 22:17 |
| ghostmansd | Let me check which results I'll get with decode_extra | 22:17 |
| lkcl | Veera[m], brilliant | 22:18 |
| lkcl | ghostmansd, i need to clear the "PU" column | 22:18 |
| Veera[m] | lkcl: Currently xauth is not installed in Talos server. May I install it? Is there any risk. Because I want ssh X11 forwarding. | 22:18 |
| lkcl | Veera[m], give me 1 second | 22:19 |
| lkcl | Veera[m], done | 22:19 |
| lkcl | https://git.libre-soc.org/?p=openpower-isa.git;a=blob;f=src/openpower/sv/sv_analysis.py;hb=HEAD#l145 | 22:20 |
| lkcl | that needs to be disabled | 22:20 |
| ghostmansd | but still it works in pysvp64asm... what part of the bigger picture do I miss? | 22:24 |
| lkcl | because there are - were - flags in the csv files which PowerDecoder is checking | 22:24 |
| lkcl | i'm taking pack/unpack out of sv_analysis entirely | 22:25 |
| ghostmansd | but how did it lead to different extras? did these flags affect those? | 22:26 |
| lkcl | yes. | 22:26 |
| lkcl | there were - and this is precisely why i removed it - TWO repeat TWO repeat TWO separate meanings for "lfs" | 22:27 |
| ghostmansd | whoa | 22:27 |
| ghostmansd | OK, so you're dropping PU flag, right? | 22:27 |
| Veera[m] | <lkcl> "Veera, done" <- thanks | 22:27 |
| lkcl | there was a bit in the 24-bit prefix "shift mode" which, if set, you were supposed to interpret the exact same 32-bit opcode (normally lfs) in a *completely* different encoding | 22:28 |
| lkcl | (SVD) | 22:28 |
| ghostmansd | I must tune insndb appropriately, I just taught it this flag :-) | 22:28 |
| lkcl | i've removed both the shift flag *and* the PU flag | 22:28 |
| ghostmansd | OK, ping me when you're ready :-) | 22:29 |
| ghostmansd | The thing is, I simply tried checking this media/audio/mp3/mp3_0_apply_window_float_basicsv.s assembly | 22:30 |
| ghostmansd | And found it stopped working for binutils | 22:30 |
| ghostmansd | Took me a while to find that lfs is changed :-) | 22:30 |
| lkcl | done | 22:30 |
| lkcl | it'll be the combination of PU being added, a bug where the lfssh flag had been removed, a total fluke that RC (the extra argument of lfssh) was set to zero... | 22:31 |
| ghostmansd | hm | 22:32 |
| ghostmansd | I also see with a new output that some opcodes were changed... | 22:32 |
| ghostmansd | sv.rldicl, sv.rldicl., sv.sradi, sv.sradi. | 22:32 |
| ghostmansd | also there's a new entry about sv.isel | 22:33 |
| ghostmansd | and also some entries switched between extra2/extra3 (both directions) | 22:33 |
| lkcl | they should all be back to where they were about a week ago | 22:35 |
| lkcl | commit e1d9815e05eafaff2260585cf10199c84bd9fe19 | 22:35 |
| lkcl | Author: Luke Kenneth Casson Leighton <lkcl@lkcl.net> | 22:35 |
| lkcl | Date: Sat Jul 30 15:29:08 2022 +0100 | 22:35 |
| lkcl | add PACK/UNPACK constants for RM-2P-1S1D-PU | 22:35 |
| ghostmansd | OK I'll check | 22:36 |
| ghostmansd | nope, still the same | 22:41 |
| ghostmansd | OK will look tomorrow, it's almost 1AM here | 22:41 |
| lkcl | i just re-added some missing CSV files | 22:41 |
| ghostmansd | I can barely understand what happens :-) | 22:41 |
| lkcl | ok | 22:41 |
| lkcl | :) | 22:41 |
| lkcl | brain. melted. | 22:41 |
| lkcl | till tomorrow | 22:41 |
| ghostmansd | OK never surrender. I found I lacked the mapping for XX_OR_ZERO stuff. | 22:54 |
| ghostmansd | checking again... | 22:54 |
| ghostmansd | yes it works!!! | 22:55 |
| ghostmansd | I'll think of how map these better, preferrably automatically | 22:56 |
| ghostmansd | we have many enums there, and any InXSel/OutSel/CRInSel/CROutSel must be mapped to some SVReg enum... | 22:57 |
| ghostmansd | I think I'll use the enum name, short and sweet | 22:57 |
| ghostmansd | You did it with strings, so it was simpler. This one checks the enumeration values, though, so it's good in other regards. Let's combine best of these worlds. | 22:58 |
| ghostmansd | Still, enough for today. | 22:58 |
| ghostmansd | Thanks for tips, lkcl! | 22:58 |
| ghostmansd | gn | 22:58 |
| *** ghostmansd <ghostmansd!~ghostmans@176.59.56.10> has quit IRC | 23:06 | |
| lkcl | hooraay | 23:10 |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!