Date:   Fri, 10 Dec 2021 13:31:23 +0100 (CET)
From:   Miroslav Benes <mbenes@suse.cz>
To:     Heiko Carstens <hca@linux.ibm.com>
cc:     Jerome Marchand <jmarchan@redhat.com>,
        Steven Rostedt <rostedt@goodmis.org>,
        linux-s390@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] recordmcount.pl: look for jgnop instruction as well as
 bcrl on s390
In-Reply-To: <alpine.LSU.2.21.2112101310080.5704@pobox.suse.cz>
Message-ID: <alpine.LSU.2.21.2112101321010.5704@pobox.suse.cz>
References: <20211210093827.1623286-1-jmarchan@redhat.com> <alpine.LSU.2.21.2112101054070.5704@pobox.suse.cz> <YbMyvJ6gz7LvoXgE@osiris> <alpine.LSU.2.21.2112101310080.5704@pobox.suse.cz>
User-Agent: Alpine 2.21 (LSU 202 2017-01-01)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Precedence: bulk

> > Otherwise I would be surprised that this would make any difference.
> 
> How come? I mean, s390 does not define HAVE_C_RECORDMCOUNT which implies 
> that recordmcount.pl is used (see scripts/Makefile.build and 
> BUILD_C_RECORDMCOUNT definition in Makefile).

Ah, sorry, that was too quick.

I guess you want to say that recordmcount is not used at all and GCC 
should do it. Because CONFIG_FTRACE_MCOUNT_USE_RECORDMCOUNT is not set, 
since FTRACE_MCOUNT_USE_CC should be set.

What a maze.

But isn't -mrecord-mcount x86_64-only option?

Miroslav