<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size:14pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">
<meta content="text/html; charset=UTF-8">
<div dir="ltr">Dear <span>Håkan</span>,
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size: 14pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, "EmojiFont", "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;">
<p><br>
</p>
<p>I now started with the software package from ~two months ago and step-by-step replaced it with the most recent versions. Everything is fine until I add the new version of NURDLIB (using the
<span>SIS3316_check_hit_reworked_rebased branch). Then I get in R3BFUSER the compile error.</span></p>
<p><span><br>
</span></p>
<p><span>Implementing the first fixes suggested by you elimates this error on the RIO4:</span></p>
<p><span><br>
</span></p>
<p><span><font size="2"><span style="font-size:10pt;">it looks like with a change of the second parameter of log_callback (both
<br>
in prototype an definition) of r3bfuser/f_user.c from unsigned to int,</span></font><br>
</span></p>
<p><br>
</p>
<p>But on startup of the DAQ I get a crash. Then we have the second fix:</p>
<p><br>
</p>
<p><font size="2"><span style="font-size:10pt;">Anyhow, I suspect that if you change in<br>
include/nurdlib/crate.h<br>
in the prototype of crate_dt_release_set_func to have<br>
FUNC_NONNULL((1,2)) instead of FUNC_NONNULL(())<br>
then it might work.<br>
</span></font><br>
</p>
<p>This still results in a crash of DAQ, as g_counter_ms in f_user.c is zero at line 981. The immediate crash of DAQ on startup is avoided by a workaround:</p>
<p><br>
</p>
<p></p>
<div><span style="font-size: 10pt;"> /* diff = crate_counter_get_diff(g_counter_ms); */</span><br>
<span style="font-size: 10pt;"> diff = 1;</span></div>
<br>
<p></p>
<p>And, (!) contrary to what I thought to having found yesterday (!), with this modifications the DAQ is now running as expected also with a SIS3316 module being enabled.</p>
<p><br>
</p>
<p><br>
</p>
<p><br>
</p>
<p>Thus, it looks like that all problems seem to originate from a change in NURDLIB.<br>
</p>
<p><br>
</p>
<p><br>
</p>
<p>I hope this helps with fixing the underlying issues.</p>
<p><br>
</p>
<p><br>
</p>
<p><br>
</p>
<p>Best greetings</p>
<p>Günter<br>
</p>
<p><br>
</p>
<br>
</div>
<hr style="display:inline-block; width:98%" tabindex="-1">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>Von:</b> subexp-daq <subexp-daq-bounces@lists.chalmers.se> im Auftrag von Håkan T Johansson <f96hajo@chalmers.se><br>
<b>Gesendet:</b> Donnerstag, 18. Juli 2024 17:39:43<br>
<b>An:</b> Discuss use of Nurdlib, TRLO II, drasi and UCESB.<br>
<b>Betreff:</b> Re: [subexp-daq] New warning meassage in R3BFUSER</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt">
<div class="PlainText"><br>
Dear Günter,<br>
<br>
thanks! Yes, what I was looking for.<br>
<br>
r3bfuser - same as new version (so should not be at fault ;) )<br>
<br>
Good news (to me) is that the nurdlib is in the same repo.<br>
<br>
There are changes in sis3316-code in nurdlib between the working c36ae0a <br>
and non-working ccac28b which are beyond my understanding. Unless urgent <br>
for you, I would suggest that we wait with that until Hans is back from <br>
vacation. ;)<br>
<br>
Meanwhile, if you like, you could of course start out at the working <br>
version combination, and update drasi and trloii to current version. One <br>
at a time. The interfaces between those and also those and nurdlib are <br>
small and should not have seen changes lately, so I would expect any such <br>
combinations to compile.<br>
<br>
Cheers,<br>
Håkan<br>
<br>
<br>
On Thu, 18 Jul 2024, Weber, Guenter Dr. wrote:<br>
<br>
> <br>
> Here are the hashes of the (old) software combination that runs without issues:<br>
> <br>
> <br>
> (base) mbsdaq@atpnbg012:~/mbsrun/rio4/2024_mcalstruck/nurdlib$ git rev-parse --short HEAD<br>
> c36ae0a<br>
> <br>
> (base) mbsdaq@atpnbg012:~/mbsrun/rio4/2024_mcalstruck/trloii$ git rev-parse --short HEAD<br>
> 0020559<br>
> <br>
> (base) mbsdaq@atpnbg012:~/mbsrun/rio4/2024_mcalstruck/drasi$ git rev-parse --short HEAD<br>
> 1c08b62<br>
> <br>
> (base) mbsdaq@atpnbg012:~/mbsrun/rio4/2024_mcalstruck/r3bfuser$ git rev-parse --short HEAD<br>
> d09918b<br>
> <br>
> I hope this is what you were asking for.<br>
> <br>
> <br>
> <br>
> Best greetings<br>
> Günter<br>
> <br>
> _______________________________________________________________________________________________________________________________________<br>
> Von: subexp-daq <subexp-daq-bounces@lists.chalmers.se> im Auftrag von Håkan T Johansson <f96hajo@chalmers.se><br>
> Gesendet: Donnerstag, 18. Juli 2024 16:50:42<br>
> An: Discuss use of Nurdlib, TRLO II, drasi and UCESB.<br>
> Betreff: Re: [subexp-daq] New warning meassage in R3BFUSER <br>
> <br>
> On Thu, 18 Jul 2024, Weber, Guenter Dr. wrote:<br>
> <br>
> ><br>
> > Dear Håkan,<br>
> ><br>
> ><br>
> > we have to DAQ systems with identical hardware. Whenever, there is a new version (TRLOII,<br>
> > NURDLIB, DRASI, etc.) from your side that we want to test, I am setting up one of them<br>
> > with the most recent software version.<br>
> ><br>
> ><br>
> > So, right now we have a running system that is roughly on the software status of two<br>
> > months ago (if I remember corretly, this was after the repository change as we first had<br>
> > to ask you to recreate the SIS3316 development branch) with some modifications from our<br>
> > side to the SIS3316 code (committed about six weeks ago).<br>
> <br>
> Just so I follow correctly: both systems are tested able to run with some<br>
> (specific) software version?<br>
> <br>
> Could you send the git hashes of nurdlib/r3bfuser/trloii/drasi of that<br>
> working combination?<br>
> <br>
> > And we have the 'new' system<br>
> > that has identical hardware and the software status from last week. <br>
> <br>
> <br>
> Cheers,<br>
> Håkan<br>
> <br>
> <br>
> <br>
> ><br>
> ><br>
> ><br>
> > Best greetings<br>
> ><br>
> > Günter<br>
> ><br>
> ><br>
> ><br>
> ><br>
> > _________________________________________________________________________________________<br>
> > Von: subexp-daq <subexp-daq-bounces@lists.chalmers.se> im Auftrag von Håkan T Johansson<br>
> > <f96hajo@chalmers.se><br>
> > Gesendet: Donnerstag, 18. Juli 2024 15:47:46<br>
> > An: Discuss use of Nurdlib, TRLO II, drasi and UCESB.<br>
> > Betreff: Re: [subexp-daq] New warning meassage in R3BFUSER <br>
> ><br>
> > Dear Günter,<br>
> ><br>
> > did I follow this thread correctly that these tests are with 'new'<br>
> > hardware, i.e. a setup which has no working reference software?<br>
> ><br>
> > You have / are using the SIS3316 modules in another physical setup with<br>
> > some fairly recent branch in the previous nurdlib repository?<br>
> ><br>
> > <a href="https://gitlab.com/chalmers-subexp/nurdlib.prelgpl21/-/network/master">
https://gitlab.com/chalmers-subexp/nurdlib.prelgpl21/-/network/master</a><br>
> ><br>
> > --<br>
> ><br>
> > If so, there are three changes at the same time:<br>
> ><br>
> > - Nurdlib updates (also split by the repository change).<br>
> ><br>
> > - The SIS3316 code branch adapted from the previous to the new repository.<br>
> ><br>
> > - A new hardware setup.<br>
> ><br>
> > Cheers,<br>
> > Håkan<br>
> ><br>
> ><br>
> ><br>
> > On Thu, 18 Jul 2024, Weber, Guenter Dr. wrote:<br>
> ><br>
> > ><br>
> > > Dear Håkan,<br>
> > ><br>
> > ><br>
> > > with diff = 1 the DAQ is running as long as only the VULOM4B is readout. Adding a<br>
> > SIS3316<br>
> > > module results in an readout error of that module, followed by a reset of the DAQ.<br>
> > ><br>
> > ><br>
> > > Thus, I am currently unable to test SIS3316_check_hit_reworked_rebased. Sorry :-(<br>
> > ><br>
> > ><br>
> > ><br>
> > ><br>
> > > Best greetings<br>
> > ><br>
> > > Günter<br>
> ><br>
> ><br>
> <br>
></div>
</span></font></div>
</body>
</html>