Hi, thanks, Mike. It looks like the original patch somehow did not make into the linux-scsi list at all for some reason: tried to search for it but could not find it.
We are re-submitting it just to make sure. Best regards, Karen -----Original Message----- From: Mike Christie [mailto:micha...@cs.wisc.edu] Sent: Mon 12/7/2009 8:34 AM To: open-iscsi@googlegroups.com Cc: james.bottom...@hansenpartnership.com; da...@davemloft.net; Steve Wise; Karen Xie; linux-s...@vger.kernel.org; linux-ker...@vger.kernel.org; net...@vger.kernel.org; Rakesh Ranjan Subject: Re: [PATCH 1/1] cxgb3i: Fix a login over vlan issue Rakesh Ranjan wrote: > Fix a login over vlan issue, when parent interface is vlan and we are using > cxgb3i sepecific > private ip address in '/etc/iscsi/ifaces/' iface file. > > Acked-by: Karen Xie <k...@chelsio.com> > Signed-off-by: Rakesh Ranjan <rak...@chelsio.com> > --- > drivers/scsi/cxgb3i/cxgb3i_offload.c | 34 > +++++++++++++++++++++++++++++++++- > 1 files changed, 33 insertions(+), 1 deletions(-) > > diff --git a/drivers/scsi/cxgb3i/cxgb3i_offload.c > b/drivers/scsi/cxgb3i/cxgb3i_offload.c > index c1d5be4..66d52e4 100644 > --- a/drivers/scsi/cxgb3i/cxgb3i_offload.c > +++ b/drivers/scsi/cxgb3i/cxgb3i_offload.c > @@ -1440,6 +1440,10 @@ void cxgb3i_c3cn_release(struct s3_conn *c3cn) > static int is_cxgb3_dev(struct net_device *dev) > { > struct cxgb3i_sdev_data *cdata; > + struct net_device *ndev = dev; > + > + if (dev->priv_flags & IFF_802_1Q_VLAN) > + ndev = vlan_dev_real_dev(dev); > > write_lock(&cdata_rwlock); > list_for_each_entry(cdata, &cdata_list, list) { > @@ -1447,7 +1451,7 @@ static int is_cxgb3_dev(struct net_device *dev) > int i; > > for (i = 0; i < ports->nports; i++) > - if (dev == ports->lldevs[i]) { > + if (ndev == ports->lldevs[i]) { > write_unlock(&cdata_rwlock); > return 1; > } > @@ -1566,6 +1570,26 @@ out_err: > return -1; > } > > +/** > + * cxgb3i_find_dev - find the interface associated with the given address > + * @ipaddr: ip address > + */ > +static struct net_device * > +cxgb3i_find_dev(__be32 ipaddr) > +{ > + struct flowi fl; > + int err; > + struct rtable *rt; > + > + memset(&fl, 0, sizeof(fl)); > + fl.nl_u.ip4_u.daddr = ipaddr; > + > + err = ip_route_output_key(&init_net, &rt, &fl); > + if (!err) > + return (&rt->u.dst)->dev; > + > + return NULL; > +} > > /** > * cxgb3i_c3cn_connect - initiates an iscsi tcp connection to a given address > @@ -1581,6 +1605,7 @@ int cxgb3i_c3cn_connect(struct net_device *dev, struct > s3_conn *c3cn, > struct cxgb3i_sdev_data *cdata; > struct t3cdev *cdev; > __be32 sipv4; > + struct net_device *dstdev; > int err; > > c3cn_conn_debug("c3cn 0x%p, dev 0x%p.\n", c3cn, dev); > @@ -1591,6 +1616,13 @@ int cxgb3i_c3cn_connect(struct net_device *dev, struct > s3_conn *c3cn, > c3cn->daddr.sin_port = usin->sin_port; > c3cn->daddr.sin_addr.s_addr = usin->sin_addr.s_addr; > > + dstdev = cxgb3i_find_dev(usin->sin_addr.s_addr); > + if (!dstdev || !is_cxgb3_dev(dstdev)) > + return -ENETUNREACH; > + > + if (dstdev->priv_flags & IFF_802_1Q_VLAN) > + dev = dstdev; > + > rt = find_route(dev, c3cn->saddr.sin_addr.s_addr, > c3cn->daddr.sin_addr.s_addr, > c3cn->saddr.sin_port, Looks sane. I am not a expert on the networks apis being used though. -- You received this message because you are subscribed to the Google Groups "open-iscsi" group. To post to this group, send email to open-is...@googlegroups.com. To unsubscribe from this group, send email to open-iscsi+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/open-iscsi?hl=en.