looping

Hi I am writing a kernel module.

I need a continuous loop that will look for tcp traffic and read from skb to store each source ip address from ip_hdr->saddr into an array

I can get it to read packets one at a time but I need it to loop, but when I try to implement a continuous loop the system crashes/hangs.

Can anyone offer any advice/possible fixes?

Cheers
codey-06Asked:
Who is Participating?

[Product update] Infrastructure Analysis Tool is now available with Business Accounts.Learn More

x
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

Anthony2000Commented:
Here is an example of using a thread in the kernel module.

http://www.scs.ch/~frey/linux/kernelthreads.html

I think your problem might be that your loop is running and using all available kernel/process time. Thus, giving the result of maybe locking up your machine. Is that what is happening?
0
codey-06Author Commented:
Hi yes you are correct, the loop is using up all the kernel resources.  Is there a quick fix?
0
Anthony2000Commented:
Have you tried some kind of sleep (I cannot remember the API call). This may give the OS and other processes time to run.
0
Big Business Goals? Which KPIs Will Help You

The most successful MSPs rely on metrics – known as key performance indicators (KPIs) – for making informed decisions that help their businesses thrive, rather than just survive. This eBook provides an overview of the most important KPIs used by top MSPs.

codey-06Author Commented:
Thank you. If anyone could offer a bit more help it would be very much appreciated?
0
sarabandeCommented:
if you added an infinite or time consuming loop into a kernel module that was needed by others you were blocking all those activities dependent on those calls and the system hangs and some processes would crash or fail because of timeouts. so, the first thing you need is to start a thread where you would add the loop and the normal kernel function could proceed. of course you may not create a new thread for every call but use one thread for all requests. in the thread you would run an infinite loop and at end of loop you should add a call to usleep which would make that the thread waits a little and so other processes and threads could be scheduled by the os. the usleep takes microseconds as argument and i would use at least 1000 (1 millisecond) to not slow down the rest of the system cause that also would depend on your other actions made in the loop.

Sara
0
Anthony2000Commented:
Here is another related article.

http://www.linuxjournal.com/article/8144
0
Anthony2000Commented:
Also, if you are not familiar with this book, its also a good reference.

http://lwn.net/Kernel/LDD3/
0

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
Anthony2000Commented:
Also, take a look at the functions:
set_current_state(TASK_INTERRUPTIBLE);
schedule_timeout (signed long timeout);

See chapter 7 of the book link I provided.
0
codey-06Author Commented:
Brilliant thank you
0
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
C

From novice to tech pro — start learning today.