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 ;)
next prev parent 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