public inbox for linux-next@vger.kernel.org
 help / color / mirror / Atom feed
From: "Amadeusz Sławiński" <amade@asmblr.net>
To: Cezary Rojewski <cezary.rojewski@intel.com>
Cc: Brahmajit Das <listout@listout.xyz>,
	Mark Brown <broonie@kernel.org>, Takashi Iwai <tiwai@suse.de>,
	<linux-sound@vger.kernel.org>, <linux-next@vger.kernel.org>,
	<liam.r.girdwood@linux.intel.com>,
	<peter.ujfalusi@linux.intel.com>, <tiwai@suse.com>,
	Richard Biener <rguenther@suse.de>,
	Richard Biener <rguenth@gcc.gnu.org>
Subject: Re: [PATCH] ASoC: Intel: avs: replace strcmp with sysfs_streq
Date: Tue, 30 Dec 2025 13:25:42 +0100	[thread overview]
Message-ID: <20251230132542.27c99ab5@fedora> (raw)
In-Reply-To: <4b85c90a-727e-43a2-b1bc-91dd00045e90@intel.com>

On Tue, 30 Dec 2025 10:36:27 +0100
Cezary Rojewski <cezary.rojewski@intel.com> wrote:

> On 2025-12-30 9:32 AM, Brahmajit Das wrote:
> > 
> > Not sure if it would help but I tracked or narrowed down the issue to
> > this section of the code in the avs_condpaths_walk function.
> > 
> > 
> > 	if (avs_tplg_path_template_id_equal(&template->source,
> > 										&template->sink) && dir)
> > 		continue;
> > 
> > If I just comment that code segment out, then it builds successfully;
> > both with and without the hard coded name length.  
> 
> Outstanding! Thank you for this exercise.

Yes, I also managed to reproduce it on my machine.

You can also just reverse the order in
avs_tplg_path_template_id_equal() to:
return !strcmp(id->tplg_name, id2->tplg_name) && id->id == id2->id;
which for some reason makes the error disappear.

> > I was also looking into GCC bugzilla and there seems to lot of reports
> > saying these kind of error could be bogus or false positive. I'm not
> > familiar with GCC internal or experienced in compilers to say/understand
> > whether this case in particular is a false positive or not.  
> 
> Wouldn't be the first time, though having successful builds ain't wrong 
> either. After all we have git log and incremental changes/reverts are 
> not a problem.
> 

I've bisected it down on GCC side to:
https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;a=commitdiff;h=fc62834533f357125b9c1934f80c2ba249adbf9e
in particular it seems to be caused by code added in gcc/tree-ssa-pre.cc
I'm not that familiar with how GCC works, but to me it seems like maybe
it loses size information during this conversion.

I would make normal bug report, but apparently I can't just make GCC
bugzilla account, so adding Richard to CC, while I go through request
process for an account.

@Richard
Hi, we seem to have hit a slight problem while building kernel with pre
release GCC caused by one of your patches, full thread is available
here:
https://lore.kernel.org/linux-sound/20251221185531.6453-1-listout@listout.xyz/T/#u
and my reasoning above.

> > BTW I can't seem to send the email to Amadeusz, hence removing them from
> > the CC-list  
> >> msmtp: recipient address amade@asmblr.net not accepted by the server  
> 
> That's weird. The @linux.intel.com is gone but what you mentioned is the 
> correct one.

Yup, that's weird, but nothing lost, I'm subscribed to mailing list ;)

  reply	other threads:[~2025-12-30 12:25 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-21 18:55 [PATCH] ASoC: Intel: avs: replace strcmp with sysfs_streq Brahmajit Das
2025-12-23 17:07 ` Mark Brown
2025-12-23 18:24   ` Amadeusz Sławiński
2025-12-23 20:57     ` Brahmajit Das
2025-12-24 10:05       ` Takashi Iwai
2025-12-24 11:35         ` Mark Brown
2025-12-29 10:03           ` Cezary Rojewski
2025-12-30  7:59             ` Brahmajit Das
2025-12-30  8:32             ` Brahmajit Das
2025-12-30  9:36               ` Cezary Rojewski
2025-12-30 12:25                 ` Amadeusz Sławiński [this message]
2025-12-30 14:47                   ` Richard Biener

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20251230132542.27c99ab5@fedora \
    --to=amade@asmblr.net \
    --cc=broonie@kernel.org \
    --cc=cezary.rojewski@intel.com \
    --cc=liam.r.girdwood@linux.intel.com \
    --cc=linux-next@vger.kernel.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=listout@listout.xyz \
    --cc=peter.ujfalusi@linux.intel.com \
    --cc=rguenth@gcc.gnu.org \
    --cc=rguenther@suse.de \
    --cc=tiwai@suse.com \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox