search

p4lang / behavioral-model

The reference P4 software switch
Apache License 2.0
532 stars 327 forks source link

Increase port_map/lag_map sizes to 512 #1165

Closed smolkaj closed 1 year ago

smolkaj commented 1 year ago

Fixes https://github.com/p4lang/behavioral-model/issues/1162

smolkaj commented 1 year ago

@antoninbas is this what you had in mind? @tiantianz for visibility

smolkaj commented 1 year ago

I think it would make sense to update

https://github.com/p4lang/behavioral-model/blob/27c235944492ef55ba061fcf658b4d8102d53bd8/pdfixed/include/bm/pdfixed/pd_pre.h#L31-L32

as well, even though that code is unlikely to be used any more. I don't think this number shows up anywhere else.

Done.

smolkaj commented 1 year ago

I also had to remove some unnecessary copies of the form for (const auto foo : bar) { ...] to amke clang++-10 CI happy.

smolkaj commented 1 year ago

@antoninbas CI now passes.