GitHub pull request #3607 of commit 6de72d2c681e63438fcc742b1394531e413b4173, no merge conflicts.sha1origin/pr/3607/mergeghprbActualCommit6de72d2c681e63438fcc742b1394531e413b4173ghprbActualCommitAuthorghprbActualCommitAuthorEmailghprbAuthorRepoGitUrlhttps://github.com/kinvolk/bcc.gitghprbTriggerAuthorghprbTriggerAuthorEmailghprbTriggerAuthorLoginyonghong-songghprbTriggerAuthorLoginMention@yonghong-songghprbPullId3607ghprbTargetBranchmasterghprbSourceBranchmauricio/upstream/fix-filtering-by-containers-opensnoopGIT_BRANCHmauricio/upstream/fix-filtering-by-containers-opensnoopghprbPullAuthorEmailmauriciovasquezbernal@gmail.comghprbPullAuthorLoginmauriciovasquezbernalghprbPullAuthorLoginMention@mauriciovasquezbernalghprbPullDescriptionGitHub pull request #3607 of commit 6de72d2c681e63438fcc742b1394531e413b4173, no merge conflicts.ghprbPullTitlesrc/python: fix filtering by containers when kfunc are supportedghprbPullLinkhttps://github.com/iovisor/bcc/pull/3607ghprbPullLongDescriptionThe filtering by mount namespace logic has to access\r\ncurrent_task->nsproxy->mnt_ns->ns.inum to get the mount namespace id. Before\r\nthis commit, that line was written in C natural syntax and we're relying on the\r\nBCC rewriter to transform that to valid eBPF code by emitting some\r\nbpf_probe_read calls.\r\n\r\nThis support was not working when using opensnoop in systems supporting kfuncs\r\nbecause in this case the BCC rewriter doesn't transform that line and the\r\nverifier claims about an invalid memory access:\r\n\r\n7: (85) call bpf_get_current_task#35; return\r\ncurrent_task->nsproxy->mnt_ns->ns.inum; 8: (79) r1 = *(u64 *)(r0 +2896) R0\r\ninvalid mem access 'inv'\r\n\r\nThis commit fixes that by explicitly using bpf_probe_kernel_read() instead of\r\nthe C natural syntax.\r\n\r\n--- \r\n\r\nIt's not yet 100% clear that this is the best solution to that problem. I've [asked](https://cilium.slack.com/archives/C4XCTGYEM/p1630681479111500) for some more information about that error but I have received no response yet. If anybody here has any comment it's highly appreciated. \r\n\r\ncc @alban ghprbCommentBody[buildbot, test this please]ghprbGhRepositoryiovisor/bccghprbCredentialsId6d3daf13-69b8-48b1-9c8f-ec5353264113false<a title="src/python: fix filtering by containers when kfunc are supported" href="https://github.com/iovisor/bcc/pull/3607">PR #3607</a>: src/python: fix filtering b...#114914068197460664bcc-pr #11491149false11499201SUCCESS1631581005672https://buildbot.iovisor.org/jenkins/user/a.s.protopopov/my-views/view/all/job/bcc-pr/1149/git1149https://buildbot.iovisor.org/jenkins/user/a.s.protopopov/my-views/view/all/job/bcc-pr/label=fc25/1149/1149https://buildbot.iovisor.org/jenkins/user/a.s.protopopov/my-views/view/all/job/bcc-pr/label=fc26/1149/1149https://buildbot.iovisor.org/jenkins/user/a.s.protopopov/my-views/view/all/job/bcc-pr/label=fc27/1149/1149https://buildbot.iovisor.org/jenkins/user/a.s.protopopov/my-views/view/all/job/bcc-pr/label=fc28/1149/1149https://buildbot.iovisor.org/jenkins/user/a.s.protopopov/my-views/view/all/job/bcc-pr/label=ubuntu1604/1149/1149https://buildbot.iovisor.org/jenkins/user/a.s.protopopov/my-views/view/all/job/bcc-pr/label=ubuntu1710/1149/1149https://buildbot.iovisor.org/jenkins/user/a.s.protopopov/my-views/view/all/job/bcc-pr/label=ubuntu1804/1149/