<div dir="ltr"><div dir="ltr"></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 24, 2021 at 1:26 AM Valdis Klētnieks <<a href="mailto:valdis.kletnieks@vt.edu">valdis.kletnieks@vt.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, 23 Mar 2021 22:36:33 -0400, Aruna Hewapathirane said:<br>
<br>
> I also see:<br>
> CC      drivers/cpufreq/cpufreq_ondemand.o<br>
> drivers/cpufreq/cpufreq_ondemand.c: In function ‘od_set_powersave_bias’:<br>
> drivers/cpufreq/cpufreq_ondemand.c:446:1: warning: the frame size of 1032<br>
> bytes is larger than 1024 bytes [-Wframe-larger-than=]<br>
>   446 | }<br>
>       | ^<br>
> This is what causes the compile time errors with possible missing firmware<br>
<br>
No it's not. That's just a *warning* and doesn't stop the build.  The reason<br>
that we have that warning is because on most architectures, the entire kernel<br>
stack has to fit into 2 contiguous 4k pages.<br>
<br></blockquote><div>Oh dear that's right it does not stop the build what was I thinking ..  <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
And a perfectly legal stack can be something like TCP delivering an NFS packet<br>
which then goes to the NFS code, which then hands it off to the VFS layer,<br>
which checks the page cache, which hands it off to XFS, which schedules I/O to<br>
the filesystem, which gets handed to the block subsystem, which realizes that<br>
the target partition is an LVM partition, and LVM realizes that the "physical<br>
volume" is actually a dm-crypt, and then dm-crypt does some cryptography and<br>
calls the block subsystem to write to the physical disk. Feel free to check the code<br>
path yourself... :)<br>
<br></blockquote><div><br></div><div>Oh my... and oh dear ( again! )<br></div><div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
And all of that has to fit in 8K of stack. That's why we warn when things have<br>
a large stack frame entry.<br></blockquote><div><br></div><div>I still don't understand why the 8K barrier ? Why can't we make it say 16K ?  <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Anyhow, "possible missing firmware" isn't even at compile time - it's during<br>
the build of the initramfs at *install* time.  And the most likely cause of<br>
that is that missing external firmware that's not GPL - it's all confined to 2<br>
graphics drivers - amdgpu and i915, so not at all related to a warning against<br>
cpufreq...<br>
<br></blockquote><div>Understood and my apologies :-)<br></div><div><br></div><div> </div></div></div>