[5.16,137/285] x86: Annotate call_on_stack()

Message ID	20220412062947.621591678@linuxfoundation.org
State	New
Headers	show Return-Path: <stable-owner@kernel.org> From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, stable@vger.kernel.org, "Peter Zijlstra (Intel)" <peterz@infradead.org>, Josh Poimboeuf <jpoimboe@redhat.com>, Sasha Levin <sashal@kernel.org> Subject: [PATCH 5.16 137/285] x86: Annotate call_on_stack() Date: Tue, 12 Apr 2022 08:29:54 +0200 Message-Id: <20220412062947.621591678@linuxfoundation.org> In-Reply-To: <20220412062943.670770901@linuxfoundation.org> References: <20220412062943.670770901@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	None \| expand [5.16,002/285] um: fix and optimize xor select template for CONFIG64 and timetravel mode [5.16,004/285] KVM: x86/pmu: Use different raw event masks for AMD and Intel [5.16,009/285] drm: Add orientation quirk for GPD Win Max [5.16,010/285] ath5k: fix OOB in ath5k_eeprom_read_pcal_info_5111 [5.16,012/285] drm/edid: remove non_desktop quirk for HPN-3515 and LEN-B800. [5.16,016/285] drm/amd/display: Use PSR version selected during set_psr_caps [5.16,017/285] usb: gadget: tegra-xudc: Do not program SPARAM [5.16,018/285] usb: gadget: tegra-xudc: Fix control endpoints definitions [5.16,019/285] usb: cdnsp: fix cdnsp_decode_trb function to properly handle ret value [5.16,021/285] selftests, xsk: Fix bpf_res cleanup test [5.16,023/285] drm/amdkfd: Ensure mm remain valid in svm deferred_list work [5.16,024/285] drm/amdkfd: svm range restore work deadlock when process exit [5.16,027/285] ath11k: pci: fix crash on suspend if board file is not found [5.16,028/285] ath11k: mhi: use mhi_sync_power_up() [5.16,029/285] net/smc: Send directly when TCP_CORK is cleared [5.16,030/285] drm/bridge: Add missing pm_runtime_put_sync [5.16,032/285] scsi: mvsas: Replace snprintf() with sysfs_emit() [5.16,033/285] scsi: bfa: Replace snprintf() with sysfs_emit() [5.16,034/285] drm/v3d: fix missing unlock [5.16,035/285] power: supply: axp20x_battery: properly report current when discharging [5.16,036/285] mt76: mt7921: fix crash when startup fails. [5.16,037/285] mt76: dma: initialize skip_unmap in mt76_dma_rx_fill [5.16,039/285] libbpf: Fix build issue with llvm-readelf [5.16,044/285] scsi: smartpqi: Fix rmmod stack trace [5.16,046/285] PCI: aardvark: Fix support for MSI interrupts [5.16,048/285] usb: ehci: add pci device support for Aspeed platforms [5.16,049/285] KVM: arm64: Do not change the PMU event filter after a VCPU has run [5.16,052/285] PCI: pciehp: Add Qualcomm quirk for Command Completed erratum [5.16,053/285] scsi: mpi3mr: Fix reporting of actual data transfer size [5.16,058/285] net/mlx5e: Disable TX queues before registering the netdev [5.16,059/285] usb: dwc3: pci: Set the swnode from inside dwc3_pci_quirks() [5.16,060/285] iwlwifi: mvm: Correctly set fragmented EBS [5.16,061/285] iwlwifi: fix small doc mistake for iwl_fw_ini_addr_val [5.16,062/285] iwlwifi: mvm: move only to an enabled channel [5.16,064/285] rtw89: fix RCU usage in rtw89_core_txq_push() [5.16,067/285] dm: requeue IO if mapping table not yet available [5.16,068/285] drm/amdkfd: make CRAT table missing message informational only [5.16,069/285] vfio/pci: Stub vfio_pci_vga_rw when !CONFIG_VFIO_PCI_VGA [5.16,070/285] scsi: pm8001: Fix pm80xx_pci_mem_copy() interface [5.16,073/285] scsi: pm8001: Fix tag leaks on error [5.16,075/285] mt76: mt7915: fix injected MPDU transmission to not use HW A-MSDU [5.16,076/285] powerpc/64s/hash: Make hash faults work in NMI context [5.16,086/285] Bluetooth: use memset avoid memory leaks [5.16,087/285] bnxt_en: Eliminate unintended link toggle during FW reset [5.16,088/285] PCI: endpoint: Fix misused goto label [5.16,089/285] MIPS: fix fortify panic when copying asm exception handlers [5.16,093/285] can: isotp: set default value for N_As to 50 micro seconds [5.16,095/285] riscv: Fixed misaligned memory access. Fixed pointer comparison. [5.16,097/285] net: limit altnames to 64k total [5.16,099/285] platform/x86: hp-wmi: Fix SW_TABLET_MODE detection method [5.16,100/285] platform/x86: hp-wmi: Fix 0x05 error code reported by several WMI calls [5.16,101/285] net: sfp: add 2500base-X quirk for Lantech SFP module [5.16,106/285] Bluetooth: Fix use after free in hci_send_acl [5.16,107/285] netfilter: conntrack: revisit gc autotuning [5.16,110/285] ceph: fix memory leak in ceph_readdir when note_last_dentry returns error [5.16,112/285] init/main.c: return 1 from handled __setup() functions [5.16,114/285] clk: si5341: fix reported clk_rate when output divider is 2 [5.16,121/285] clk: rockchip: drop CLK_SET_RATE_PARENT from dclk_vop* on rk3568 [5.16,125/285] w1: w1_therm: fixes w1_seq for ds28ea00 sensors [5.16,126/285] NFSv4.2: fix reference count leaks in _nfs42_proc_copy_notify() [5.16,127/285] NFSv4: Protect the state recovery thread against direct reclaim [5.16,128/285] habanalabs: fix possible memory leak in MMU DR fini [5.16,130/285] xen: delay xen_hvm_init_time_ops() if kdump is boot on vcpu>=32 [5.16,131/285] clk: ti: Preserve node in ti_dt_clocks_register() [5.16,133/285] SUNRPC/xprt: async tasks mustnt block waiting for memory [5.16,135/285] NFS: swap IO handling is slightly different for O_DIRECT IO [5.16,137/285] x86: Annotate call_on_stack() [5.16,139/285] serial: samsung_tty: do not unlock port->lock for uart_write_wakeup() [5.16,143/285] NFS: nfsiod should not block forever in mempool_alloc() [5.16,144/285] NFS: Avoid writeback threads getting stuck in mempool_alloc() [5.16,146/285] parisc: Fix CPU affinity for Lasi, WAX and Dino chips [5.16,148/285] mm: fix race between MADV_FREE reclaim and blkdev direct IO read [5.16,150/285] rtc: Check return value from mc146818_get_time() [5.16,152/285] drm/amdgpu: fix off by one in amdgpu_gfx_kiq_acquire() [5.16,154/285] Revert "NFSv4: Handle the special Linux file open access mode" [5.16,155/285] NFSv4: fix open failure with O_ACCMODE flag [5.16,157/285] scsi: core: Fix sbitmap depth in scsi_realloc_sdev_budget_map() [5.16,158/285] scsi: zorro7xx: Fix a resource leak in zorro7xx_remove_one() [5.16,160/285] net: sfc: add missing xdp queue reinitialization [5.16,161/285] net/tls: fix slab-out-of-bounds bug in decrypt_internal [5.16,164/285] ice: Clear default forwarding VSI during VSI release [5.16,165/285] mctp: Fix check for dev_hard_header() result [5.16,167/285] net: ipv4: fix route with nexthop object delete warning [5.16,168/285] net: stmmac: Fix unset max_speed difference between DT and non-DT platforms [5.16,171/285] drm/imx: dw_hdmi-imx: Fix bailout in error cases of probe [5.16,173/285] regulator: atc260x: Fix missing active_discharge_on setting [5.16,174/285] arch/arm64: Fix topology initialization for core scheduling [5.16,176/285] bnxt_en: reserve space inside receive page for skb_shared_info [5.16,178/285] sfc: Do not free an empty page_ring [5.16,180/285] RDMA/mlx5: Add a missing update of cache->last_add [5.16,181/285] IB/cm: Cancel mad on the DREQ event when the state is MRA_REP_RCVD [5.16,183/285] sctp: count singleton chunks in assoc user stats [5.16,185/285] ice: Set txq_teid to ICE_INVAL_TEID on ring creation [5.16,187/285] ipv6: Fix stats accounting in ip6_pkt_drop [5.16,188/285] ice: synchronize_rcu() when terminating rings [5.16,190/285] ice: clear cmd_type_offset_bsz for TX rings [5.16,193/285] net: openvswitch: fix leak of nested actions [5.16,194/285] rxrpc: fix a race in rxrpc_exit_net() [5.16,195/285] net: sfc: fix using uninitialized xdp tx_queue [5.16,200/285] drbd: Fix five use after free bugs in get_initial_state [5.16,206/285] SUNRPC: Handle low memory situations in call_status() [5.16,208/285] iommu/omap: Fix regression in probe for NULL pointer dereference [5.16,213/285] scsi: mpt3sas: Fix use after free in _scsih_expander_node_remove() [5.16,214/285] scsi: ufs: ufs-pci: Add support for Intel MTL [5.16,215/285] Revert "mmc: sdhci-xenon: fix annoying 1.8V regulator warning" [5.16,219/285] mmc: core: Fixup support for writeback-cache for eMMC and SD [5.16,220/285] lz4: fix LZ4_decompress_safe_partial read out of bound [5.16,222/285] mmmremap.c: avoid pointless invalidate_range_start/end on mremap(old_size=0) [5.16,223/285] mm/mempolicy: fix mpol_new leak in shared_policy_replace [5.16,226/285] io_uring: implement compat handling for IORING_REGISTER_IOWQ_AFF [5.16,228/285] x86/pm: Save the MSR validity status at context setup [5.16,229/285] x86/speculation: Restore speculation related MSRs during S3 resume [5.16,233/285] btrfs: remove device item and update super block in the same transaction [5.16,234/285] btrfs: avoid defragging extents whose next extents are not targets [5.16,235/285] btrfs: prevent subvol with swapfile from being deleted [5.16,236/285] spi: core: add dma_map_dev for __spi_unmap_msg() [5.16,237/285] qed: fix ethtool register dump [5.16,240/285] drbd: fix an invalid memory access caused by incorrect use of list iterator [5.16,241/285] gpio: Restrict usage of GPIO chip irq members before initialization [5.16,245/285] ata: sata_dwc_460ex: Fix crash due to OOB write [5.16,247/285] perf/core: Inherit event_caps [5.16,249/285] fbdev: Fix unregistering of framebuffers without device [5.16,255/285] drm/amdgpu/vcn: Fix the register setting for vcn1 [5.16,258/285] drm/amdgpu: dont use BACO for reset in S3 [5.16,259/285] SUNRPC: Ensure we flush any closed sockets before xs_xprt_free() [5.16,260/285] KVM: SVM: Allow AVIC support on system w/ physical APIC ID > 255 [5.16,262/285] net/smc: send directly on setting TCP_NODELAY [5.16,263/285] Revert "selftests: net: Add tls config dependency for tls selftests" [5.16,265/285] selftests/bpf: Fix u8 narrow load checks for bpf_sk_lookup remote_port [5.16,266/285] rtc: mc146818-lib: fix signedness bug in mc146818_get_time() [5.16,267/285] perf build: Dont use -ffat-lto-objects in the python feature test when building wi... [5.16,269/285] tools build: Filter out options and warnings not supported by clang [5.16,270/285] tools build: Use $(shell ) instead of `` to get embedded libperls ccopts [5.16,271/285] dmaengine: Revert "dmaengine: shdma: Fix runtime PM imbalance on error" [5.16,274/285] x86/fpu/xstate: Fix the ARCH_REQ_XCOMP_PERM implementation [5.16,276/285] powerpc: Fix virt_addr_valid() for 64-bit Book3E & 32-bit [5.16,278/285] x86/bug: Prevent shadowing in __WARN_FLAGS [5.16,279/285] sched: Teach the forced-newidle balancer about CPU affinity limitation. [5.16,281/285] irqchip/gic-v4: Wait for GICR_VPENDBASER.Dirty to clear before descheduling [5.16,283/285] irqchip/gic, gic-v3: Prevent GSI to SGI translations [5.16,284/285] mm/sparsemem: fix mem_section will never be NULL gcc 12 warning

Message ID

20220412062947.621591678@linuxfoundation.org

State

New

Headers

From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: linux-kernel@vger.kernel.org
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        stable@vger.kernel.org,
        "Peter Zijlstra (Intel)" <peterz@infradead.org>,
        Josh Poimboeuf <jpoimboe@redhat.com>,
        Sasha Levin <sashal@kernel.org>
Subject: [PATCH 5.16 137/285] x86: Annotate call_on_stack()
Date: Tue, 12 Apr 2022 08:29:54 +0200
Message-Id: <20220412062947.621591678@linuxfoundation.org>
In-Reply-To: <20220412062943.670770901@linuxfoundation.org>
References: <20220412062943.670770901@linuxfoundation.org>
User-Agent: quilt/0.66
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

Series

None | expand

Commit Message

Greg Kroah-Hartman April 12, 2022, 6:29 a.m. UTC

From: Peter Zijlstra <peterz@infradead.org>

[ Upstream commit be0075951fde739f14ee2b659e2fd6e2499c46c0 ]

vmlinux.o: warning: objtool: page_fault_oops()+0x13c: unreachable instruction

0000 000000000005b460 <page_fault_oops>:
...
0128    5b588:  49 89 23                mov    %rsp,(%r11)
012b    5b58b:  4c 89 dc                mov    %r11,%rsp
012e    5b58e:  4c 89 f2                mov    %r14,%rdx
0131    5b591:  48 89 ee                mov    %rbp,%rsi
0134    5b594:  4c 89 e7                mov    %r12,%rdi
0137    5b597:  e8 00 00 00 00          call   5b59c <page_fault_oops+0x13c>    5b598: R_X86_64_PLT32   handle_stack_overflow-0x4
013c    5b59c:  5c                      pop    %rsp

vmlinux.o: warning: objtool: sysvec_reboot()+0x6d: unreachable instruction

0000 00000000000033f0 <sysvec_reboot>:
...
005d     344d:  4c 89 dc                mov    %r11,%rsp
0060     3450:  e8 00 00 00 00          call   3455 <sysvec_reboot+0x65>        3451: R_X86_64_PLT32    irq_enter_rcu-0x4
0065     3455:  48 89 ef                mov    %rbp,%rdi
0068     3458:  e8 00 00 00 00          call   345d <sysvec_reboot+0x6d>        3459: R_X86_64_PC32     .text+0x47d0c
006d     345d:  e8 00 00 00 00          call   3462 <sysvec_reboot+0x72>        345e: R_X86_64_PLT32    irq_exit_rcu-0x4
0072     3462:  5c                      pop    %rsp

Both cases are due to a call_on_stack() calling a __noreturn function.
Since that's an inline asm, GCC can't do anything about the
instructions after the CALL. Therefore put in an explicit
ASM_REACHABLE annotation to make sure objtool and gcc are consistently
confused about control flow.

Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Acked-by: Josh Poimboeuf <jpoimboe@redhat.com>
Link: https://lore.kernel.org/r/20220308154319.468805622@infradead.org
Signed-off-by: Sasha Levin <sashal@kernel.org>
---
 arch/x86/include/asm/irq_stack.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/x86/include/asm/irq_stack.h b/arch/x86/include/asm/irq_stack.h
index ae9d40f6c706..05af249d6bec 100644
--- a/arch/x86/include/asm/irq_stack.h
+++ b/arch/x86/include/asm/irq_stack.h
@@ -99,7 +99,8 @@ 
 }
 
 #define ASM_CALL_ARG0							\
-	"call %P[__func]				\n"
+	"call %P[__func]				\n"		\
+	ASM_REACHABLE
 
 #define ASM_CALL_ARG1							\
 	"movq	%[arg1], %%rdi				\n"		\

[5.16,137/285] x86: Annotate call_on_stack()

Commit Message

Patch