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 X-Spam-Level: X-Spam-Status: No, score=-2.3 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5153BC606CF for ; Mon, 8 Jul 2019 22:54:10 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 2CDDF216FD for ; Mon, 8 Jul 2019 22:54:10 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728259AbfGHWyJ (ORCPT ); Mon, 8 Jul 2019 18:54:09 -0400 Received: from mx1.redhat.com ([209.132.183.28]:43414 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725815AbfGHWyI (ORCPT ); Mon, 8 Jul 2019 18:54:08 -0400 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id D705230820DD; Mon, 8 Jul 2019 22:54:04 +0000 (UTC) Received: from treble (ovpn-112-43.rdu2.redhat.com [10.10.112.43]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 6D4495D9E5; Mon, 8 Jul 2019 22:54:02 +0000 (UTC) Date: Mon, 8 Jul 2019 17:53:59 -0500 From: Josh Poimboeuf To: Alexei Starovoitov Cc: Ingo Molnar , Borislav Petkov , "H. Peter Anvin" , Song Liu , Thomas Gleixner , Steven Rostedt , Kairui Song , Daniel Borkmann , Alexei Starovoitov , Peter Zijlstra , LKML , linux-tip-commits@vger.kernel.org Subject: Re: [tip:x86/urgent] bpf: Fix ORC unwinding in non-JIT BPF code Message-ID: <20190708225359.ewk44pvrv6a4oao7@treble> References: <881939122b88f32be4c374d248c09d7527a87e35.1561685471.git.jpoimboe@redhat.com> <20190706202942.GA123403@gmail.com> <20190707013206.don22x3tfldec4zm@treble> <20190707055209.xqyopsnxfurhrkxw@treble> <20190708223834.zx7u45a4uuu2yyol@treble> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20180716 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.47]); Mon, 08 Jul 2019 22:54:08 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jul 08, 2019 at 03:49:33PM -0700, Alexei Starovoitov wrote: > > > Sorry for delay. I'm mostly offgrid until next week. > > > As far as -fno-gcse.. I don't mind as long as it doesn't hurt performance. > > > Which I suspect it will :( > > > All these indirect gotos are there for performance. > > > Single indirect goto and a bunch of jmp select_insn > > > are way slower, since there is only one instruction > > > for cpu branch predictor to work with. > > > When every insn is followed by "jmp *jumptable" > > > there is more room for cpu to speculate. > > > It's been long time, but when I wrote it the difference > > > between all indirect goto vs single indirect goto was almost 2x. > > > > Just to clarify, -fno-gcse doesn't get rid of any of the indirect jumps. > > It still has 166 indirect jumps. It just gets rid of the second > > optimization, where the jumptable address is placed in a register. > > what about other functions in core.c ? > May be it's easier to teach objtool to recognize that pattern? The GCC man page actually recommends using -fno-gcse for computed goto code, for better performance. So if that's actually true, then it would be win-win because objtool wouldn't need a change for it. Otherwise I can teach objtool to recognize the new pattern. > > If you have a benchmark which is relatively easy to use, I could try to > > run some tests. > > modprobe test_bpf > selftests/bpf/test_progs > both print runtime. > Some of test_progs have high run-to-run variations though. Thanks, I'll give it a shot. -- Josh