7 This section describes the workflow and some best practices for developing
8 the Libre-SOC hardware. We use nmigen, yosys and symbiyosys, and this
9 page is intended not just to help you get set up, it is intended to
10 help advise you of some tricks and practices that will help you become
11 effective team contributors.
13 It is particularly important to bear in mind that we are not just
14 "developing code", here: we are creating a "lasting legacy educational
15 resource" for other people to learn from, and for businesses and students
16 alike to be able to use, learn from and augment for their own purposes.
18 It is also important to appreciate and respect that we are funded under
19 NLNet's Privacy and Enhanced Trust Programme <http://nlnet.nl/PET>. Full
20 transparency, readability, documentation, effective team communication
21 and formal mathematical proofs for all code at all levels is therefore
24 Therefore, we need not only to be "self-sufficient" (absolutely
25 under no circumstances critically reliant on somebody else's servers
26 **or protocols**) we also need to ensure that everything (including
27 all communication such as the mailing list archives) are recorded,
28 replicable, and accessible in perpetuity. Use of slack or a "forum"
29 either actively prevents or makes that much harder.
31 # Collaboration resources
33 The main message here: **use the right tool for the right job**.
35 * mailing list: general communication and discussion.
36 * irc channel #libre-soc on irc.libera.chat: real(ish)-time communication.
37 * bugtracker: task-orientated, goal-orientated *focussed* discussion.
38 * ikiwiki: document store, information store, and (editable) main website
39 * git repositories: code stores (**not binary or auto-generated output store**)
40 * ftp server (<https://ftp.libre-soc.org/>): large (temporary,
41 auto-generated) file store.
43 Note also the lack of a "forum" in the above list. this is very
44 deliberate. forums are a serious distraction when it comes to technical
45 heavily goal-orientated development. recent internet users may enjoy
46 looking up the "AOL metoo postings" meme.
48 Note also the complete lack of "social platforms". if we wanted to tell
49 everybody how much better each of us are than anyone else in the team,
50 how many times we made a commit (look at me, look at me, i'm so clever),
51 and how many times we went to the bathroom, we would have installed a
52 social media based project "management" system.
54 ## Main contact method: mailing list
56 To respect the transparency requirements, conversations need to be
57 public and archived (i.e not skype, not telegram, not discord,
58 and anyone seriously suggesting slack will be thrown to the
59 lions). Therefore we have a mailing list. Everything goes through
60 there. <https://lists.libre-soc.org/mailman/listinfo/libre-soc-dev>
61 therefore please do google "mailing list etiquette" and at the very
62 minimum look up and understand the following:
64 * This is a technical mailing list with complex topics. Top posting
65 is completely inappropriate. Don't do it unless you have mitigating
66 circumstances, and even then please apologise and explain ("hello sorry
67 using phone at airport flight soon, v. quick reply: ....")
68 * Always trim context but do not cut excessively to the point where people
69 cannot follow the discussion. Especially do not cut the attribution
70 ("On monday xxx wrote") of something that you are actually replying
72 * Use inline replies i.e. reply at the point in the relevant part of
73 the conversation, as if you were actually having a conversation.
74 * Follow standard IETF reply formatting, using ">" for cascaded
75 indentation of other people's replies. If using gmail, please: SWITCH
76 OFF RICH TEXT EDITING.
77 * Please for god's sake do not use "my replies are in a different
78 colour". Only old and highly regarded people still using AOL are allowed
79 to get away with that (such as Mitch).
80 * Start a new topic with a relevant subject line. If an existing
81 discussion changes direction, change the subject line to reflect the
82 new topic (or start a new conversation entirely, without using the
84 * DMARC is a pain on the neck. Try to avoid GPG signed messages. sigh.
85 * Don't send massive attachments. Put them online (no, not on facebook or
86 google drive or anywhere else that demands privacy violations) and provide
87 the link. Which should not require any kind of login to access. ask the
88 listadmin if you don't have anywhere suitable: FTP access can be arranged.
90 ### Actionable items from mailing list
92 If discussions result in any actionable items, it is important not to
93 lose track of them. Create a bugreport, find the discussion in the
94 archives <https://lists.libre-soc.org/pipermail/libre-soc-dev/>,
95 and put the link actually in the bugtracker as one of the comments.
97 At some point in any discussion, the sudden realisation may dawn on one
98 or more people that this is an "actionable" discussion. at that point
99 it may become better to use <https://bugs.libre-soc.org/>
100 itself to continue the discussion rather than to keep on dropping copies
101 of links into the bugtracker. The bugtracker sends copies of comments
102 *to* the list however this is 'one-way' (note from lkcl: because this
103 involves running an automated perl script from email, on every email,
104 on the server, that is a high security risk, and i'm not doing it. sorry.)
106 ### Mailing list != editable document store
108 Also, please do not use the mailing list as an "information or document
109 store or poor-man's editor" **including not sending large images**.
110 We have the wiki for that. Edit a page and
111 tell people what you did (summarise rather than drop the entire contents
112 at the list) and include the link to the page.
114 Or, if it is more appropriate, commit a document (or source code)
115 into the relevant git repository then look up the link in the gitweb
116 source tree browser and post that (in the bugtracker or mailing list)
117 See <https://git.libre-soc.org/>
119 ### gmail "spam"ifying the list
121 See <https://blog.kittycooper.com/2014/05/keeping-my-mailing-list-emails-out-of-gmails-spam-folder/>
123 Basically it is possible to select any message from the list, create a
124 "filter" (under "More"), and, on the 2nd dialog box, click the "never
125 send this to Spam" option.
129 bugzilla. old and highly effective. sign up in the usual way. any
130 problems, ask on the list.
132 Please do not ask for the project to be transferred to github or other
133 proprietary nonfree service "because it's soooo convenient", as the
134 lions are getting wind and gout from overfeeding on that one.
138 Runs the main libre-soc.org site (including this page). effective,
139 stunningly light on resources, and uses a git repository not a database.
140 That means it can be edited offline.
142 Usual deal: register an account and you can start editing and contributing
145 Hint: to create a new page, find a suitable page that would link to it,
146 first, then put the link in of the page you want to create, as if the
147 page already exists. Save that page, and you will find a question mark
148 next to the new link you created. click that link, and it will fire up a
149 "create new page" editor.
151 Wiki pages are formatted in [[markdown|ikiwiki/markdown]] syntax.
153 Hint again: the wiki is backed by a git repository. Don't go overboard
154 but at the same time do not be afraid that you might "damage" or "lose"
155 pages. Although it would be a minor pain, the pages can always be
156 reverted or edited by the sysadmins to restore things if you get in a tiz.
158 Assistance in creating a much better theme greatly appreciated. e.g.
159 <http://www.math.cmu.edu/~gautam/sj/blog/20140720-ikiwiki-navbar.html>
163 We use git. More on this below. We also use
164 [gitolite3](https://gitolite.com/gitolite/) running on a dedicated server.
165 again, it is extremely effective and low resource utilisation. Reminder:
166 lions are involved if github is mentioned.
168 [gitweb](https://git.wiki.kernel.org/index.php/Gitweb) is provided which
169 does a decent job. <https://git.libre-soc.org/>
171 [Git](https://en.wikipedia.org/wiki/Git) does version control, ie it
172 tracks changes to files so that previous versions can be got back or
175 Checklist page [[HDL_workflow/git_checklist]]
179 <https://ftp.libre-soc.org/> is available for storing large files
180 that do not belong in a git repository, if we have (or ever need)
181 any. Images (etc.) if small and appropriate should go into the
182 wiki, however .tgz archives (etc.) and, at some point, binaries,
183 should be on the ftp server.
185 Ask on the list if you have a file that belongs on the ftp server.
189 As an aside: all this is "old school" and run on a single core 512MB
190 VM with only a 20GB HDD allocation. it costs only 8 GBP per month from
191 mythic-beasts and means that the project is in no way dependent on anyone
192 else - not microsoft, not google, not facebook, not amazon.
194 We tried [gitlab](https://about.gitlab.com/). it didn't go well. please
195 don't ask to replace the above extremely resource-efficient services
200 RAM is the biggest requirement. Minimum 16GB, the more the better (32
201 or 64GB starts to reach "acceptable" levels. Disk space is not hugely
202 critical: 256GB SSD should be more than adequate. Simulations and
203 FPGA compilations however are where raw processing power is a must.
204 High end Graphics Cards are nonessential.
206 What is particularly useful is to have hi-res screens (curved is
207 *strongly* recommended if the LCD is over 24in wide, to avoid eyeballs
208 going "prism" through long term use), and to have several of them: the
209 more the better. Either a DisplayLink UD160A (or more modern variant)
210 or simply using a second machine (lower spec hardware because it will
211 run editors) is really effective.
213 Also it is really recommended to have a UHD monitor (4k - 3840x2160),
214 or at least 2560x1200. If given a choice, 4:3 aspect ratio is better
215 than 16:9 particularly when using several of them. However, caveat
216 (details below): please when editing do not assume that everyone will
217 have access to such high resolution screens.
221 First install and become familiar with
222 [Debian](https://www.debian.org/) ([Ubuntu](https://ubuntu.com/)
224 must) for standardisation cross-team and so that toolchain installation
225 is greatly simplified. yosys in particular warns that trying to use
226 Windows, BSD or MacOS will get you into a world of pain.
228 Only a basic GUI desktop is necessary: fvwm2, xfce4, lxde are perfectly
229 sufficient (alongside wicd-gtk for network management). Other more
230 complex desktops can be used however may consume greater resources.
232 # editors and editing
234 Whilst this is often a personal choice, the fact that many editors are
235 GUI based and run full-screen with the entire right hand side *and* middle
236 *and* the majority of the left side of the hi-res screen entirely unused
237 and bereft of text leaves experienced developers both amused and puzzled.
239 At the point where such full-screen users commit code with line lengths
240 well over 160 characters, that amusement quickly evaporates.
242 Where the problems occur with full-screen editor usage is when a project
243 is split into dozens if not hundreds of small files (as this one is). At
244 that point it becomes pretty much essential to have as many as six to
245 eight files open *and on-screen* at once, without overlaps i.e. not in
246 hidden tabs, next to at least two if not three additional free and clear
247 terminals into which commands are regularly and routinely typed (make,
248 git commit, nosetests3 etc). Illustrated with the following 3840x2160
249 screenshot (click to view full image), where *every one* of those 80x70
250 xterm windows is *relevant to the task at hand*.
252 [[!img 2020-01-24_11-56.png size=640x ]]
254 (hint/tip: fvwm2 set up with "mouse-over to raise focus, rather than
255 additionally requiring a mouse click, can save a huge amount of cumulative
256 development time here, switching between editor terminal(s) and the
259 Once this becomes necessary, it it turn implies that having greater
260 than 80 chars per line - and running editors full-screen - is a severe
261 hinderance to an essential *and highly effective* workflow technique.
263 Additionally, care should be taken to respect that not everyone will have
264 200+ column editor windows and the eyesight of a hawk. They may only have
265 a 1280 x 800 laptop which barely fits two 80x53 xterms side by side.
266 Consequently, having excessively long functions is also a hindrance to
267 others, as such developers with limited screen resources would need to
268 continuously page-up and page-down to read the code even of a single
271 This helps explain in part, below, why compliance with
272 [pep8](https://pep8.org/) is enforced, including its 80 character limit.
273 In short: not everyone has the same "modern" GUI workflow or has access
274 to the same computing resources as you, so please do respect that.
276 More on this concept is
277 [here](https://www.linuxjournal.com/content/line-length-limits).
278 Note *very pointedly* that Linus Torvalds *specifically* states that
279 he does not want Linux kernel development to become the exclusive
280 domain of the "wealthy". That means **no** to assumptions about
281 access to ultra-high resolution screens.
283 # Software prerequisites<a name="software-prerequisites"></a>
285 **Please make sure if you install manually that you install dependencies
286 in strict order. Failing to adhere to this will result in pip3 downloading
287 unauthorised older software versions. See
288 <http://lists.libre-soc.org/pipermail/libre-soc-dev/2021-September/003666.html>**
290 Whilst many resources online advocate "`sudo`" in front of all root-level
291 commands below, this quickly becomes tiresome. run "`sudo bash`", get a
292 root prompt, and save yourself some typing.
295 * apt-get install vim exuberant-ctags
296 * apt-get install build-essential
297 * apt-get install git python3.7 python3.7-dev python3-nose
298 * apt-get install graphviz xdot gtkwave
299 * apt-get install python3-venv
300 * apt-get install python-virtualenv # this is an alternative to python3-venv
301 * apt-get install tcl-dev libreadline-dev bison flex libffi-dev iverilog
302 * return to user prompt (ctrl-d)
304 (The above assumes that you are running Debian.)
306 This will get you python3 and other tools that are
307 needed. [graphviz](https://graphviz.org/) is essential
308 for showing the interconnections between cells, and
309 [gtkwave](http://gtkwave.sourceforge.net/) is essential for debugging.
311 If you would like to save yourself a lot more typing, check out the
312 [dev-env-setup](https://git.libre-soc.org/?p=dev-env-setup.git;a=summary)
313 repository, examine the scripts there and use them to automate much of
316 If you would like just to install only the apt dependencies use
317 [install-hdl-apt-reqs](https://git.libre-soc.org/?p=dev-env-setup.git;a=blob;f=install-hdl-apt-reqs;hb=HEAD) instead.
319 This page gives more details and a step by step process : [[HDL_workflow/devscripts]]
323 Look up good tutorials on how to use git effectively. There are so many
324 it is hard to recommend one. This is however essential. If you are not
325 comfortable with git, and you let things stay that way, it will seriously
326 impede development progress.
328 If working all day you should expect to be making at least two commits per
329 hour, so should become familiar with it very quickly. If you are *not*
330 doing around 2 commits per hour, something is wrong and you should read
331 the workflow instructions below more carefully, and also ask for advice
334 Worth noting: *this project does not use branches*. All code is committed
335 to master and we *require* that it be either zero-impact additions or that
336 relevant unit tests pass 100%. This ensures that people's work does not
337 get "lost" or isolated and out of touch due to major branch diversion,
338 and that people communicate and coordinate with each other.
340 This is not a hard rule: under special cirmstances branches can be useful.
341 They should not however be considered "routine".
343 For advice on commit messages see
344 [here](https://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html),
345 and [here](https://github.com/torvalds/subsurface-for-dirk/blob/master/README.md#contributing)).
349 Follow the source code (git clone) instructions here, do **not** use
350 the "stable" version (do not download the tarball):
351 <https://github.com/YosysHQ/yosys>
353 Or, alternatively, use the
354 [hdl-tools-yosys](https://git.libre-soc.org/?p=dev-env-setup.git;a=blob;f=hdl-tools-yosys;hb=HEAD)
355 script (which also installs symbiyosys and its dependencies)
357 Do not try to use a fixed revision of yosys (currently 0.9), nmigen is
358 evolving and frequently interacts with yosys.
360 [Yosys](https://github.com/YosysHQ/yosys is a framework for Verilog RTL.
361 [Verilog](https://en.wikipedia.org/wiki/Verilog) is a hardware description
363 RTL [Register Transfer
364 Level](https://en.wikipedia.org/wiki/Register-transfer_level)
365 models how data moves between
366 [registers](https://en.wikipedia.org/wiki/Hardware_register).
370 To install follow the [instructions
371 here](https://symbiyosys.readthedocs.io/en/latest/install.html)
372 Once done look at [A simple BMC
373 example](https://symbiyosys.readthedocs.io/en/latest/quickstart.html)
375 You do not have to install all of those (avy, boolector can be left
376 out if desired) however the more that are installed the more effective
377 the formal proof scripts will be (less resource utilisation in certain
380 [SymbiYosys](https://symbiyosys.readthedocs.io/en/latest/) (sby) is a
381 front-end driver program for Yosys-based formal hardware verification
386 *nmigen is a registered trademark of M-Labs <https://uspto.report/TM/88980893>*
388 **PLEASE NOTE: it is critical to install nmigen as the first dependency
389 prior to installing any further python-based Libre-SOC HDL repositories.
390 If "pip3 list" shows that nmigen has been auto-installed please remove it**
392 [nmigen](https://m-labs.hk/gateware/nmigen/) may be installed as follows:
396 * git clone https://gitlab.com/nmigen/nmigen.git
399 * python3 setup.py develop
402 Testing can then be carried out with "python3 setup.py test"
404 nmigen is a Python toolbox for building complex digital hardware.
406 ## Softfloat and sfpy
408 These are a test suite dependency for the
409 [ieee754fpu](https://www.gaisler.com/index.php/products/ipcores/ieee754fpu)
410 library, and will be changed in the future to use Jacob's
411 [simple-soft-float](https://crates.io/crates/simple-soft-float) library.
412 In the meantime, sfpy can be built as follows:
414 git clone --recursive https://github.com/billzorn/sfpy.git
417 git apply ../softposit_sfpy_build.patch
418 git apply /path/to/ieee754fpu/SoftPosit.patch
419 cd ../berkely-softfloat-3
420 # Note: Do not apply the patch included in sfpy for berkely-softfloat,
421 # it contains the same changes as this one
422 git apply /path/to/ieee754fpu/berkeley-softfloat.patch
425 # prepare a virtual environment for building
428 # or, if you prefer the old way:
429 # virtualenv -p python3 .env
431 # install dependencies
432 source .env/bin/activate
433 pip3 install --upgrade -r requirements.txt
438 make inplace -j$(nproc)
442 deactivate # deactivates venv, optional
443 pip3 install dist/sfpy*.whl
445 You can test your installation by doing the following:
448 >>> from sfpy import Posit8
451 It should print out `Posit8(1.3125)`
453 ## qemu, cross-compilers, gdb
455 As we are doing POWER ISA, POWER ISA compilers, toolchains and
456 emulators are required.
457 Again, if you want to save yourself some typing, use the dev scripts.
458 [install-hdl-apt-reqs](https://git.libre-soc.org/?p=dev-env-setup.git;a=blob;f=install-hdl-apt-reqs;hb=HEAD)
459 script will install the qemu;
460 [ppc64-gdb-gcc](https://git.libre-soc.org/?p=dev-env-setup.git;a=blob;f=ppc64-gdb-gcc;hb=HEAD)
461 script will install the toolchain and the corresponding debugger.
462 The steps are provided below only for reference; when in doubt,
463 consider checking and running the scripts.
465 Install powerpc64 gcc:
467 apt-get install gcc-8-powerpc64-linux-gnu
471 apt-get install qemu-system-ppc
473 Install gdb from source. Obtain the required tarball matching
474 the version of gcc (8.3) from here <https://ftp.gnu.org/gnu/gdb/>,
477 cd gdb-8.3 (or other location)
480 ../configure --srcdir=.. --host=x86_64-linux --target=powerpc64-linux-gnu
484 [gdb](https://en.wikipedia.org/wiki/GNU_Debugger) lets you debug running
485 programs. [qemu](https://www.qemu.org/) emulates processors, you can
486 run programs under qemu.
488 ## power-instruction-analyzer (pia)
490 We have a custom tool built in Rust by programmerjake to help analyze
491 the OpenPower instructions' execution on *actual* hardware.
495 curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh
497 Make sure we have the correct and up-to-date rust compiler (rustc & cargo):
499 rustup default stable
502 Install the Python extension from git source by doing the following:
504 git clone https://salsa.debian.org/Kazan-team/power-instruction-analyzer.git pia
506 ./libre-soc-install.sh
510 As this is an actual ASIC, we do not rely on an FPGA's JTAG TAP
511 interface, instead require a full complete independent implementation
512 of JTAG. Staf Verhaegen has one, with a full test suite, and it is
513 superb and well-written. The Libre-SOC version includes DMI (Debug
516 git clone https://git.libre-soc.org/git/c4m-jtag.git/
518 python3 setup.py develop
520 Included is an IDCODE tap point, Wishbone Master (for direct memory read
521 and write, fully independent of the core), IOPad redirection and testing,
522 and general purpose shift register capability for any custom use.
524 We added a DMI to JTAG bridge in LibreSOC which is
525 directly connected to the core, to access registers and
526 to be able to start and stop the core and change the PC.
527 In combination with the JTAG Wishbone interface the test
528 [ASIC](https://en.wikipedia.org/wiki/Application-specific_integrated_circuit)
529 can have a bootloader uploaded directly into onboard
530 [SRAM](https://en.wikipedia.org/wiki/Static_random-access_memory) and
533 [Chips4Makers](https://chips4makers.io/) make it possible for makers
534 and hobbyists to make their own open source chips.
536 [JTAG](https://en.wikipedia.org/wiki/JTAG) (Joint Test Action Group) is
537 an industry standard for verifying designs and testing printed circuit
538 boards after manufacture.
541 bus](https://en.wikipedia.org/wiki/Wishbone_%28computer_bus%29) is an open
542 source hardware computer bus intended to let the parts of an integrated
543 circuit communicate with each other.
546 See [[HDL_workflow/coriolis2]] page, for those people doing layout work.
550 A portable FPGA place and route tool.
552 See [[HDL_workflow/nextpnr]] page for installation instructions of nextpnr with ECP5 support for Lattice FPGA ECP5 series. Also see
553 [[HDL_workflow/ECP5_FPGA]] for connecting up to JTAG with a ULX3S
554 and the Lattice VERSA_ECP5.
558 An open source place and route framework for Xilinx FPGAs using Project Xray. We will use it for Xilinx 7-series FPGAs like Artix-7.
560 One of the ways to get Arty A7 100t Digilent FPGA board working.
562 See [[HDL_workflow/nextpnr-xilinx]] for installation instructions and dependencies.
567 The fastest Verilog and SystemVerilog simulator. It compiles Verilog to C++ or SystemC.
569 Advise use only v4.106 at the moment.
571 See [[HDL_workflow/verilator]] page for installation instructions.
575 GHDL is a shorthand for G Hardware Design Language. It is a VHDL analyzer, compiler, simulator and (experimental) synthesizer that can process (nearly) any VHDL design.
577 VHDL is an acronym for Very High Speed Integrated Circuit (VHSIC) Hardware Description Language (HDL), which is a programming language used to describe a logic circuit by function, data flow behavior, or structure.
579 Unlike some other simulators, GHDL is a compiler: it directly translates a VHDL file to machine code, without using an intermediary language such as C or C++. Therefore, the compiled code should be faster and the analysis time should be shorter than with a compiler using an intermediary language.
581 GHDL aims at implementing VHDL as defined by IEEE 1076. It supports the 1987, 1993 and 2002 revisions and, partially, 2008. PSL is also partially supported.
583 See [[HDL_workflow/ghdl]] page for installation instructions.
587 Icarus Verilog is a Verilog simulation and synthesis tool. It operates as a compiler, compiling source code written in Verilog (IEEE-1364) into some target format.
589 See [[HDL_workflow/iverilog]] page for installation instructions.
593 cocotb is a COroutine based COsimulation TestBench environment for verifying VHDL and SystemVerilog RTL using Python.
595 See [[HDL_workflow/cocotb]] page for installation instructions.
599 A fully open source toolchain for the development of FPGAs. Currently it targets Xilinx 7-series, Lattice iCE40 and ECP5, Quicklogic EOS S3.
601 One way to get the Arty A7 100t Digilent FPGA board working.
603 See [[HDL_workflow/symbiflow]] for installation instructions
606 ## FPGA/Board Boot-Loaders-Programmers
608 Open source FPGA/Board boot-loaders and programmers for ULX3S, ECP5 and
611 Currently these programs dfu-util, openFPGALoader, ujprog, fujprog,
612 xc3sprog and ecpprog are going to be used.
614 See [[HDL_workflow/fpga-boot-loaders-progs]] for installation instructions and dependencies.
616 ## ls2 peripheral fabric
620 # Registering for git repository access<a name="gitolite3_access"></a>
622 After going through the onboarding process and having agreed to take
623 responsibility for certain tasks, ask on the mailing list for git
624 repository access, sending in a public key (`id_rsa.pub`). If you do
625 not have one then generate it with `ssh-keygen -t rsa`. You will find it
628 NEVER SEND ANYONE THE PRIVATE KEY. By contrast the public key, on
629 account of being public, is perfectly fine to make... err... public.
631 Create a file `~/.ssh/config` with the following lines:
633 Host git.libre-soc.org
636 Test that you have access with this command:
638 ssh -v -p922 gitolite3@git.libre-soc.org
640 Please note: **DO NOT TYPE A PASSWORD** - the server gets hit by a lot of
641 port-scanning, and detection of password failures are used to instantly
644 Wait for the Project Admin to confirm that the ssh key has been added
645 to the required repositories. Once confirmed, you can clone any of the
646 repos at https://git.libre-soc.org/:
648 git clone gitolite3@git.libre-soc.org:REPONAME.git
650 Alternatively, the .ssh/config can be skipped and this used:
652 git clone ssh://gitolite3@git.libre-soc.org:922/REPONAME.git
654 Note: **DO NOT ATTEMPT TO LOG IN TO THE SERVER WITH A PERSONAL ACCOUNT**.
655 fail2ban is running and, due to repeated persistent port-scanning spammers
656 is set up to instantly ban any unauthorised ssh access for up to two weeks.
657 This keeps log file sizes down on the server (which is resource-constrained).
658 If you are wondering why this is done, it's a *lot* of port-scans.
660 Therefore, *only* ssh in to server with the gitolite3 account, *only*
661 on port 922, and *only* once the systems administrator has given you
662 the all-clear that the ssh key has been added.
666 Although there are methods online which describe how (and why) these
667 settings are normally done, honestly it is simpler and easier to open
668 ~/.gitconfig and add them by hand.
670 core.autocrlf is a good idea to ensure that anyone adding DOS-formatted
671 files they don't become a pain. pull.rebase is something that is greatly
672 preferred for this project because it avoids the mess of "multiple
673 extra merge git tree entries", and branch.autosetuprebase=always will,
674 if you want it, always ensure that a new git checkout is set up with rebase.
683 autosetuprebase = always
685 # Checking out the HDL repositories
687 Before running the following, install the
688 dependencies. This is easiest done with this script
689 <https://git.libre-soc.org/?p=dev-env-setup.git;a=blob;f=install-hdl-apt-reqs;hb=HEAD>
691 **It is critically important to install these in STRICT order, otherwise
692 pip3 interferes and performs unauthorised downloads without informing
693 you of what it is doing**.
697 * git clone https://gitlab.com/nmigen/nmigen
698 * git clone https://gitlab.com/nmigen/nmigen-boards
699 * git clone https://gitlab.com/nmigen/nmigen-soc
700 * git clone https://gitlab.com/nmigen/nmigen-stdio
701 * git clone gitolite3@git.libre-soc.org:c4m-jtag.git
702 * git clone gitolite3@git.libre-soc.org:nmutil.git
703 * git clone gitolite3@git.libre-soc.org:openpower-isa.git
704 * git clone gitolite3@git.libre-soc.org:ieee754fpu.git
705 * git clone gitolite3@git.libre-soc.org:soc.git
707 In each of these directories, **in the order listed**, track down the
708 `setup.py` file, then, as root (`sudo bash`), run the following:
710 * python3 setup.py develop
712 The reason for using "develop" mode is that the code may be edited
713 in-place yet still imported "globally". There are variants on this theme
714 for multi-user machine use however it is often just easier to get your
715 own machine these days.
717 The reason for the order is because soc depends on ieee754fpu, and
718 ieee754fpu depends on nmutil. If you do not follow the listed order
719 pip3 will go off and download an arbitrary version without your
722 If "`python3 setup.py install`" is used it is a pain: edit, then
723 install. edit, then install. It gets extremely tedious, hence why
724 "develop" was created.
726 If you prefer you can use this script instead: of course you checked it
727 in advance and accept full responsibility.
728 <https://git.libre-soc.org/?p=dev-env-setup.git;a=blob;f=hdl-dev-repos;hb=HEAD>
734 * new members, add yourself to the [[about_us]] page and create yourself
735 a home page using someone else's page as a template.
736 * communicate on the mailing list or the bugtracker an intent to take
737 responsibility for a particular task.
738 * assign yourself as the bug's owner
739 * *keep in touch* about what you are doing, and why you are doing it.
740 * edit your home page regularly, particularly to track tasks so that
741 they can be paid by NLNet.
742 * if you cannot do something that you have taken responsibility for,
743 then unless it is a dire personal emergency please say so, on-list. we
744 won't mind. we'll help sort it out.
746 Regarding the above it is important that you read, understand, and agree
747 to the [[charter]] because the charter is about ensuring that we operate
748 as an effective organisation. It's *not* about "setting rules and meting
753 for actual code development
757 * plan in advance to write not just code but a full test suite for
758 that code. **this is not optional**. large python projects that do not
759 have unit tests **FAIL** (see separate section below).
760 * Prioritise writing formal proofs and a single clear unit test that is more
761 like a "worked example".
762 We receive NLNet funds for writing formal proofs, plus they
763 cover corner cases and take far less time to write
765 ### Commit tested or zero-dependent code
767 * only commit code that has been tested (or is presently unused). other
768 people will be depending on you, so do take care not to screw up.
769 not least because, as it says in the [[charter]] it will be your
770 responsibility to fix. that said, do not feel intimidated: ask for help
771 and advice, and you'll get it straight away.
775 * commit often. several times a day, and "git push" it. this is
776 collaboration. if something is left even overnight uncommitted and not
777 pushed so that other people can see it, it is a red flag.
779 yourself thinking "i'll commit it when it's finished" or "i don't want to
780 commit something that people might criticise" *this is not collaboration*,
781 it is making yourself a bottleneck. pair-programming is supposed to help
782 avoid this kind of thing however pair-programming is difficult to organise
783 for remote collaborative libre projects (suggestions welcomed here)
785 ### Enable editor auto-detection of file changes by external programs
787 This is important. "`git pull`" will merge in changes. If you then
788 arbitrarily save a file without re-loading it, you risk destroying
791 You can avoid damaging the repositories by following some simple procedures:
793 run appropriate unit tests
795 run appropriate unit tests again (checks other people's work)
796 git diff # and actually read and review the output
797 git status # check for any missing files
798 git commit # with appropriate arguments and message
799 git push # always always always do this
801 ### Absolutely no auto-generated output
803 * **do not commit autogenerated output**. write a shell script and commit
804 that, or add a `Makefile` to run the command that generates the output, but
805 **do not** add the actual output of **any** command to the repository.
806 ever. this is really important. even if it is a human-readable file
807 rather than a binary object file.
808 * it is very common to add PDFs (the result of running `latex2pdf`) or
809 configure.in (the result of running `automake`), they are an absolute
810 nuisance and interfere hugely with git diffs, as well as waste hard
811 disk space *and* network bandwidth. don't do it.
812 * do not add multi-megabyte or multi-gigabyte "test data".
813 use shell scripts and commit that, which automatically downloads the
814 "test data" from a well-known known-good reliable location instead.
816 ### Write commands that do tasks and commit those
818 * if the command needed to create any given autogenerated output is not
819 currently in the list of known project dependencies, first consult on
820 the list if it is okay to make that command become a hard dependency of
821 the project (hint: java, node.js php and .NET commands may cause delays
822 in response time due to other list participants laughing hysterically),
823 and after a decision is made, document the dependency and how its source
824 code is obtained and built (hence why it has to be discussed carefully)
825 * if you find yourself repeating commands regularly, chances are high
826 that someone else will need to run them, too. clearly this includes
827 yourself, therefore, to make everyone's lives easier including your own,
828 put them into a `.sh` shell script (and/or a `Makefile`), commit them to
829 the repository and document them at the very minimum in the README,
830 INSTALL.txt or somewhere in a docs folder as appropriate. if unsure,
831 ask on the mailing list for advice.
833 ### Keep commits single-purpose
835 * edit files making minimal *single purpose* modifications (even if
836 it involves multiple files. Good extreme example: globally changing
837 a function name across an entire codebase is one purpose, one commit,
838 yet hundreds of files. miss out one of those files, requiring multiple
839 commits, and it actually becomes a nuisance).
841 ### Run unit tests prior to commits
843 * prior to committing make sure that relevant unit tests pass, or that
844 the change is a zero-impact addition (no unit tests fail at the minimum)
846 ### Do not break existing code
848 * keep working code working **at all times**. find ways to ensure that
849 this is the case. examples include writing alternative classes that
850 replace existing functionality and adding runtime options to select
851 between old and new code.
853 ### Small commits with relevant commit message
855 * commit no more than around 5 to 10 lines at a time, with a CLEAR message
856 (no "added this" or "changed that").
857 * if as you write you find that the commit message involves a *list* of
858 changes or the word "and", then STOP. do not proceed: it is a "red flag"
859 that the commit has not been properly broken down into separate-purpose
860 commits. ask for advice on-list on how to proceed.
862 ### Exceptions to small commit: atomic single purpose commit
864 * if it is essential to commit large amounts of code, ensure that it
865 is **not** in use **anywhere** by any other code. then make a *small*
866 (single purpose) followup commit which actually puts that code into use.
868 This last rule is kinda flexible, because if you add the code *and* add
869 the unit test *and* added it into the main code *and* ran all relevant
870 unit tests on all cascade-impacted areas by that change, that's perfectly
871 fine too. however if it is the end of a day, and you need to stop and
872 do not have time to run the necessary unit tests, do *not* commit the
873 change which integrates untested code: just commit the new code (only)
874 and follow up the next day *after* running the full relevant unit tests.
876 ### Why such strict rules?
878 The reason for all the above is because python is a dynamically typed
879 language. make one tiny change at the base level of the class hierarchy
880 and the effect may be disastrous.
882 It is therefore worth reiterating: make absolutely certain that you *only*
883 commit working code or zero-impact code.
885 Therefore, if you are absolutely certain that a new addition (new file,
886 new class, new function) is not going to have any side-effects, committing
887 it (a large amount of code) is perfectly fine.
889 As a general rule, however, do not use this an an excuse to write code
890 first then write unit tests as an afterthought. write *less* code *in
891 conjunction* with its (more basic) unit tests, instead. then, folliw up with
892 additions and improvements.
894 The reason for separating out commits to single purpose only becomes
895 obvious (and regretted if not followed) when, months later, a mistake
896 has to be tracked down and reverted. if the commit does not have an
897 easy-to-find message, it cannot even be located, and once found, if the
898 commit confuses several unrelated changes, not only the diff is larger
899 than it should be, the reversion process becomes extremely painful.
901 ### PHP-style python format-strings
903 As the name suggests, "PHP-style" is not given as a compliment.
904 Format-strings - `f"{variable} {pythoncodefragment}" are a nightmare
905 to read. The lesson from PHP, Zope and Plone: when code is embedded,
906 the purpose of the formatting - the separation of the format from
907 the data to be placed in it - is merged, and consequently become
910 By contrast, let us imagine a situation where 12 variables need to
911 be inserted into a string, four of which are the same variablename:
913 x = "%s %s %s %s %s %s %s %s %s %s %s %s" % (var1, var2, var3,
918 This is just as unreadable, but for different reasons. Here it *is*
919 useful to do this as:
921 x = f"{var1} {var2} {var3}" \
923 f"{var3} {var4} {var1}"
925 As a general rule, though, format-specifiers should be strongly
926 avoided, given that they mix even variable-names directly inside
929 This additionally gives text editors (and online web syntax
930 highlighters) the opportunity to colour syntax-highlight the
931 ASCII string (the format) from the variables to be inserted *into*
932 that format. gitweb for example (used by this project) cannot
933 highlight string-formatted code.
935 It turns out that colour is processed by the **opposite** hemisphere
936 of the brain from written language. Thus, colour-syntax-highlighting
937 is not just a "nice-to-have", it's **vital** for easier and faster
938 identification of context and an aid to rapid understanding.
940 Anything that interferes with that - such as python format-strings -
941 has to take a back seat, regardless of its perceived benefits.
943 **If you absolutely must** use python-format-strings, **only** do
944 so by restricting to variables. Create temporary variables if you
952 * all code needs to conform to pep8. use either pep8checker or better
953 run autopep8. however whenever committing whitespace changes, *make a
954 separate commit* with a commit message "whitespace" or "autopep8 cleanup".
955 * pep8 REQUIRES no more than 80 chars per line. this is non-negotiable. if
956 you think you need greater than 80 chars, it *fundamentally* indicates
957 poor code design. split the code down further into smaller classes
960 ### Docstring checker
962 * TBD there is a docstring checker. at the minimum make sure to have
963 an SPD license header, module header docstring, class docstring and
964 function docstrings on at least non-obvious functions.
966 ### Clear code commenting and docstrings
968 * make liberal but not excessive use of comments. describe a group of
969 lines of code, with terse but useful comments describing the purpose,
970 documenting any side-effects, and anything that could trip you or other
971 developers up. unusual coding techniques should *definitely* contain
974 ### Only one class per module (ish)
976 * unless they are very closely related, only have one module (one class)
977 per file. a file only 25 lines long including imports and docstrings
978 is perfectly fine however don't force yourself. again, if unsure,
981 ### File and Directory hierarchy
983 * *keep files short and simple*. see below as to why
984 * create a decent directory hierarchy but do not go mad. ask for advice
989 * please do not use "from module import \*". it is extremely bad practice,
990 causes unnecessary resource utilisation, makes code readability and
991 tracking extremely difficult, and results in unintended side-effects.
993 Example: often you want to find the code from which a class was imported.
994 nirmally you go to the top of the file, check the imports, and you know
995 exactly which file has the class because of the import path. by using
996 wildcards, you have absolutely *no clue* which wildcard imported which
999 Example: sometimes you may accidentally have duplicate code maintained
1000 in two or more places. editing one of them you find, puzzlingly, that
1001 the code behaves in some files with the old behaviour, but in others it
1002 works. after a massive amount of investigation, you find that the working
1003 files happen to have a wildcard import of the newer accidental duplicate
1004 class **after** the wildcard import of the older class with exactly the
1005 same name. if you had used explicit imports, you would have spotted
1006 the double import of the class from two separate locations, immediately.
1008 Really. don't. use. wildcards.
1010 More about this here:
1012 * <https://www.asmeurer.com/removestar/>
1013 * <https://rules.sonarsource.com/python/RSPEC-2208>
1015 ### Keep file and variables short but clear
1017 * try to keep both filenames and variable names short but not ridiculously
1018 obtuse. an interesting compromise on imports is "from ridiculousfilename
1019 import longsillyname as lsn", and to assign variables as well: "comb =
1020 m.d.comb" followed by multiple "comb += nmigen_stmt" lines is a good trick
1021 that can reduce code indentation by 6 characters without reducing clarity.
1023 Additionally, use comments just above an obtuse variable in order to
1024 help explain what it is for. In combination with keeping the the module
1025 itself short, other readers will not need to scroll back several pages
1026 in order to understand the code.
1028 Yes it is tempting to actually use the variables as
1029 self-explanatory-comments and generally this can be extremely good
1030 practice. the problem comes when the variable is so long that a function
1031 with several parameters csn no longer fit on a single line, and takes
1032 up five to ten lines rather than one or two. at that point, the length
1033 of the code is adversely affected and thus so is readability by forcing
1034 readers to scroll through reams of pages.
1036 It is a tricky balance: basically use your common sense, or just ask
1037 someone else, "can you understand this code?"
1039 ### Reasons for code structure
1041 Regarding code structure: we decided to go with small modules that are
1042 both easy to analyse, as well as fit onto a single page and be readable
1043 when displayed as a visual graph on a full UHD monitor. this is done
1046 * using the capability of nmigen (TODO crossref to example) output the
1047 module to a yosys ilang (.il) file
1048 * in a separate terminal window, run yosys
1049 * at the yosys prompt type "read_ilang modulename.il"
1050 * type "show top" and a graphviz window should appear. note that typing
1051 show, then space, then pressing the tab key twice will give a full list
1052 of submodules (one of which will be "top")
1054 You can now fullsize the graphviz window and scroll around. if it looks
1055 reasonably obvious at 100% zoom, i.e the connections can be clearly
1056 related in your mind back to the actual code (by matching the graph names
1057 against signals and modules in the original nmigen code) and the words are
1058 not tiny when zoomed out, and connections are not total incomprehensible
1059 spaghetti, then congratulations, you have well-designed code. If not,
1060 then this indicates a need to split the code further into submodules
1061 and do a bit more work.
1063 The reasons for doing a proper modularisation job are several-fold:
1065 * firstly, we will not be doing a full automated layout-and-hope
1066 using alliance/coriolis2, we will be doing leaf-node thru tree node
1067 half-automated half-manual layout, finally getting to the floorplan,
1068 then revising and iteratively adjusting.
1069 * secondly, examining modules at the gate level (or close to it) is just
1070 good practice. poor design creeps in by *not* knowing what the tools
1071 are actually doing (word to experienced developers: yes, we know that
1072 the yosys graph != final netlist).
1073 * thirdly, unit testing, particularly formal proofs, is far easier on
1074 small sections of code, and complete in a reasonable time.
1076 ## Special warning / alert to vim users!
1078 Some time around the beginning of 2019 some bright spark decided that
1079 an "auto-recommend-completion-of-stuff" option would be a nice, shiny
1080 idea to enable by default from that point onwards.
1082 This incredibly annoying "feature" results in tabs (or spaces) being
1083 inserted "on your behalf" when you press return on one line, for your
1084 "convenience" of not needing to type lots of spaces/tabs just to get
1085 to the same indentation level.
1087 Of course, this "feature", if you press return on one line in edit
1088 mode and then press "escape", leaves a bundle-of-joy extraneous
1089 whitespace **exactly** where you don't want it, and didn't ask for it,
1090 pooped all over your file.
1092 Therefore, *please*: **before** running "git commit", get into the
1093 habit of always running "git diff", and at the very minimum
1094 speed-skim the entire diff, looking for tell-tale "red squares"
1095 (these show up under bash diff colour-syntax-highlighting) that
1096 inform you that, without your knowledge or consent, vim has
1097 "helpfully" inserted extraneous whitespace.
1099 Remove them **before** git committing because they are not part
1100 of the actual desired code-modifications, and committing them
1101 is a major and constant distraction for reviewers about actual
1102 important things like "the code that actually *usefully* was
1103 modified for that commit"
1105 This has the useful side-effect of ensuring that, right before
1106 the commit, you've got the actual diff right in front of you
1107 in the xterm window, on which you can base the "commit message".
1111 For further reading, see the wikipedia page on
1112 [Test-driven Development](https://en.wikipedia.org/wiki/Test-driven_development)
1114 This deserves its own special section. It is extremely important to
1115 appreciate that without unit tests, python projects are simply unviable.
1116 Python itself has over 25,000 individual tests.
1118 This can be quite overwhelming to a beginner developer, especially one
1119 used to writing scripts of only 100 lines in length.
1121 Thanks to Samuel Falvo we learned that writing unit tests as a formal
1122 proof is not only shorter, it's also far more readable and also, if
1123 written properly, provides 100% coverage of corner-cases that would
1124 otherwise be overlooked or require tens to hundreds of thousands of
1127 No this is not a joke or even remotely hypothetical, this is an actual
1130 The ieee754fpu requires several hundreds of thousands of tests to be
1131 run (currently needing several days to run them all), and even then we
1132 cannot be absolutely certain that all possible combinations of input have
1133 been tested. With 2^128 permutations to try with 2 64 bit FP numbers
1134 it is simply impossible to even try.
1136 This is where formal proofs come into play.
1138 Samuel illustrated to us that "ordinary" unit tests can then be written
1139 to *augment* the formal ones, serving the purpose of illustrating how
1140 to use the module, more than anything.
1142 However it is appreciated that writing formal proofs is a bit of a
1143 black art. This is where team collaboration particularly kicks in,
1144 so if you need help, ask on the mailing list.
1146 ## Don't comment out unit tests: add them first (as failures) and fix code later
1148 Unit tests serve an additional critical purpose of keeping track of code
1149 that needs to be written. In many cases, you write the unit test *first*,
1150 despite knowing full well that the code doesn't even exist or is completely
1151 broken. The unit test then serves as a constant and important reminder
1152 to actually fix (or write) the code.
1154 Therefore, *do not* comment out unit tests just because they "don't work".
1155 If you absolutely must stop a unit test from running, **do not delete it**.
1156 Simply mark it with an appropriate
1157 ["skip" decorator](https://docs.python.org/3/library/unittest.html#skipping-tests-and-expected-failures),
1158 preferably with a link to a URL in the [bugtracker](https://bugs.libre-soc.org/)
1159 with further details as to why the unit test should not be run.
1161 # Task management guidelines
1163 1. Create the task in appropriate "Product" section with appropriate
1164 "Component" section. Most code tasks generally use "Libre-SOC's
1166 2. Fill in "Depends on" and "Blocks" section whenever appropriate.
1167 Also add as many related ("See Also") links to other bugreports
1168 as possible. bugreports are never isolated.
1169 3. Choose the correct task for a budget allocation. Usually the parent
1171 4. Choose the correct NLnet milestone. The best practice is to check
1172 the parent task for a correct milestone.
1173 5. Assign the budget to the task in `"USER=SUM"` form, where "USER"
1174 corresponds to your username and "SUM" corresponds to the actual
1175 budget in EUR. There may be multiple users.
1176 6. When the task is completed, you can begin writing an RFP.
1177 **DO NOT submit it without explicit authorisation and review**.
1178 Leave out your bank and personal address details if you prefer
1179 when sending to the Team Manager for review.
1180 7. Once the RFP is written, notify the Team Manager and obtain their
1181 explicit approval to send it.
1182 8. Once approval is received and the RFP sent, update the `"USER=SUM"`
1183 field to include the submitted date:
1184 `"USER={amount=SUM, submitted=SDATE}"`. The SDATE is entered in
1186 9. Once the task is paid, again notify the Team Manager (IRC is fine),
1187 and update `"USER={amount=SUM, submitted=SDATE}"`
1188 to `"USER={amount=SUM, submitted=SDATE, paid=PDATE}"`. The PDATE is
1189 entered in `YYYY-MM-DD` form, too.
1191 Throughout all of this you should be using budget-sync to check the
1192 database consistency
1193 <https://git.libre-soc.org/?p=utils.git;a=blob;f=README.txt;hb=HEAD>
1195 [[!img bugzilla_RFP_fields.jpg size=640x ]]
1199 Find appropriate tutorials for nmigen and yosys, as well as symbiyosys.
1201 * Robert Baruch's nmigen tutorials look really good:
1202 <https://github.com/RobertBaruch/nmigen-tutorial>
1203 * Although a verilog example this is very useful to do
1204 <https://symbiyosys.readthedocs.io/en/latest/quickstart.html#first-step-a-simple-bmc-example>
1205 * This tutorial looks pretty good and will get you started
1206 <https://web.archive.org/web/20210123052724/http://blog.lambdaconcept.com/doku.php?id=nmigen:nmigen_install>
1207 and walks not just through simulation, it takes you through using
1209 * There exist several nmigen examples which are also executable
1210 <https://gitlab.com/nmigen/nmigen/tree/master/examples/> exactly as
1211 described in the above tutorial (python3 filename.py -h)
1212 * More nmigen tutorials at [[learning_nmigen]]