DSP bug?

M

Thread Starter

Mario de Sousa

Juan Carlos Orozco wrote:
>
> Mario de Sousa wrote:
> >
> > Juan Carlos Orozco wrote:
> > > I am having difficulties running two dsp modules (with different module
> > > names) concurrently.
(...)
> >
> > There is either an error in the configuration file (have you looked in
> > the log file to see if any warnings are being spit out?), or a bug in
> > the
> > DSP module. It could be something else that seems hardly related. If you
> > have no warnings in the log file, could you email me the configuration
> > files
> > so I can try and replicate the error and track it down?

Any news on this?


Mario.


----------------------------------------------------------------------------
Mario J. R. de Sousa
[email protected]
----------------------------------------------------------------------------

The box said it requires Windows 95 or better, so I installed Linux

_______________________________________________
LinuxPLC mailing list
[email protected]
http://linuxplc.org/mailman/listinfo/linuxplc
 
J

Juan Carlos Orozco

Unfortunately I had a sudden increase in my work load. I was hoping to try and debug them myself but haven't find the time to do it. In the meantime I will send you an example that I can not find what is wrong in the configuration but doesn't work. I will first give it a quick review to avoid sending dumb mistakes. I have stripped it as much as I could to have the simplest case that will still reproduce the problem. I know your to-do list is very long but you may be in a better position to catch the bug than me (if there is one).

Regards,

Juan Carlos Orozco

ACElab Industrial Automation
[email protected]
www.ace-lab.com


_______________________________________________
LinuxPLC mailing list
[email protected]
http://linuxplc.org/mailman/listinfo/linuxplc
 
J

Juan Carlos Orozco

Mario,

Great News!!! I finally found the bug in the dsp module. I am glad this one didn't makeit to the first release.

The bug is in the add.c program. The problem is that the number of points is not initialized before entering the for loop. A solution is to insert the next text in line 134 of add.c:

parm->add.num_pts = 0;

I also checked your code on the typeconv.c program to see if the error was not also there but I found that it is OK because it is implemented differently, you assign the value after the for loop instead of incrementing
it on the for loop therefore avoiding the initialization requirement. Which by the way could be other solution to add.c. The line in your typeconv.c program after the for loop goes like this:

parm->typeconv.num_pts = pt_count;

Regards,

Juan Carlos Orozco

ACElab Industrial Automation
[email protected]
www.ace-lab.com

_______________________________________________
LinuxPLC mailing list
[email protected]
http://linuxplc.org/mailman/listinfo/linuxplc
 
M

Mario de Sousa

Arrrghh!!! Really stupid of me!!!

Thanks Juan!

The fix is in the cvs now!


Mario.

--
----------------------------------------------------------------------------
Mario J. R. de Sousa
[email protected]
----------------------------------------------------------------------------

The box said it requires Windows 95 or better, so I installed Linux

_______________________________________________
LinuxPLC mailing list
[email protected]
http://linuxplc.org/mailman/listinfo/linuxplc
 
Top