<div dir="ltr"><div>May be that one could be removed?  We may just put ASSERT_SERIALIZED or ASSERT_NOT_SERIALIZED there, since we have setup the serializer properly for wlan in the last release cycle.<br><br></div>Best Regards,<br>
sephe<br><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 8, 2013 at 8:55 PM, Sascha Wildner <span dir="ltr"><<a href="mailto:saw@online.de" target="_blank">saw@online.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Fri, 08 Nov 2013 13:26:45 +0100, Andrey Oktyabrskiy <<a href="mailto:ano@bestmx.net" target="_blank">ano@bestmx.net</a>> wrote:<br>

<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
$ make buildkernel KERNCONF=ANO<br>
...<br></div><div class="im">
/usr/src/sys/dev/netif/iwn/if_<u></u>iwn.c: In function 'iwn_read_firmware':<br>
/usr/src/sys/dev/netif/iwn/if_<u></u>iwn.c:5676:2: error: implicit declaration of function 'IS_SERIALIZED' [-Werror=implicit-function-<u></u>declaration]<br>
/usr/src/sys/dev/netif/iwn/if_<u></u>iwn.c:5676:2: error: nested extern declaration of 'IS_SERIALIZED' [-Werror=nested-externs]<br>
cc1: all warnings being treated as errors<br>
*** Error code 1<br>
</div></blockquote>
<br>
Yeah, it's a known issue and comes from compiling without INVARIANTS. IIRC I looked at it once and it didn't look so easy. It is also mentioned in <a href="http://bugs.dragonflybsd.org/issues/2449" target="_blank">http://bugs.dragonflybsd.org/<u></u>issues/2449</a> (unfortunately closed prematurely).<br>

<br>
CC'ing Matt since he both added the iwn code in question (<u></u>b8b498ed88a72ccd6a0b0fd0d10a20<u></u>61576f3a47) and added iwn to the default configs too (<u></u>ae80e77cae0ebec810b46ecf9bf924<u></u>28e28b8f7c). :)<br>

<br>
We could #define INVARIANTS for the iwn driver only, but it would be kind of a hack. Any better ideas?<span class="HOEnZb"><font color="#888888"><br>
<br>
Sascha<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>Tomorrow Will Never Die
</div>