From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751582AbeBZX3L (ORCPT ); Mon, 26 Feb 2018 18:29:11 -0500 Received: from ozlabs.org ([103.22.144.67]:37755 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750969AbeBZX3J (ORCPT ); Mon, 26 Feb 2018 18:29:09 -0500 Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=canb.auug.org.au Date: Tue, 27 Feb 2018 10:28:59 +1100 From: Stephen Rothwell To: Networking , David Miller Cc: Daniel Borkmann , Alexei Starovoitov , Linux-Next Mailing List , Linux Kernel Mailing List Subject: Re: linux-next: manual merge of the bpf-next tree with the bpf tree Message-ID: <20180227102859.466d6d78@canb.auug.org.au> In-Reply-To: <20180226114147.2ab5c647@canb.auug.org.au> References: <20180226114147.2ab5c647@canb.auug.org.au> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; boundary="Sig_/pW2pXFrdKTN4VWUmV2ml=Gs"; protocol="application/pgp-signature" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --Sig_/pW2pXFrdKTN4VWUmV2ml=Gs Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Hi Dave, On Mon, 26 Feb 2018 11:41:47 +1100 Stephen Rothwell = wrote: > > Today's linux-next merge of the bpf-next tree got a conflict in: >=20 > tools/testing/selftests/bpf/test_verifier.c >=20 > between commit: >=20 > ca36960211eb ("bpf: allow xadd only on aligned memory") >=20 > from the bpf tree and commit: >=20 > 23d191a82c13 ("bpf: add various jit test cases") >=20 > from the bpf-next tree. >=20 > I fixed it up (see below) and can carry the fix as necessary. This > is now fixed as far as linux-next is concerned, but any non trivial > conflicts should be mentioned to your upstream maintainer when your tree > is submitted for merging. You may also want to consider cooperating > with the maintainer of the conflicting tree to minimise any particularly > complex conflicts. >=20 > --=20 > Cheers, > Stephen Rothwell >=20 > diff --cc tools/testing/selftests/bpf/test_verifier.c > index 437c0b1c9d21,c987d3a2426f..000000000000 > --- a/tools/testing/selftests/bpf/test_verifier.c > +++ b/tools/testing/selftests/bpf/test_verifier.c > @@@ -11163,64 -11140,95 +11166,153 @@@ static struct bpf_test tests[] =3D= =20 > .result =3D REJECT, > .prog_type =3D BPF_PROG_TYPE_TRACEPOINT, > }, > + { > + "xadd/w check unaligned stack", > + .insns =3D { > + BPF_MOV64_IMM(BPF_REG_0, 1), > + BPF_STX_MEM(BPF_DW, BPF_REG_10, BPF_REG_0, -8), > + BPF_STX_XADD(BPF_W, BPF_REG_10, BPF_REG_0, -7), > + BPF_LDX_MEM(BPF_DW, BPF_REG_0, BPF_REG_10, -8), > + BPF_EXIT_INSN(), > + }, > + .result =3D REJECT, > + .errstr =3D "misaligned stack access off", > + .prog_type =3D BPF_PROG_TYPE_SCHED_CLS, > + }, > + { > + "xadd/w check unaligned map", > + .insns =3D { > + BPF_ST_MEM(BPF_DW, BPF_REG_10, -8, 0), > + BPF_MOV64_REG(BPF_REG_2, BPF_REG_10), > + BPF_ALU64_IMM(BPF_ADD, BPF_REG_2, -8), > + BPF_LD_MAP_FD(BPF_REG_1, 0), > + BPF_RAW_INSN(BPF_JMP | BPF_CALL, 0, 0, 0, > + BPF_FUNC_map_lookup_elem), > + BPF_JMP_IMM(BPF_JNE, BPF_REG_0, 0, 1), > + BPF_EXIT_INSN(), > + BPF_MOV64_IMM(BPF_REG_1, 1), > + BPF_STX_XADD(BPF_W, BPF_REG_0, BPF_REG_1, 3), > + BPF_LDX_MEM(BPF_W, BPF_REG_0, BPF_REG_0, 3), > + BPF_EXIT_INSN(), > + }, > + .fixup_map1 =3D { 3 }, > + .result =3D REJECT, > + .errstr =3D "misaligned value access off", > + .prog_type =3D BPF_PROG_TYPE_SCHED_CLS, > + }, > + { > + "xadd/w check unaligned pkt", > + .insns =3D { > + BPF_LDX_MEM(BPF_W, BPF_REG_2, BPF_REG_1, > + offsetof(struct xdp_md, data)), > + BPF_LDX_MEM(BPF_W, BPF_REG_3, BPF_REG_1, > + offsetof(struct xdp_md, data_end)), > + BPF_MOV64_REG(BPF_REG_1, BPF_REG_2), > + BPF_ALU64_IMM(BPF_ADD, BPF_REG_1, 8), > + BPF_JMP_REG(BPF_JLT, BPF_REG_1, BPF_REG_3, 2), > + BPF_MOV64_IMM(BPF_REG_0, 99), > + BPF_JMP_IMM(BPF_JA, 0, 0, 6), > + BPF_MOV64_IMM(BPF_REG_0, 1), > + BPF_ST_MEM(BPF_W, BPF_REG_2, 0, 0), > + BPF_ST_MEM(BPF_W, BPF_REG_2, 3, 0), > + BPF_STX_XADD(BPF_W, BPF_REG_2, BPF_REG_0, 1), > + BPF_STX_XADD(BPF_W, BPF_REG_2, BPF_REG_0, 2), > + BPF_LDX_MEM(BPF_W, BPF_REG_0, BPF_REG_2, 1), > + BPF_EXIT_INSN(), > + }, > + .result =3D REJECT, > + .errstr =3D "BPF_XADD stores into R2 packet", > + .prog_type =3D BPF_PROG_TYPE_XDP, > + }, > + { > + "jit: lsh, rsh, arsh by 1", > + .insns =3D { > + BPF_MOV64_IMM(BPF_REG_0, 1), > + BPF_MOV64_IMM(BPF_REG_1, 0xff), > + BPF_ALU64_IMM(BPF_LSH, BPF_REG_1, 1), > + BPF_ALU32_IMM(BPF_LSH, BPF_REG_1, 1), > + BPF_JMP_IMM(BPF_JEQ, BPF_REG_1, 0x3fc, 1), > + BPF_EXIT_INSN(), > + BPF_ALU64_IMM(BPF_RSH, BPF_REG_1, 1), > + BPF_ALU32_IMM(BPF_RSH, BPF_REG_1, 1), > + BPF_JMP_IMM(BPF_JEQ, BPF_REG_1, 0xff, 1), > + BPF_EXIT_INSN(), > + BPF_ALU64_IMM(BPF_ARSH, BPF_REG_1, 1), > + BPF_JMP_IMM(BPF_JEQ, BPF_REG_1, 0x7f, 1), > + BPF_EXIT_INSN(), > + BPF_MOV64_IMM(BPF_REG_0, 2), > + BPF_EXIT_INSN(), > + }, > + .result =3D ACCEPT, > + .retval =3D 2, > + }, > + { > + "jit: mov32 for ldimm64, 1", > + .insns =3D { > + BPF_MOV64_IMM(BPF_REG_0, 2), > + BPF_LD_IMM64(BPF_REG_1, 0xfeffffffffffffffULL), > + BPF_ALU64_IMM(BPF_RSH, BPF_REG_1, 32), > + BPF_LD_IMM64(BPF_REG_2, 0xfeffffffULL), > + BPF_JMP_REG(BPF_JEQ, BPF_REG_1, BPF_REG_2, 1), > + BPF_MOV64_IMM(BPF_REG_0, 1), > + BPF_EXIT_INSN(), > + }, > + .result =3D ACCEPT, > + .retval =3D 2, > + }, > + { > + "jit: mov32 for ldimm64, 2", > + .insns =3D { > + BPF_MOV64_IMM(BPF_REG_0, 1), > + BPF_LD_IMM64(BPF_REG_1, 0x1ffffffffULL), > + BPF_LD_IMM64(BPF_REG_2, 0xffffffffULL), > + BPF_JMP_REG(BPF_JEQ, BPF_REG_1, BPF_REG_2, 1), > + BPF_MOV64_IMM(BPF_REG_0, 2), > + BPF_EXIT_INSN(), > + }, > + .result =3D ACCEPT, > + .retval =3D 2, > + }, > + { > + "jit: various mul tests", > + .insns =3D { > + BPF_LD_IMM64(BPF_REG_2, 0xeeff0d413122ULL), > + BPF_LD_IMM64(BPF_REG_0, 0xfefefeULL), > + BPF_LD_IMM64(BPF_REG_1, 0xefefefULL), > + BPF_ALU64_REG(BPF_MUL, BPF_REG_0, BPF_REG_1), > + BPF_JMP_REG(BPF_JEQ, BPF_REG_0, BPF_REG_2, 2), > + BPF_MOV64_IMM(BPF_REG_0, 1), > + BPF_EXIT_INSN(), > + BPF_LD_IMM64(BPF_REG_3, 0xfefefeULL), > + BPF_ALU64_REG(BPF_MUL, BPF_REG_3, BPF_REG_1), > + BPF_JMP_REG(BPF_JEQ, BPF_REG_3, BPF_REG_2, 2), > + BPF_MOV64_IMM(BPF_REG_0, 1), > + BPF_EXIT_INSN(), > + BPF_MOV32_REG(BPF_REG_2, BPF_REG_2), > + BPF_LD_IMM64(BPF_REG_0, 0xfefefeULL), > + BPF_ALU32_REG(BPF_MUL, BPF_REG_0, BPF_REG_1), > + BPF_JMP_REG(BPF_JEQ, BPF_REG_0, BPF_REG_2, 2), > + BPF_MOV64_IMM(BPF_REG_0, 1), > + BPF_EXIT_INSN(), > + BPF_LD_IMM64(BPF_REG_3, 0xfefefeULL), > + BPF_ALU32_REG(BPF_MUL, BPF_REG_3, BPF_REG_1), > + BPF_JMP_REG(BPF_JEQ, BPF_REG_3, BPF_REG_2, 2), > + BPF_MOV64_IMM(BPF_REG_0, 1), > + BPF_EXIT_INSN(), > + BPF_LD_IMM64(BPF_REG_0, 0x952a7bbcULL), > + BPF_LD_IMM64(BPF_REG_1, 0xfefefeULL), > + BPF_LD_IMM64(BPF_REG_2, 0xeeff0d413122ULL), > + BPF_ALU32_REG(BPF_MUL, BPF_REG_2, BPF_REG_1), > + BPF_JMP_REG(BPF_JEQ, BPF_REG_2, BPF_REG_0, 2), > + BPF_MOV64_IMM(BPF_REG_0, 1), > + BPF_EXIT_INSN(), > + BPF_MOV64_IMM(BPF_REG_0, 2), > + BPF_EXIT_INSN(), > + }, > + .result =3D ACCEPT, > + .retval =3D 2, > + }, > +=20 > }; > =20 > static int probe_filter_length(const struct bpf_insn *fp) This conflict is now between the bfp tree and the net-next tree. --=20 Cheers, Stephen Rothwell --Sig_/pW2pXFrdKTN4VWUmV2ml=Gs Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEENIC96giZ81tWdLgKAVBC80lX0GwFAlqUmDsACgkQAVBC80lX 0GyJdQgAoC+RwCc+n5j/Uhb7HhoPp7LQHWrnm09OpCot8RiZh9MLEpXhoS5U29pm Irjv6n/Lve4pECzK//aK5uodkngTzKUj3lWqpUfw4TQlrtP8TBRPjOGd9HN3ZNNy OQyaoz9ZC+uCCzq6ylb4Ks9BUwqdkZp5s7T56gcFmXMCoard8WOUgUVcJuOVOdS8 GttUKEiTdOuhxVsRref4oDKKCqDfEbKnv4eVLC0YvoylGcJiSUJhvZ9w81tXGl7v 6pGb2wxVF4qjEyV4XtXj384jQUdjNNai1FwQylSpHrSjXy/ZUClBcuu4uyRBdNjQ Z8OK9MDjj4mIejMGQ0W9r+NZGkpcfQ== =yuOQ -----END PGP SIGNATURE----- --Sig_/pW2pXFrdKTN4VWUmV2ml=Gs--