<div dir="ltr"><div class="gmail_default" style="font-size:small"></div><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Nov 3, 2021 at 10:09 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</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">Hi!<br>
> Um, I can't recall why the V3 development for .arch<br>
> was suspended.<br>
> <br>
> Maybe we thought there is not much sense to replace<br>
> ifdef __arch__ from code at that moment.<br>
> <br>
> Now, if that can benefit the tst_test metadata more in the next runltp-ng.<br>
> Should we keep going to work out the patch V3?<br>
> <br>
> @Cyril Hrubis <<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>>  any suggestions?<br>
<br>
Hmm, I guess that I said that it still makes to add the metadata, at<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Agreed.</div><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">
least array of supported architectures in the test_test structure would<br>
be a good addition.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I guess defining .arch as a string and making a valid check will be enough.</div><div class="gmail_default" style="font-size:small">Array for that sounds a bit complicated in use.</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">
<br>
However the hard part would be keeping the actual code and metadata in<br>
sync, we still have to keep the ifdefs in the code.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Yes, some inline assemble<span style="color:rgb(51,51,51);font-size:14px"> require ifdefs.</span><br></div><div class="gmail_default" style="font-size:small"><span style="color:rgb(51,51,51);font-size:14px"><br></span></div><div class="gmail_default" style="font-size:small"><span style="color:rgb(51,51,51);font-size:14px">Btw, I look back at the reviews and find Jan said:</span></div><div class="gmail_default" style="font-size:small"><span style="color:rgb(51,51,51);font-size:14px">    "</span>I can see how tst_on_arch() would be useful. Test is valid </div><div class="gmail_default" style="font-size:small">     on all arches, but needs different input/constants/code/etc."</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">That may be a slight reason for keeping tst_on_arch.</div></div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>