From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B2F14EC875F for ; Fri, 8 Sep 2023 01:40:09 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239163AbjIHBkL (ORCPT ); Thu, 7 Sep 2023 21:40:11 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33278 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235744AbjIHBkJ (ORCPT ); Thu, 7 Sep 2023 21:40:09 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8CEE919BD; Thu, 7 Sep 2023 18:40:05 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 74584C433C8; Fri, 8 Sep 2023 01:40:04 +0000 (UTC) Date: Thu, 7 Sep 2023 21:40:20 -0400 From: Steven Rostedt To: Zheng Yejian Cc: , , , , , Shuah Khan Subject: Re: [PATCH] selftests/ftrace: Test toplevel-enable for instance Message-ID: <20230907214020.34981ffd@gandalf.local.home> In-Reply-To: <20230710175415.2ec0061f@gandalf.local.home> References: <20230509203659.1173917-1-zhengyejian1@huawei.com> <20230609174626.66659892@gandalf.local.home> <20230710175415.2ec0061f@gandalf.local.home> X-Mailer: Claws Mail 3.19.1 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 10 Jul 2023 17:54:36 -0400 Steven Rostedt wrote: > Hi Shuah, > > I think this dropped through the cracks. Can you take this through your > tree? ping? > > -- Steve > > On Fri, 9 Jun 2023 17:46:26 -0400 > Steven Rostedt wrote: > > > On Wed, 10 May 2023 04:36:59 +0800 > > Zheng Yejian wrote: > > > > > 'available_events' is actually not required by > > > 'test.d/event/toplevel-enable.tc' and its Existence has been tested in > > > 'test.d/00basic/basic4.tc'. > > > > > > So the require of 'available_events' can be dropped and then we can add > > > 'instance' flag to test 'test.d/event/toplevel-enable.tc' for instance. > > > > > > Test result show as below: > > > # ./ftracetest test.d/event/toplevel-enable.tc > > > === Ftrace unit tests === > > > [1] event tracing - enable/disable with top level files [PASS] > > > [2] (instance) event tracing - enable/disable with top level files [PASS] > > > > > > # of passed: 2 > > > # of failed: 0 > > > # of unresolved: 0 > > > # of untested: 0 > > > # of unsupported: 0 > > > # of xfailed: 0 > > > # of undefined(test bug): 0 > > > > > > Signed-off-by: Zheng Yejian > > > > Acked-by: Steven Rostedt (Google) > > > > -- Steve > > > > > --- > > > tools/testing/selftests/ftrace/test.d/event/toplevel-enable.tc | 3 ++- > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > diff --git a/tools/testing/selftests/ftrace/test.d/event/toplevel-enable.tc b/tools/testing/selftests/ftrace/test.d/event/toplevel-enable.tc > > > index 93c10ea42a68..8b8e1aea985b 100644 > > > --- a/tools/testing/selftests/ftrace/test.d/event/toplevel-enable.tc > > > +++ b/tools/testing/selftests/ftrace/test.d/event/toplevel-enable.tc > > > @@ -1,7 +1,8 @@ > > > #!/bin/sh > > > # SPDX-License-Identifier: GPL-2.0 > > > # description: event tracing - enable/disable with top level files > > > -# requires: available_events set_event events/enable > > > +# requires: set_event events/enable > > > +# flags: instance > > > > > > do_reset() { > > > echo > set_event