{"action":"created","issue":{"url":"https://api.github.com/repos/Igalia/pflua/issues/225","labels_url":"https://api.github.com/repos/Igalia/pflua/issues/225/labels{/name}","comments_url":"https://api.github.com/repos/Igalia/pflua/issues/225/comments","events_url":"https://api.github.com/repos/Igalia/pflua/issues/225/events","html_url":"https://github.com/Igalia/pflua/issues/225","id":83951647,"number":225,"title":"Error with recent LuaJIT","user":{"login":"lukego","id":13791,"avatar_url":"https://avatars.githubusercontent.com/u/13791?v=3","gravatar_id":"","url":"https://api.github.com/users/lukego","html_url":"https://github.com/lukego","followers_url":"https://api.github.com/users/lukego/followers","following_url":"https://api.github.com/users/lukego/following{/other_user}","gists_url":"https://api.github.com/users/lukego/gists{/gist_id}","starred_url":"https://api.github.com/users/lukego/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lukego/subscriptions","organizations_url":"https://api.github.com/users/lukego/orgs","repos_url":"https://api.github.com/users/lukego/repos","events_url":"https://api.github.com/users/lukego/events{/privacy}","received_events_url":"https://api.github.com/users/lukego/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":19,"created_at":"2015-06-02T10:02:52Z","updated_at":"2015-06-10T16:21:53Z","closed_at":null,"body":"Howdy!\r\n\r\nOn the Snabb Switch `next` branch there is a regression in the `apps.packet_filter.pcap_filter` selftest. This seems to be caused by the combination of upgrading pflua and luajit. Is there perhaps a new bug introduced in LuaJIT that affects pflua?\r\n\r\nHere is what happens:\r\n\r\n```\r\n$ sudo ./snabb snsh -t apps.packet_filter.pcap_filter\r\nselftest: pcap_filter\r\n../deps/pflua/src/pf/optimize.lua:78: invalid value (table) at index 4 in table for 'concat'\r\nstack traceback:\r\n        core/main.lua:116: in function <core/main.lua:114>\r\n        [C]: in function 'f'\r\n        ../deps/pflua/src/pf/optimize.lua:78: in function 'cfkey'\r\n        ../deps/pflua/src/pf/optimize.lua:281: in function 'simplify'\r\n        ../deps/pflua/src/pf/optimize.lua:217: in function 'simplify'\r\n        ../deps/pflua/src/pf/optimize.lua:216: in function 'simplify'\r\n        ../deps/pflua/src/pf/optimize.lua:216: in function 'simplify'\r\n        ../deps/pflua/src/pf/optimize.lua:216: in function 'simplify'\r\n        ../deps/pflua/src/pf/optimize.lua:740: in function 'f'\r\n        ../deps/pflua/src/pf/utils.lua:158: in function 'fixpoint'\r\n        ../deps/pflua/src/pf/optimize.lua:749: in function 'optimize'\r\n        ...\r\n```\r\n\r\nand `git bisect` on luajit says this is the problem commit:\r\n\r\n```\r\nb82fc3d Bump table allocations retroactively if they grow later on.\r\n```\r\n\r\nI have tested with LuaJIT `4da1bb6` that does seem to contain a fix for the commit above, but the problem persists."},"comment":{"url":"https://api.github.com/repos/Igalia/pflua/issues/comments/110820772","html_url":"https://github.com/Igalia/pflua/issues/225#issuecomment-110820772","issue_url":"https://api.github.com/repos/Igalia/pflua/issues/225","id":110820772,"user":{"login":"kbara","id":1588250,"avatar_url":"https://avatars.githubusercontent.com/u/1588250?v=3","gravatar_id":"","url":"https://api.github.com/users/kbara","html_url":"https://github.com/kbara","followers_url":"https://api.github.com/users/kbara/followers","following_url":"https://api.github.com/users/kbara/following{/other_user}","gists_url":"https://api.github.com/users/kbara/gists{/gist_id}","starred_url":"https://api.github.com/users/kbara/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kbara/subscriptions","organizations_url":"https://api.github.com/users/kbara/orgs","repos_url":"https://api.github.com/users/kbara/repos","events_url":"https://api.github.com/users/kbara/events{/privacy}","received_events_url":"https://api.github.com/users/kbara/received_events","type":"User","site_admin":false},"created_at":"2015-06-10T16:21:53Z","updated_at":"2015-06-10T16:21:53Z","body":"458a40b7242aefe2f0893019f0451fb2f2deccd9 fixes the new test case, but pflua-compile keeps having both of the above backtraces, unfortunately."}}