Re: [PATCH v2] x86/Intel: avoid infinite recursion in i386_intel_simplify_symbol() (again)

Project / Subsystem

binutils / x86/intel

Date

2026-05-18

Proposer

Jan Beulich <jbeulich@suse.com>

Source type

public_inbox

Consensus

Proposed

Sentiment

/10

All attributes

project
binutils
subsystem
x86/intel
patch_id
discussion_id
44e7ed68-60fe-4e72-9391-6f124030d975@suse.com
source_type
public_inbox
title
Re: [PATCH v2] x86/Intel: avoid infinite recursion in i386_intel_simplify_symbol() (again)
headline
Questions loop termination in i386_intel_simplify_symbol()
tldr
Jan Beulich questions the correctness of a patch aiming to prevent infinite recursion in `i386_intel_simplify_symbol()`.
proposer
Jan Beulich <jbeulich@suse.com>
consensus
Proposed
outcome
proposed
sentiment_score
technical_tradeoffs
[]
series_id
series_role
standalone
series_parts
[]
tags
  • binutils
  • x86
  • infinite recursion
  • assembly
  • bugfix
bugzilla_url
date
2026-05-18T00:00:00.000Z

Re: [PATCH v2] x86/Intel: avoid infinite recursion in i386_intel_simplify_symbol() (again)

Jan Beulich expresses concerns regarding the correctness of a patch designed to prevent infinite recursion within the i386_intel_simplify_symbol() function. They suggest that the current approach of cutting the loop and only looking for O_symbol might be insufficient. The author acknowledges the lack of a comprehensive solution and points to the need for a more robust expression walking mechanism, similar to what’s used for Intel syntax parsing.