[lkml]   [2010]   [Oct]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRE: [PATCH v2 03/10] RapidIO: Use stored ingress port number instead of register read
Hi Micha,

Sorry for delayed reply.

Micha Nelissen <> wrote:
> Bounine, Alexandre wrote:
> > struct rio_dev {
> > struct list_head global_list;
> > struct list_head net_list;
> > .....
> > ..... rest of rio_dev
> > .....
> > struct rio_switch switch[0];
> > }
> It makes sense to let rio_dev structures point to the switch they are
> attached to. That can be useful in various situations, but is not
> possible with this setup.
> If a rio_dev is a switch then rdev->rswitch->rdev == rdev holds.

But the switch is a RIO device itself and all other parts of rio_dev
are applicable to it as well. If there is situation when a device
needs to hold a pointer to the attached switch that should be a pointer
to the switch rio_dev and not to its switch-specific extension.

> > This will remove extra memory allocation, remove overlapping
> > members and clean code sections like one shown below:
> >
> > u8 hopcount = 0xff;
> > u16 destid = rdev->destid;
> >
> > if (rdev->rswitch) {
> > destid = rdev->rswitch->destid;
> > hopcount = rdev->rswitch->hopcount;
> > }
> Note that it is possible for rdev->destid to differ from
> rdev->rswitch->destid even if rswitch->rdev == rdev (for non-hosts
> agents). rswitch->destid is the destid by which we discovered the
> (and can reach it) but rdev->destid is the actual id given to the

My goal is to have one destid storage for device - endpoint or switch.
And destid should be used only for one purpose: to reach corresponding
In your statement above you suggest using rdev->destid instead of
RIO switches do not have any specific RIO ID that can be assigned to the
In this case the rswitch->switchid should work well for logical
of the switch.

I think if (for switch) we load rdev->destid with some function
from its routing role this may bring unnecessary confusion.

I also will move rswitch->hopcount to rdev->hopcount. For endpoint it
will be set
to 0xff during rio_dev initialization.


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2010-10-03 22:25    [W:0.129 / U:2.052 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site