Hi Jan,
Just one small question.
When the rtskb are created the memory indicated by the pointers present in the
struct rtskb are allocated too?
For example the rtdev pointer points to one area of allocated memory for this
rtskb, or this pointer points to a unique area that refers to the rtdev struct
of the real device?
Thanks for an answer.
Em Quinta, 31 de Agosto de 2006 17:12, o Jan Kiszka escreveu:
> Hi Jorge,
>
> Jorge Almeida wrote:
> > Hello Jan.
> >
> > Can you please give me a description of wath is done in the rtskb_acquire
> > function in the rtskb.c file line 463.
>
> That function passes the ownership of a rtskb from rtskb->pool to
> comp_pool (compensation pool). For this purpose, an empty rtskb is
> dequeued from comp_pool and enqueued in rtskb->pool, then rtskb->pool is
> set to comp_pool.
>
> >
> > The rtnet is crashing with an SIG=11 in this function when i pass the
> > cloned rtskb to the ETH_P_ALL socket.
>
> Where precisely? Already considered to install kgdb for such development?
>
> >
> > My clone function is the following:
> > /** Clone a rtskb to another, allocating a rtskb for the copy */
> > int rtskb_clone(struct rtskb *rtskb_1,struct rtskb *rtskb_2,struct
> > rtskb_queue *pool)
> > {
> >
> > rtskb_1 = alloc_rtskb(rtskb_2->len,pool);
> > if(rtskb_1 == NULL)
> > return -ENOMEM;
> >
> >
> > memcpy(rtskb_1->data, rtskb_2->data, rtskb_2->len );
> > rtskb_1->len = rtskb_2->len;
> > rtskb_1->time_stamp = rtskb_2->time_stamp;
> > rtskb_1->rtdev = rtskb_2->rtdev;
> > rtskb_1->protocol = rtskb_2->protocol;
>
> What about the things that happen in rt_eth_type_trans (except for
> rtcap_mark_incoming)? Should case the crash above, but will cause
> troubles when processing the rtskb.
>
> >
> > return 0;
> > }
> >
> > I check the path between the driver and the stack manager and only find
> > that these fields are necessary.
> >
> > the task manager does the following for the clone rtskb:
> >
> > list_for_each_entry(pt_entry, &rt_packets[hash], list_entry)
> > {
> > rt_printk("List entry protocol= %d %d name = %d\n",
> >
> > ntohs(pt_entry->type),hash,strncmp(pt_entry->name,"PACKET_SOCKET",13));
> > //the "PACKET_SOCKET" string is defined in line 188 of file
> > af_packet.c
> > //this string should be replaced by a #define declaration
> > if ( likely(ntohs(pt_entry->type) == hash &&
> > (strncmp(pt_entry->name,"PACKET_SOCKET",13) ==
> > 0 )))
> > {
> > rtdm_lock_put_irqrestore(&rt_packets_lock, context);
>
> Don't drop the lock here when taking it two lines later again.
>
> > // rt_printk("List entry 1 \n");
> >
> > sock = container_of(pt_entry, struct rtsocket,
> > prot.packet.packet_type);
>
> Fatal layering violation! This is a hack, the actual cloning should
> rather take place inside the invoked handler. Same is true for the
> enqueuing. This way you could also avoid to acquire an rtskb that was
> already taken from the socket queue (redundant work).
>
> >
> > rtdm_lock_get_irqsave(&rt_packets_lock, context);
> > err = rtskb_clone(clone_skb, skb, &sock->skb_pool);
> > rtdm_lock_put_irqrestore(&rt_packets_lock, context);
> > if(err != 0)
> > {
> > break;
> > }
> >
> > rtskb_queue_head(&sock->skb_pool,clone_skb);
> > //queue the message to fifo head here
> >
> >
> > rtdm_lock_get_irqsave(&rt_packets_lock, context);
> > pt_entry->refcount++;
> > rtdm_lock_put_irqrestore(&rt_packets_lock, context);
> > // rt_printk("List entry 2 \n");
> > err = pt_entry->handler(clone_skb, pt_entry);
> > // rt_printk("List entry 3 \n");
> > rtdm_lock_get_irqsave(&rt_packets_lock, context);
> > pt_entry->refcount--;
> > rtdm_lock_put_irqrestore(&rt_packets_lock, context);
> >
> > }
> > }
> >
> >
> > I'm not sure if the cloned rtskb shoul be inserted in the queue head or not.
> >
>
> Move that stuff to af_packet.c.
>
> Jan
>
>
--
Jorge Almeida
[EMAIL PROTECTED]
DISCLAIMER: This message may contain confidential information or privileged
material and is intended only for the individual(s) named. If you are not a
named addressee and mistakenly received this message you should not copy or
otherwise disseminate it: please delete this e-mail from your system and notify
the sender immediately. E-mail transmissions are not guaranteed to be secure or
error-free as information could be intercepted, corrupted, lost, destroyed,
arrive late or incomplete or contain viruses. Therefore, the sender does not
accept liability for any errors or omissions in the contents of this message
that arise as a result of e-mail transmissions. Please request a hard copy
version if verification is required. Critical Software, SA.
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
RTnet-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/rtnet-developers
