From: Sasha Levin Date: Wed, 18 Jan 2023 20:22:21 +0000 (-0500) Subject: Fixes for 5.10 X-Git-Tag: v4.14.304~60 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=9b2579c5691a57f218519eaeca47c621a3c35300;p=thirdparty%2Fkernel%2Fstable-queue.git Fixes for 5.10 Signed-off-by: Sasha Levin --- diff --git a/queue-5.10/btrfs-always-report-error-in-run_one_delayed_ref.patch b/queue-5.10/btrfs-always-report-error-in-run_one_delayed_ref.patch new file mode 100644 index 00000000000..da11a9f568d --- /dev/null +++ b/queue-5.10/btrfs-always-report-error-in-run_one_delayed_ref.patch @@ -0,0 +1,80 @@ +From 4c6cbb23d73b80bc50c4aa9a189a3fbdd5a439d8 Mon Sep 17 00:00:00 2001 +From: Sasha Levin +Date: Mon, 26 Dec 2022 09:00:40 +0800 +Subject: btrfs: always report error in run_one_delayed_ref() + +From: Qu Wenruo + +[ Upstream commit 39f501d68ec1ed5cd5c66ac6ec2a7131c517bb92 ] + +Currently we have a btrfs_debug() for run_one_delayed_ref() failure, but +if end users hit such problem, there will be no chance that +btrfs_debug() is enabled. This can lead to very little useful info for +debugging. + +This patch will: + +- Add extra info for error reporting + Including: + * logical bytenr + * num_bytes + * type + * action + * ref_mod + +- Replace the btrfs_debug() with btrfs_err() + +- Move the error reporting into run_one_delayed_ref() + This is to avoid use-after-free, the @node can be freed in the caller. + +This error should only be triggered at most once. + +As if run_one_delayed_ref() failed, we trigger the error message, then +causing the call chain to error out: + +btrfs_run_delayed_refs() +`- btrfs_run_delayed_refs() + `- btrfs_run_delayed_refs_for_head() + `- run_one_delayed_ref() + +And we will abort the current transaction in btrfs_run_delayed_refs(). +If we have to run delayed refs for the abort transaction, +run_one_delayed_ref() will just cleanup the refs and do nothing, thus no +new error messages would be output. + +Reviewed-by: Anand Jain +Signed-off-by: Qu Wenruo +Signed-off-by: David Sterba +Signed-off-by: Sasha Levin +--- + fs/btrfs/extent-tree.c | 7 +++++-- + 1 file changed, 5 insertions(+), 2 deletions(-) + +diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c +index 284294620e9f..7d9b8050b09c 100644 +--- a/fs/btrfs/extent-tree.c ++++ b/fs/btrfs/extent-tree.c +@@ -1684,6 +1684,11 @@ static int run_one_delayed_ref(struct btrfs_trans_handle *trans, + BUG(); + if (ret && insert_reserved) + btrfs_pin_extent(trans, node->bytenr, node->num_bytes, 1); ++ if (ret < 0) ++ btrfs_err(trans->fs_info, ++"failed to run delayed ref for logical %llu num_bytes %llu type %u action %u ref_mod %d: %d", ++ node->bytenr, node->num_bytes, node->type, ++ node->action, node->ref_mod, ret); + return ret; + } + +@@ -1935,8 +1940,6 @@ static int btrfs_run_delayed_refs_for_head(struct btrfs_trans_handle *trans, + if (ret) { + unselect_delayed_ref_head(delayed_refs, locked_ref); + btrfs_put_delayed_ref(ref); +- btrfs_debug(fs_info, "run_one_delayed_ref returned %d", +- ret); + return ret; + } + +-- +2.35.1 + diff --git a/queue-5.10/btrfs-fix-trace-event-name-typo-for-flush_delayed_re.patch b/queue-5.10/btrfs-fix-trace-event-name-typo-for-flush_delayed_re.patch new file mode 100644 index 00000000000..3c8ba4b4e47 --- /dev/null +++ b/queue-5.10/btrfs-fix-trace-event-name-typo-for-flush_delayed_re.patch @@ -0,0 +1,37 @@ +From 65936c81bbb028c672d43ede75c164abc7cd0304 Mon Sep 17 00:00:00 2001 +From: Sasha Levin +Date: Wed, 14 Dec 2022 11:06:07 +0900 +Subject: btrfs: fix trace event name typo for FLUSH_DELAYED_REFS + +From: Naohiro Aota + +[ Upstream commit 0a3212de8ab3e2ce5808c6265855e528d4a6767b ] + +Fix a typo of printing FLUSH_DELAYED_REFS event in flush_space() as +FLUSH_ELAYED_REFS. + +Reviewed-by: Johannes Thumshirn +Signed-off-by: Naohiro Aota +Reviewed-by: David Sterba +Signed-off-by: David Sterba +Signed-off-by: Sasha Levin +--- + include/trace/events/btrfs.h | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/include/trace/events/btrfs.h b/include/trace/events/btrfs.h +index ecd24c719de4..041be3ce1071 100644 +--- a/include/trace/events/btrfs.h ++++ b/include/trace/events/btrfs.h +@@ -95,7 +95,7 @@ struct btrfs_space_info; + EM( FLUSH_DELALLOC, "FLUSH_DELALLOC") \ + EM( FLUSH_DELALLOC_WAIT, "FLUSH_DELALLOC_WAIT") \ + EM( FLUSH_DELAYED_REFS_NR, "FLUSH_DELAYED_REFS_NR") \ +- EM( FLUSH_DELAYED_REFS, "FLUSH_ELAYED_REFS") \ ++ EM( FLUSH_DELAYED_REFS, "FLUSH_DELAYED_REFS") \ + EM( ALLOC_CHUNK, "ALLOC_CHUNK") \ + EM( ALLOC_CHUNK_FORCE, "ALLOC_CHUNK_FORCE") \ + EM( RUN_DELAYED_IPUTS, "RUN_DELAYED_IPUTS") \ +-- +2.35.1 + diff --git a/queue-5.10/f2fs-let-s-avoid-panic-if-extent_tree-is-not-created.patch b/queue-5.10/f2fs-let-s-avoid-panic-if-extent_tree-is-not-created.patch new file mode 100644 index 00000000000..86a49a01dad --- /dev/null +++ b/queue-5.10/f2fs-let-s-avoid-panic-if-extent_tree-is-not-created.patch @@ -0,0 +1,70 @@ +From b25dfc5a9a1d91bf1f24de2e6517d1af4337ef10 Mon Sep 17 00:00:00 2001 +From: Sasha Levin +Date: Wed, 21 Dec 2022 16:14:10 -0800 +Subject: f2fs: let's avoid panic if extent_tree is not created + +From: Jaegeuk Kim + +[ Upstream commit df9d44b645b83fffccfb4e28c1f93376585fdec8 ] + +This patch avoids the below panic. + +pc : __lookup_extent_tree+0xd8/0x760 +lr : f2fs_do_write_data_page+0x104/0x87c +sp : ffffffc010cbb3c0 +x29: ffffffc010cbb3e0 x28: 0000000000000000 +x27: ffffff8803e7f020 x26: ffffff8803e7ed40 +x25: ffffff8803e7f020 x24: ffffffc010cbb460 +x23: ffffffc010cbb480 x22: 0000000000000000 +x21: 0000000000000000 x20: ffffffff22e90900 +x19: 0000000000000000 x18: ffffffc010c5d080 +x17: 0000000000000000 x16: 0000000000000020 +x15: ffffffdb1acdbb88 x14: ffffff888759e2b0 +x13: 0000000000000000 x12: ffffff802da49000 +x11: 000000000a001200 x10: ffffff8803e7ed40 +x9 : ffffff8023195800 x8 : ffffff802da49078 +x7 : 0000000000000001 x6 : 0000000000000000 +x5 : 0000000000000006 x4 : ffffffc010cbba28 +x3 : 0000000000000000 x2 : ffffffc010cbb480 +x1 : 0000000000000000 x0 : ffffff8803e7ed40 +Call trace: + __lookup_extent_tree+0xd8/0x760 + f2fs_do_write_data_page+0x104/0x87c + f2fs_write_single_data_page+0x420/0xb60 + f2fs_write_cache_pages+0x418/0xb1c + __f2fs_write_data_pages+0x428/0x58c + f2fs_write_data_pages+0x30/0x40 + do_writepages+0x88/0x190 + __writeback_single_inode+0x48/0x448 + writeback_sb_inodes+0x468/0x9e8 + __writeback_inodes_wb+0xb8/0x2a4 + wb_writeback+0x33c/0x740 + wb_do_writeback+0x2b4/0x400 + wb_workfn+0xe4/0x34c + process_one_work+0x24c/0x5bc + worker_thread+0x3e8/0xa50 + kthread+0x150/0x1b4 + +Signed-off-by: Jaegeuk Kim +Signed-off-by: Sasha Levin +--- + fs/f2fs/extent_cache.c | 3 ++- + 1 file changed, 2 insertions(+), 1 deletion(-) + +diff --git a/fs/f2fs/extent_cache.c b/fs/f2fs/extent_cache.c +index bd16c78b5bf2..ad0b83a41226 100644 +--- a/fs/f2fs/extent_cache.c ++++ b/fs/f2fs/extent_cache.c +@@ -414,7 +414,8 @@ static bool f2fs_lookup_extent_tree(struct inode *inode, pgoff_t pgofs, + struct extent_node *en; + bool ret = false; + +- f2fs_bug_on(sbi, !et); ++ if (!et) ++ return false; + + trace_f2fs_lookup_extent_tree_start(inode, pgofs); + +-- +2.35.1 + diff --git a/queue-5.10/net-ethtool-ioctl-return-eopnotsupp-if-we-have-no-ph.patch b/queue-5.10/net-ethtool-ioctl-return-eopnotsupp-if-we-have-no-ph.patch new file mode 100644 index 00000000000..5ff2809b1c0 --- /dev/null +++ b/queue-5.10/net-ethtool-ioctl-return-eopnotsupp-if-we-have-no-ph.patch @@ -0,0 +1,38 @@ +From c859fc9d104658c8102152b3b59d5c02fdf09882 Mon Sep 17 00:00:00 2001 +From: Sasha Levin +Date: Mon, 26 Dec 2022 14:48:23 +0300 +Subject: net/ethtool/ioctl: return -EOPNOTSUPP if we have no phy stats + +From: Daniil Tatianin + +[ Upstream commit 9deb1e9fb88b1120a908676fa33bdf9e2eeaefce ] + +It's not very useful to copy back an empty ethtool_stats struct and +return 0 if we didn't actually have any stats. This also allows for +further simplification of this function in the future commits. + +Signed-off-by: Daniil Tatianin +Reviewed-by: Andrew Lunn +Signed-off-by: David S. Miller +Signed-off-by: Sasha Levin +--- + net/ethtool/ioctl.c | 3 ++- + 1 file changed, 2 insertions(+), 1 deletion(-) + +diff --git a/net/ethtool/ioctl.c b/net/ethtool/ioctl.c +index 47c2dd4a9b9f..12bf740e2fb3 100644 +--- a/net/ethtool/ioctl.c ++++ b/net/ethtool/ioctl.c +@@ -2052,7 +2052,8 @@ static int ethtool_get_phy_stats(struct net_device *dev, void __user *useraddr) + return n_stats; + if (n_stats > S32_MAX / sizeof(u64)) + return -ENOMEM; +- WARN_ON_ONCE(!n_stats); ++ if (WARN_ON_ONCE(!n_stats)) ++ return -EOPNOTSUPP; + + if (copy_from_user(&stats, useraddr, sizeof(stats))) + return -EFAULT; +-- +2.35.1 + diff --git a/queue-5.10/pnfs-filelayout-fix-coalescing-test-for-single-ds.patch b/queue-5.10/pnfs-filelayout-fix-coalescing-test-for-single-ds.patch new file mode 100644 index 00000000000..399f45548ac --- /dev/null +++ b/queue-5.10/pnfs-filelayout-fix-coalescing-test-for-single-ds.patch @@ -0,0 +1,49 @@ +From 01345bece604a3b2716dc2c05526d2e909837a4e Mon Sep 17 00:00:00 2001 +From: Sasha Levin +Date: Tue, 20 Dec 2022 12:31:29 -0500 +Subject: pNFS/filelayout: Fix coalescing test for single DS + +From: Olga Kornievskaia + +[ Upstream commit a6b9d2fa0024e7e399c26facd0fb466b7396e2b9 ] + +When there is a single DS no striping constraints need to be placed on +the IO. When such constraint is applied then buffered reads don't +coalesce to the DS's rsize. + +Signed-off-by: Olga Kornievskaia +Signed-off-by: Trond Myklebust +Signed-off-by: Sasha Levin +--- + fs/nfs/filelayout/filelayout.c | 8 ++++++++ + 1 file changed, 8 insertions(+) + +diff --git a/fs/nfs/filelayout/filelayout.c b/fs/nfs/filelayout/filelayout.c +index ae5ed3a07494..deecfb50dd7e 100644 +--- a/fs/nfs/filelayout/filelayout.c ++++ b/fs/nfs/filelayout/filelayout.c +@@ -783,6 +783,12 @@ filelayout_alloc_lseg(struct pnfs_layout_hdr *layoutid, + return &fl->generic_hdr; + } + ++static bool ++filelayout_lseg_is_striped(const struct nfs4_filelayout_segment *flseg) ++{ ++ return flseg->num_fh > 1; ++} ++ + /* + * filelayout_pg_test(). Called by nfs_can_coalesce_requests() + * +@@ -803,6 +809,8 @@ filelayout_pg_test(struct nfs_pageio_descriptor *pgio, struct nfs_page *prev, + size = pnfs_generic_pg_test(pgio, prev, req); + if (!size) + return 0; ++ else if (!filelayout_lseg_is_striped(FILELAYOUT_LSEG(pgio->pg_lseg))) ++ return size; + + /* see if req and prev are in the same stripe */ + if (prev) { +-- +2.35.1 + diff --git a/queue-5.10/rdma-srp-move-large-values-to-a-new-enum-for-gcc13.patch b/queue-5.10/rdma-srp-move-large-values-to-a-new-enum-for-gcc13.patch new file mode 100644 index 00000000000..1548fef5eda --- /dev/null +++ b/queue-5.10/rdma-srp-move-large-values-to-a-new-enum-for-gcc13.patch @@ -0,0 +1,66 @@ +From 639cce3e07b01c44b0a11de2c718ca532de2a970 Mon Sep 17 00:00:00 2001 +From: Sasha Levin +Date: Mon, 12 Dec 2022 13:04:11 +0100 +Subject: RDMA/srp: Move large values to a new enum for gcc13 + +From: Jiri Slaby (SUSE) + +[ Upstream commit 56c5dab20a6391604df9521f812c01d1e3fe1bd0 ] + +Since gcc13, each member of an enum has the same type as the enum [1]. And +that is inherited from its members. Provided these two: + SRP_TAG_NO_REQ = ~0U, + SRP_TAG_TSK_MGMT = 1U << 31 +all other members are unsigned ints. + +Esp. with SRP_MAX_SGE and SRP_TSK_MGMT_SQ_SIZE and their use in min(), +this results in the following warnings: + include/linux/minmax.h:20:35: error: comparison of distinct pointer types lacks a cast + drivers/infiniband/ulp/srp/ib_srp.c:563:42: note: in expansion of macro 'min' + + include/linux/minmax.h:20:35: error: comparison of distinct pointer types lacks a cast + drivers/infiniband/ulp/srp/ib_srp.c:2369:27: note: in expansion of macro 'min' + +So move the large values away to a separate enum, so that they don't +affect other members. + +[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=36113 + +Link: https://lore.kernel.org/r/20221212120411.13750-1-jirislaby@kernel.org +Signed-off-by: Jiri Slaby (SUSE) +Reviewed-by: Bart Van Assche +Signed-off-by: Leon Romanovsky +Signed-off-by: Sasha Levin +--- + drivers/infiniband/ulp/srp/ib_srp.h | 8 +++++--- + 1 file changed, 5 insertions(+), 3 deletions(-) + +diff --git a/drivers/infiniband/ulp/srp/ib_srp.h b/drivers/infiniband/ulp/srp/ib_srp.h +index 6818cac0a3b7..85bac20d9007 100644 +--- a/drivers/infiniband/ulp/srp/ib_srp.h ++++ b/drivers/infiniband/ulp/srp/ib_srp.h +@@ -62,9 +62,6 @@ enum { + SRP_DEFAULT_CMD_SQ_SIZE = SRP_DEFAULT_QUEUE_SIZE - SRP_RSP_SQ_SIZE - + SRP_TSK_MGMT_SQ_SIZE, + +- SRP_TAG_NO_REQ = ~0U, +- SRP_TAG_TSK_MGMT = 1U << 31, +- + SRP_MAX_PAGES_PER_MR = 512, + + SRP_MAX_ADD_CDB_LEN = 16, +@@ -79,6 +76,11 @@ enum { + sizeof(struct srp_imm_buf), + }; + ++enum { ++ SRP_TAG_NO_REQ = ~0U, ++ SRP_TAG_TSK_MGMT = BIT(31), ++}; ++ + enum srp_target_state { + SRP_TARGET_SCANNING, + SRP_TARGET_LIVE, +-- +2.35.1 + diff --git a/queue-5.10/selftests-bpf-check-null-propagation-only-neither-re.patch b/queue-5.10/selftests-bpf-check-null-propagation-only-neither-re.patch new file mode 100644 index 00000000000..e35cd1233e6 --- /dev/null +++ b/queue-5.10/selftests-bpf-check-null-propagation-only-neither-re.patch @@ -0,0 +1,92 @@ +From 8103e79150592eab30617c0adefc22210f42d4dd Mon Sep 17 00:00:00 2001 +From: Sasha Levin +Date: Thu, 22 Dec 2022 10:44:14 +0800 +Subject: selftests/bpf: check null propagation only neither reg is + PTR_TO_BTF_ID + +From: Hao Sun + +[ Upstream commit cedebd74cf3883f0384af9ec26b4e6f8f1964dd4 ] + +Verify that nullness information is not porpagated in the branches +of register to register JEQ and JNE operations if one of them is +PTR_TO_BTF_ID. Implement this in C level so we can use CO-RE. + +Signed-off-by: Hao Sun +Suggested-by: Martin KaFai Lau +Link: https://lore.kernel.org/r/20221222024414.29539-2-sunhao.th@gmail.com +Signed-off-by: Martin KaFai Lau +Signed-off-by: Sasha Levin +--- + .../bpf/prog_tests/jeq_infer_not_null.c | 9 ++++ + .../bpf/progs/jeq_infer_not_null_fail.c | 42 +++++++++++++++++++ + 2 files changed, 51 insertions(+) + create mode 100644 tools/testing/selftests/bpf/prog_tests/jeq_infer_not_null.c + create mode 100644 tools/testing/selftests/bpf/progs/jeq_infer_not_null_fail.c + +diff --git a/tools/testing/selftests/bpf/prog_tests/jeq_infer_not_null.c b/tools/testing/selftests/bpf/prog_tests/jeq_infer_not_null.c +new file mode 100644 +index 000000000000..3add34df5767 +--- /dev/null ++++ b/tools/testing/selftests/bpf/prog_tests/jeq_infer_not_null.c +@@ -0,0 +1,9 @@ ++// SPDX-License-Identifier: GPL-2.0 ++ ++#include ++#include "jeq_infer_not_null_fail.skel.h" ++ ++void test_jeq_infer_not_null(void) ++{ ++ RUN_TESTS(jeq_infer_not_null_fail); ++} +diff --git a/tools/testing/selftests/bpf/progs/jeq_infer_not_null_fail.c b/tools/testing/selftests/bpf/progs/jeq_infer_not_null_fail.c +new file mode 100644 +index 000000000000..f46965053acb +--- /dev/null ++++ b/tools/testing/selftests/bpf/progs/jeq_infer_not_null_fail.c +@@ -0,0 +1,42 @@ ++// SPDX-License-Identifier: GPL-2.0 ++ ++#include "vmlinux.h" ++#include ++#include "bpf_misc.h" ++ ++char _license[] SEC("license") = "GPL"; ++ ++struct { ++ __uint(type, BPF_MAP_TYPE_HASH); ++ __uint(max_entries, 1); ++ __type(key, u64); ++ __type(value, u64); ++} m_hash SEC(".maps"); ++ ++SEC("?raw_tp") ++__failure __msg("R8 invalid mem access 'map_value_or_null") ++int jeq_infer_not_null_ptr_to_btfid(void *ctx) ++{ ++ struct bpf_map *map = (struct bpf_map *)&m_hash; ++ struct bpf_map *inner_map = map->inner_map_meta; ++ u64 key = 0, ret = 0, *val; ++ ++ val = bpf_map_lookup_elem(map, &key); ++ /* Do not mark ptr as non-null if one of them is ++ * PTR_TO_BTF_ID (R9), reject because of invalid ++ * access to map value (R8). ++ * ++ * Here, we need to inline those insns to access ++ * R8 directly, since compiler may use other reg ++ * once it figures out val==inner_map. ++ */ ++ asm volatile("r8 = %[val];\n" ++ "r9 = %[inner_map];\n" ++ "if r8 != r9 goto +1;\n" ++ "%[ret] = *(u64 *)(r8 +0);\n" ++ : [ret] "+r"(ret) ++ : [inner_map] "r"(inner_map), [val] "r"(val) ++ : "r8", "r9"); ++ ++ return ret; ++} +-- +2.35.1 + diff --git a/queue-5.10/series b/queue-5.10/series new file mode 100644 index 00000000000..ae3d7118aaf --- /dev/null +++ b/queue-5.10/series @@ -0,0 +1,9 @@ +btrfs-fix-trace-event-name-typo-for-flush_delayed_re.patch +pnfs-filelayout-fix-coalescing-test-for-single-ds.patch +selftests-bpf-check-null-propagation-only-neither-re.patch +tools-virtio-initialize-spinlocks-in-vring_test.c.patch +net-ethtool-ioctl-return-eopnotsupp-if-we-have-no-ph.patch +rdma-srp-move-large-values-to-a-new-enum-for-gcc13.patch +btrfs-always-report-error-in-run_one_delayed_ref.patch +x86-asm-fix-an-assembler-warning-with-current-binuti.patch +f2fs-let-s-avoid-panic-if-extent_tree-is-not-created.patch diff --git a/queue-5.10/tools-virtio-initialize-spinlocks-in-vring_test.c.patch b/queue-5.10/tools-virtio-initialize-spinlocks-in-vring_test.c.patch new file mode 100644 index 00000000000..b2f6f39457b --- /dev/null +++ b/queue-5.10/tools-virtio-initialize-spinlocks-in-vring_test.c.patch @@ -0,0 +1,48 @@ +From 4f01931fd87e20aab431f4ae209beaad456783b9 Mon Sep 17 00:00:00 2001 +From: Sasha Levin +Date: Wed, 12 Oct 2022 08:29:49 +0200 +Subject: tools/virtio: initialize spinlocks in vring_test.c +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +From: Ricardo Cañuelo + +[ Upstream commit c262f75cb6bb5a63828e72ce3b8fe808e5029479 ] + +The virtio_device vqs_list spinlocks must be initialized before use to +prevent functions that manipulate the device virtualqueues, such as +vring_new_virtqueue(), from blocking indefinitely. + +Signed-off-by: Ricardo Cañuelo +Message-Id: <20221012062949.1526176-1-ricardo.canuelo@collabora.com> +Signed-off-by: Michael S. Tsirkin +Reviewed-by: Xuan Zhuo +Signed-off-by: Sasha Levin +--- + tools/virtio/vringh_test.c | 2 ++ + 1 file changed, 2 insertions(+) + +diff --git a/tools/virtio/vringh_test.c b/tools/virtio/vringh_test.c +index fa87b58bd5fa..98ff808d6f0c 100644 +--- a/tools/virtio/vringh_test.c ++++ b/tools/virtio/vringh_test.c +@@ -308,6 +308,7 @@ static int parallel_test(u64 features, + + gvdev.vdev.features = features; + INIT_LIST_HEAD(&gvdev.vdev.vqs); ++ spin_lock_init(&gvdev.vdev.vqs_list_lock); + gvdev.to_host_fd = to_host[1]; + gvdev.notifies = 0; + +@@ -455,6 +456,7 @@ int main(int argc, char *argv[]) + getrange = getrange_iov; + vdev.features = 0; + INIT_LIST_HEAD(&vdev.vqs); ++ spin_lock_init(&vdev.vqs_list_lock); + + while (argv[1]) { + if (strcmp(argv[1], "--indirect") == 0) +-- +2.35.1 + diff --git a/queue-5.10/x86-asm-fix-an-assembler-warning-with-current-binuti.patch b/queue-5.10/x86-asm-fix-an-assembler-warning-with-current-binuti.patch new file mode 100644 index 00000000000..5725a07d937 --- /dev/null +++ b/queue-5.10/x86-asm-fix-an-assembler-warning-with-current-binuti.patch @@ -0,0 +1,34 @@ +From 3665a471c3bf50e6318df8db3dddb02e8281b4d9 Mon Sep 17 00:00:00 2001 +From: Sasha Levin +Date: Tue, 3 Jan 2023 10:24:11 -0500 +Subject: x86/asm: Fix an assembler warning with current binutils + +From: Mikulas Patocka + +[ Upstream commit 55d235361fccef573990dfa5724ab453866e7816 ] + +Fix a warning: "found `movsd'; assuming `movsl' was meant" + +Signed-off-by: Mikulas Patocka +Signed-off-by: Ingo Molnar +Cc: linux-kernel@vger.kernel.org +Signed-off-by: Sasha Levin +--- + arch/x86/lib/iomap_copy_64.S | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/arch/x86/lib/iomap_copy_64.S b/arch/x86/lib/iomap_copy_64.S +index a1f9416bf67a..6ff2f56cb0f7 100644 +--- a/arch/x86/lib/iomap_copy_64.S ++++ b/arch/x86/lib/iomap_copy_64.S +@@ -10,6 +10,6 @@ + */ + SYM_FUNC_START(__iowrite32_copy) + movl %edx,%ecx +- rep movsd ++ rep movsl + RET + SYM_FUNC_END(__iowrite32_copy) +-- +2.35.1 +