From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from out-172.mta0.migadu.com (out-172.mta0.migadu.com [91.218.175.172]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 34F8819259E for ; Mon, 30 Sep 2024 16:35:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=91.218.175.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727714118; cv=none; b=T3enE9hPtqdP5Dwq2uDlIlTn2Hk86V9/lPQL2i5Ks/zS8HIpWmEb5OmzUwjr+8EKf/TbAqoqVyCPJ1Bp99WzMXSUK3let6TDJUSmEsjjCmIzXQTN8SAewPcSufUy48MVTvRu3tlLNyCL/VQ+8UdRFDkG2qciySExDlQPiUbR9wI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727714118; c=relaxed/simple; bh=WR56oJljbL/VYJ7+iW4zC3clc5fpUcWUiVCLNUn8Jmo=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=crq6D0QsoqiUNkpI5OuE7837rc/M5asaktPl028pfq5bg1LW3q/yLWFsm1qUYDcUlcMyOZRMcOcCTqqdmh9XjIVLxrdPbjIXGw4+Ei34EmtM7HhqShlD6Ii0XBsYOKq0lxheJmVOcohUzmOKBSuA0l7JGSNT10C+6nF4FgdlGo8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=P8oTKHqS; arc=none smtp.client-ip=91.218.175.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="P8oTKHqS" Message-ID: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1727714114; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=Ng5icTrswHGtbdrXLpWp/wpfiGHFRzlmRDR5mBBmL/I=; b=P8oTKHqSwZC+TU0TZJDuOZ97Km7STXiN7RfeEEN4YMZgTLgxspaZyPuI+dVFDOM2fy/qDW TAp9ByrHXTgQPu/KSWIUJznxgWhncXJdCgZ3xaSrP8jZ1yt8UwQYAT7VfU8fy5N6JOSf4z W4VExYP7/HtqGO9Q47gKn+e2ndHGCs4= Date: Mon, 30 Sep 2024 09:35:08 -0700 Precedence: bulk X-Mailing-List: bpf@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Subject: Re: [PATCH bpf-next v3 5/5] selftests/bpf: Add private stack tests Content-Language: en-GB To: Alexei Starovoitov , Jiri Olsa Cc: bpf , Alexei Starovoitov , Andrii Nakryiko , Daniel Borkmann , Kernel Team , Martin KaFai Lau References: <20240926234506.1769256-1-yonghong.song@linux.dev> <20240926234531.1771024-1-yonghong.song@linux.dev> X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Yonghong Song In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Migadu-Flow: FLOW_OUT On 9/30/24 8:05 AM, Alexei Starovoitov wrote: > On Mon, Sep 30, 2024 at 6:40 AM Jiri Olsa wrote: >> On Thu, Sep 26, 2024 at 04:45:31PM -0700, Yonghong Song wrote: >>> Some private stack tests are added including: >>> - prog with stack size greater than BPF_PSTACK_MIN_SUBTREE_SIZE. >>> - prog with stack size less than BPF_PSTACK_MIN_SUBTREE_SIZE. >>> - prog with one subprog having MAX_BPF_STACK stack size and another >>> subprog having non-zero stack size. >>> - prog with callback function. >>> - prog with exception in main prog or subprog. >>> >>> Signed-off-by: Yonghong Song >> hi, >> might be some fail on my side, but I had to include bpf_experimental.h to >> compile this.. ci seems ok >> >> CLNG-BPF [test_progs-cpuv4] verifier_private_stack.bpf.o >> progs/verifier_private_stack.c:174:2: error: call to undeclared function 'bpf_throw'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration] >> 174 | bpf_throw(0); > Yeah. Let's add bpf_experimental.h for folks like Jiri > who didn't upgrade their pahole for a long time :) > > bpf_throw will be in vmlinux.h ;) Okay, will do.