From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 0CBCC3858420; Mon, 27 Nov 2023 16:07:57 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 0CBCC3858420 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1701101278; bh=lryCZ6zq+YW/iDVQrHey2LumimSv7j1BzSa90tqtGKw=; h=From:To:Subject:Date:In-Reply-To:References:From; b=QHHz0DHZOSXeCFxujUzsJqlwbk5yPgJf1GUxCidcNHGx3uUZ2BEODhcS3JFoUp0+3 0Zu+XYYzOA9FYuJIMsSP/jcbOXVUHWdxvQAthcH+jBW7+LuJS6UyqJYFjj0JXXNjWq 9Xb+9XO4NxMwoM+oqfO74EHui4Ej+asqUSMANnVI= From: "wcohen at redhat dot com" To: systemtap@sourceware.org Subject: [Bug runtime/31074] On aarch64 the systemtap.base/set_kernel.stp triggers "Unable to handle kernel paging request" Date: Mon, 27 Nov 2023 16:07:56 +0000 X-Bugzilla-Reason: AssignedTo X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: systemtap X-Bugzilla-Component: runtime X-Bugzilla-Version: unspecified X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: wcohen at redhat dot com X-Bugzilla-Status: ASSIGNED X-Bugzilla-Resolution: X-Bugzilla-Priority: P2 X-Bugzilla-Assigned-To: systemtap at sourceware dot org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://sourceware.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 List-Id: https://sourceware.org/bugzilla/show_bug.cgi?id=3D31074 --- Comment #4 from William Cohen --- Looking a bit more at the stp_tracepoint.c code. If e was NULL for hlist_for_each_entry should, it should exit the for loop rather than doing = the strcmp: https://elixir.bootlin.com/linux/v6.5.11/source/include/linux/list.h#L1053 The problem attemtped access is occurring when tracepoints are being remove= d.=20 It is a bit surprising the that the similar code in add_tracepoint doesn't encounter a similar problem earlier around with virtually identical code in add_tracepoint: https://sourceware.org/git/?p=3Dsystemtap.git;a=3Dblob;f=3Druntime/linux/st= p_tracepoint.c;h=3D508948dce4fd438bde6d9d155035faba2abd0ee1;hb=3DHEAD#l146 The suggested patch isn't making much sense to me.=20 e is a local variable that would no longer be initialized before the "if(!e){..." check. --=20 You are receiving this mail because: You are the assignee for the bug.=