I am modifying openlane scripts tcl commands all tcl and jus open-source-silicon.dev #openlane

I am modifying openlane/scripts/tcl_commands/all.t...

Matthew Guthaus

11/12/2021, 9:37 PM

I am modifying openlane/scripts/tcl_commands/all.tcl and just adding a puts or puts_info output in a function. This never gets shown during my run... I'm really confused why. If I look at the file in docker (using make mount) it is there. I'm sure that the code is being executed because it is the only function named save_state and outputs "Saving Runtime Environment". Neither of these exists anywhere in openlane. @User @User?

Copy code

proc save_state {args} {
    set ::env(INIT_ENV_VAR_ARRAY) [split [array names ::env] " "]
    puts_info "FOOSaving Runtime Environment"

and definitely gets called:

Copy code

[INFO]: Saving Runtime Environment
invalid command name "17"
    while executing
"17"
    ("uplevel" body line 1)
    invoked from within
"uplevel #0 ${cmd}"
    (procedure "set_log" line 3)
    invoked from within
"set_log ::env($index) $escaped_env_var $::env(GLB_CFG_FILE) 1"
    (procedure "save_state" line 9)
    invoked from within
"save_state"
    (procedure "flow_fail" line 6)
    invoked from within
"flow_fail"
    (procedure "try_catch" line 25)
    invoked from within

Mitch Bailey

11/13/2021, 12:36 AM

@Matthew Guthaus Don't know if this helps, but I have several OpenLane/openlane directories. Can you check that

OPENLANE_ROOT

is pointing to the directory hierarchy that you're modifying? I made changes to

save_state

and

prep

and they were reflected in the docker run. I guess if you're already in docker, you could check

/openLANE_flow/scripts/tcl_commands/all.tcl

to see if the changes are actually there.

Matthew Guthaus

11/13/2021, 1:00 AM

@Mitch Bailey yeah, it definitely is. I logged into the docker image to confirm this

Matthew Guthaus

11/13/2021, 1:00 AM

I see the changes in the /openLANE_flow directory even!

Matthew Guthaus

11/13/2021, 1:02 AM

Actually, in docker it is now just /openlane

Matthew Guthaus

11/13/2021, 1:04 AM

@Mitch Bailey do you still see /openLANE_flow and not /openlane?

Matthew Guthaus

11/13/2021, 1:05 AM

Can confirm, mpw-two tag had /openLANE_flow, but mpw-3a has /openlane

Mitch Bailey

11/13/2021, 1:08 AM

Good point! I'm still mainly working with mpw-two. Maybe that's why I'm not seeing the same results. Let me switch to mpw-3a and see what I get.

Mitch Bailey

11/13/2021, 10:08 AM

@Matthew Guthaus I modified the

spm

to use a bus clock and was able to duplicate the problem. It'll I think what happens is that the

<design>/config.tcl

has the definition

set ::env(CLOCK_PORT) {clk[1]}

, however when then variable is written to

<design>/runs/<tag>/config.tcl

it's converted to ``set ::env(CLOCK_PORT) "clk[1]"` . Maybe you've already come up with a solution but it looks like If you change

scripts/utils/utils.tcl

like this

Copy code

proc set_log {var val filepath log_flag} {
-        set cmd "set ${var} \"${val}\""
+       if { [string first {[} "$val"] != -1 } {
+               set cmd "set ${var} \{${val}\}"
+       } else {
+               set cmd "set ${var} \"${val}\""
+       }
         uplevel #0 ${cmd}
         set global_cfg_file [open $filepath a+]

it works with

set ::env(CLOCK_PORT) {clk[1]}

. This will quote with

{}

only the variables that contain

. You can't quote all variable with

{}

because the python report program parses the

config.tcl

file and can't handle

{}

for the values that it needs. Shall I do a PR?

👍 1

Mitch Bailey

11/13/2021, 10:10 AM

Still don't know why your changes aren't reflected. Are you using

flow.tcl

caravel_user_project/Makefile

Maximo Balestrini

11/13/2021, 11:43 AM

I checked my caravel_user_project Makefile (using

make -n xxxx

) and it seems to be mounting my openlane directory in the docker's`/openLANE_Flow`, but I guess it then uses the one in

/openlane

so it doesn't see the changes.

Matthew Guthaus

11/13/2021, 3:34 PM

I'm using the Makefile to run the flow. I only see /openlane in the newer images as it replaces the older directory. It is my OpenLane installation mounted with the changes so I'm not sure how the changes aren't run. I'll play with it more today ..

donn

11/13/2021, 3:35 PM

All I can say is, like two weeks ago, the OpenLane Makefile has been updated to use just

/openlane

instead of

/openLANE_flow

. If the caravel makefile still tries to mount to

/openLANE_flow

, that’s a problem.

donn

11/13/2021, 3:36 PM

By a problem I mean a bug report

Matthew Guthaus

11/13/2021, 4:02 PM

@Mitch Bailey Yeah, I think that should be a PR. I'm wondering if there are more cases that are missed though. What do you mean about the "python report program"?

Mitch Bailey

11/13/2021, 4:11 PM

[ERROR]: during executing: "openroad -python /openlane/scripts/generate_reports.py

It looks like it tries to calculate utilization or something from 2 of the variable in the saved config.tcl file. I guess, since it's python, it can convert

quoted strings, but not

{}

quoted strings.

Matthew Guthaus

11/13/2021, 6:26 PM

This literally shows that my scripts don't match what is run:

Copy code

[INFO]: Saving Runtime Environment
invalid command name "17"
    while executing
"17"
    ("uplevel" body line 1)
    invoked from within
"uplevel #0 ${cmd}"
    (procedure "set_log" line 3)
    invoked from within
"set_log ::env($index) $escaped_env_var $::env(GLB_CFG_FILE) 1"
    (procedure "save_state" line 9)
    invoked from within
"save_state"
    (procedure "flow_fail" line 6)
    invoked from within
"flow_fail"
    (procedure "try_catch" line 25)
    invoked from within
"try_catch $::env(OPENROAD_BIN) -exit $::env(SCRIPTS_DIR)/openroad/or_opendp.tcl |& tee $::env(TERMINAL_OUTPUT) [index_file $::env(opendp_log_file_tag)..."
    (procedure "detailed_placement_or" line 6)
    invoked from within
"detailed_placement_or"
    (procedure "run_routing" line 32)
    invoked from within
"run_routing"
    (procedure "run_routing_step" line 10)
    invoked from within
"[lindex $step_exe 0] [lindex $step_exe 1] "
    (procedure "run_non_interactive_mode" line 43)
    invoked from within
"run_non_interactive_mode {*}$argv"
    invoked from within
"if { [info exists flags_map(-interactive)] || [info exists flags_map(-it)] } {
        puts_info "Running interactively"
        if { [info exists arg_values(-file)..."
    (file "/openlane/flow.tcl" line 356)
make[1]: *** [Makefile:43: user_project_wrapper] Error 1
make[1]: Leaving directory '/home/mrg/demo/openram_testchip/openlane'
make: *** [Makefile:70: user_project_wrapper] Error 2
mrg@diode ~/demo/openram_testchip (main)$ echo $OPENLANE_ROOT
/home/mrg/demo/openlane
mrg@diode ~/demo/openram_testchip (main)$ cd $OPENLANE_ROOT
mrg@diode ~/demo/openlane ((HEAD detached at mpw-3a))$ make mount
cd /home/mrg/demo/openlane && \
        docker run -it --rm -v /home/mrg/demo/openlane:/openlane -v /home/mrg/demo:/home/mrg/demo -e PDK_ROOT=/home/mrg/demo  --user 1000:1000 efabless/openlane:mpw-3a
bash-4.2$ grep FOO /openlane/scripts/tcl_commands/all.tcl
    puts_info "FOOSaving Runtime Environment"

Matthew Guthaus

11/13/2021, 6:26 PM

Note: "Saving" vs "FOOSaving"

Matthew Guthaus

11/13/2021, 6:27 PM

(I also had your patch which wasn't found either)

Matthew Guthaus

11/13/2021, 7:57 PM

I can replicate this very quickly with this change. At line 330 of flow.tcl add:

puts_info {HIHIHI}

exit 1

It should exit before it runs the flow, but mine doesn't! Yet the change is shown in flow.tcl inside docker.

Matthew Guthaus

11/13/2021, 8:36 PM

I've finally figured this out and it is an openlane/caravel bug. The mpw-3a docker image has a pre-installed copy of all the openlane scripts in /openlane. The caravel Makefile is supposed to mount the copy from $OPENLANE_ROOT in /openLANE_flow (this is with the mpw-3a caravel tag):

Copy code

docker run -it -v $(OPENLANE_ROOT):/openLANE_flow \
                -v $(PDK_ROOT):$(PDK_ROOT) \
                -v $(PWD)/..:/project \
                -v $(CARAVEL_ROOT):$(CARAVEL_ROOT) \
                -e PDK_ROOT=$(PDK_ROOT) \
                -e CARAVEL_ROOT=$(CARAVEL_ROOT) \
                -u $(shell id -u $(USER)):$(shell id -g $(USER)) \
                $(OPENLANE_IMAGE_NAME) sh -c $(OPENLANE_BASIC_COMMAND);\

So, there are TWO COPIES of the openlane scripts when running a design using the Caravel Makefile. One in /openlane and one in /openLANE_flow. The one that ships with the docker image is used according to the command line:

Copy code

OPENLANE_BASIC_COMMAND = "cd /project/openlane && flow.tcl -design ./$* -save_path .. -save -tag $* -overwrite"

but that is a mistake! @User @User

Matthew Guthaus

11/13/2021, 8:39 PM

@Jesse Cirimelli-Low @Hadir Khan ^^ FYI

Maximo Balestrini

11/13/2021, 8:43 PM

Yes, I think the problem is that they didn't update the docker mounting point on caravel from openLANE_flow to openlane in the Makefiles. When you do that the openlane version inside the docker gets "overwritten" by yours

Matthew Guthaus

11/13/2021, 8:44 PM

Argh, 1.5 days 😞

Matthew Guthaus

11/13/2021, 8:46 PM

Can confirm. Fixing that mount fixes the problem as it over-rides with version that ships in the docker image.

Matthew Guthaus

11/13/2021, 8:47 PM

@jeffdi This should be fixed in caravel as it may be screwing up a bunch of people... They aren't using the version they think they are.

Matthew Guthaus

11/13/2021, 8:51 PM

I submitted a PR.

Matthew Guthaus

11/13/2021, 8:58 PM

Actually, it's worse than that. It always use the relative copy of openlane at ../openlane since it also mounts "-v $(PWD)/..:/project"

Matthew Guthaus

11/13/2021, 9:05 PM

and executes:

Copy code

OPENLANE_BASIC_COMMAND = "cd /project/openlane && flow.tcl -design ./$* -save_path .. -save -tag $* -overwrite"

Matthew Guthaus

11/13/2021, 9:11 PM

I should tag @Manar Abdelatty as well ^^

Matthew Guthaus

11/13/2021, 9:42 PM

Ah, looks like /project/openlane is just the working directory. The $PATH means that flow.tcl resolves to /openlane/flow.tcl. However, the mount point for OPENLANE_ROOT should be to /openlane not /openLANE_flow to allow this to be set properly.

Matthew Guthaus

11/13/2021, 9:48 PM

https://github.com/efabless/caravel-lite/pull/3

22 Views

Open in Slack

Previous Next