EPICS Controls Argonne National Laboratory

Experimental Physics and
Industrial Control System

2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  2019  2020  <20212022  2023  2024  Index 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  2019  2020  <20212022  2023  2024 
<== Date ==> <== Thread ==>

Subject: Re: [Merge] ~dirk.zimoch/epics-base:FilterForInfoFields into epics-base:7.0
From: Dirk Zimoch via Core-talk <core-talk at aps.anl.gov>
To: mp+399414 at code.launchpad.net
Date: Wed, 10 Mar 2021 13:43:49 -0000
I am at commit 663e6442e47ea7329217f60b6ca3e5c6d636620d which is based on 3ba778c08bfbf6364bdbefa41f16022b4c0f41d5, before your refactory was merged in.
I have not pushed the re-base on top of f571c5950bfe476ef79d4c143f50e2f9e0d5edc7.

What fails: I get garbage instead of the correct data. Don't know yet where it came from. I suspect it has to do with the new dbfl_has_copy() logic.

I could fix my code by adding a dummy pfl->u.r.dtor in order to claim "ownership" of the data for the logic in dbAccess.c. The desctructor does nothing, as the data does not need to be destroyed, but otherwise dbAccess.c takes a few wrong branches, it seems.

I have not analyzed the exact path of execution before and after. I have checked which  your commits (cherry-picked onto my branch) broke my code and then which modified line. I got the first problem with commit 85822f in dbAccess.c line 918. Reverting that line restored my functionality. But later commits broke it again. So I checked what changed there.

I found out that adding a dtor seems to fix everything. I will commit than now and rebase this branch.
-- 
https://code.launchpad.net/~dirk.zimoch/epics-base/+git/epics-base/+merge/399414
Your team EPICS Core Developers is requested to review the proposed merge of ~dirk.zimoch/epics-base:FilterForInfoFields into epics-base:7.0.

References:
[Merge] ~dirk.zimoch/epics-base:FilterForInfoFields into epics-base:7.0 Dirk Zimoch via Core-talk

Navigate by Date:
Prev: Re: [Merge] ~dirk.zimoch/epics-base:FilterForInfoFields into epics-base:7.0 Ben Franksen via Core-talk
Next: Build failed: epics-base base-alarm-msg-653 AppVeyor via Core-talk
Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  2019  2020  <20212022  2023  2024 
Navigate by Thread:
Prev: Re: [Merge] ~dirk.zimoch/epics-base:FilterForInfoFields into epics-base:7.0 Ben Franksen via Core-talk
Next: Re: [Merge] ~dirk.zimoch/epics-base:FilterForInfoFields into epics-base:7.0 Dirk Zimoch via Core-talk
Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  2019  2020  <20212022  2023  2024 
ANJ, 10 Mar 2021 Valid HTML 4.01! · Home · News · About · Base · Modules · Extensions · Distributions · Download ·
· Search · EPICS V4 · IRMIS · Talk · Bugs · Documents · Links · Licensing ·