Experimental Physics and
Industrial Control System

"Jeff Hill" <[email protected]> · Thu, 13 Jan 2011 10:30:07 -0700

Not a problem, I think that I did also review the changes, but it's not
always possible (efficient) to detect issues when staring at the code for a
limited length of time.

Hopefully soon we can better automate client / server test permutations so
that the regression tests provide better coverage.

Jeff
______________________________________________________
Jeffrey O. Hill           Email        [email protected]
LANL MS H820              Voice        505 665 1831
Los Alamos NM 87545 USA   FAX          505 665 5107

Message content: TSPA

With sufficient thrust, pigs fly just fine. However, this is
not necessarily a good idea. It is hard to be sure where they
are going to land, and it could be dangerous sitting under them
as they fly overhead. -- RFC 1925

> -----Original Message-----
> From: Michael Abbott [mailto:[email protected]]
> Sent: Thursday, January 13, 2011 9:52 AM
> To: Mark Rivers
> Cc: Jeff Hill; [email protected]
> Subject: RE: large arrays native element count wrong with Channel Access
> inEPICS 3.14.12
> 
> That appears to be my fault, I'm very sorry about that.  Bug 701673 is a
> revert of this commit:
> 
> http://bazaar.launchpad.net/~michael-abbott/epics-base/dynamic-
> array/revision/12076
> 
> Unfortunately I'm rusty enough that I can't see the error right now, and
> I'm tied up away from the office for a few days now.  Thanks very much
> for
> finding and fixing this.
> 
> I don't remember at all why that commit was created in the first place,
> as
> it seems rather pointless really.
> 
> I guess this shows the hazards of working on a code base very part time
> :(
> 
> 
> On Wed, 12 Jan 2011, Mark Rivers wrote:
> 
> > Hi Jeff,
> >
> > Thanks for the quick response.  I installed the patch and it fixes the
> problem.
> >
> > Mark
> >
> >
> > ________________________________
> >
> > From: [email protected] on behalf of Jeff Hill
> > Sent: Wed 1/12/2011 5:21 PM
> > To: [email protected]
> > Subject: RE: large arrays native element count wrong with Channel
> Access inEPICS 3.14.12
> >
> >
> >
> > I committed a fix to the R3.14 branch. An overzealous optimization was
> > installed. See bug 701673 at launchpad for the details.
> >
> > https://bugs.launchpad.net/epics-base/+bug/701673
> >
> > Jeff
> > ______________________________________________________
> > Jeffrey O. Hill           Email        [email protected]
> > LANL MS H820              Voice        505 665 1831
> > Los Alamos NM 87545 USA   FAX          505 665 5107
> >
> > Message content: TSPA
> >
> > With sufficient thrust, pigs fly just fine. However, this is
> > not necessarily a good idea. It is hard to be sure where they
> > are going to land, and it could be dangerous sitting under them
> > as they fly overhead. -- RFC 1925
> >
> >
> > > -----Original Message-----
> > > From: Jeff Hill [mailto:[email protected]]
> > > Sent: Wednesday, January 12, 2011 9:27 AM
> > > To: '[email protected]'
> > > Subject: RE: Problem using large arrays with Channel Access in EPICS
> > > 3.14.12
> > >
> > > Yes, there were two new features merged into CA for the R3.14.12
> patch
> > > release. I will have a look.
> > >
> > > PS: IMHO, we can consider that risk is reduced if only patches are
> merged
> > > into patch releases. Perhaps new features are best placed in point
> > > releases.
> > >
> > > Jeff
> > > ______________________________________________________
> > > Jeffrey O. Hill           Email        [email protected]
> > > LANL MS H820              Voice        505 665 1831
> > > Los Alamos NM 87545 USA   FAX          505 665 5107
> > >
> > > Message content: TSPA
> > >
> > > With sufficient thrust, pigs fly just fine. However, this is
> > > not necessarily a good idea. It is hard to be sure where they
> > > are going to land, and it could be dangerous sitting under them
> > > as they fly overhead. -- RFC 1925
> > >
> > >
> > > > -----Original Message-----
> > > > From: [email protected] [mailto:tech-talk-
> > > > [email protected]] On Behalf Of Andrew Johnson
> > > > Sent: Tuesday, January 11, 2011 2:43 PM
> > > > To: [email protected]
> > > > Subject: Re: Problem using large arrays with Channel Access in
> EPICS
> > > > 3.14.12
> > > >
> > > > On Tuesday 11 January 2011 15:21:32 Till Straumann wrote:
> > > > > Could be a limitation of the CA protocol
> > > >
> > > > No, this is as a result of changes in src/rsrv between 3.14.11 and
> > > > 3.14.12
> > > > because large arrays work fine in 3.14.11.
> > > >
> > > > We introduced two major new features in the latest release, CA over
> TCP
> > > > and
> > > > dynamic array sizing, and one of them (most probably the latter)
> has
> > > > broken
> > > > the correct reporting of the element count.  The CA protocol solved
> the
> > > > problem of only a having a 16 bit m_count field a long time ago,
> but it
> > > > looks
> > > > like that hack^H^H^H^Hsolution didn't get propagated into the
> latest
> > > > code.
> > > >
> > > > - Andrew
> > > > --
> > > > If a man is offered a fact which goes against his instincts, he
> will
> > > > scrutinize it closely, and unless the evidence is overwhelming, he
> will
> > > > refuse to believe it.  If, on the other hand, he is offered
> something
> > > > which affords a reason for acting in accordance to his instincts,
> he
> > > > will accept it even on the slightest evidence.  -- Bertrand Russell
> >
> >
> >
> >
> >
> >

Experimental Physics and Industrial Control System

Experimental Physics and
Industrial Control System