2009-09-25 22:18:49

by Karen Xie

[permalink] [raw]
Subject: [PATCH v2 0/2] cxgb3/cxgb3i: added support of private MAC address and provisioning packet handler for iSCSI

From: Karen Xie <[email protected]>

The two patches added support of private MAC address, which is derived from
the network (i.e., public) MAC address, and provisioning packet handler for
iSCSI traffic only.

Best regards,
Karen


2009-09-25 22:24:14

by David Miller

[permalink] [raw]
Subject: Re: [PATCH v2 0/2] cxgb3/cxgb3i: added support of private MAC address and provisioning packet handler for iSCSI

From: [email protected]
Date: Fri, 25 Sep 2009 15:25:36 -0700

> From: Karen Xie <[email protected]>
>
> The two patches added support of private MAC address, which is derived from
> the network (i.e., public) MAC address, and provisioning packet handler for
> iSCSI traffic only.

I think I'm going to ask you to resubmit the drivers/net patch
after the necessary drivers/scsi changes get merged by the scsi
folks.

Thank you.

2009-09-25 22:28:36

by Karen Xie

[permalink] [raw]
Subject: RE: [PATCH v2 0/2] cxgb3/cxgb3i: added support of private MAC address and provisioning packet handler for iSCSI

Hmm, I am wondering how could this merge activity to be coordinated? If
only the driver/scsi change is merged, then it won't compile either,
since it requires the driver/net change.

Is it possible to merge both patches to the scsi or net tree?

Thanks,
Karen

-----Original Message-----
From: David Miller [mailto:[email protected]]
Sent: Friday, September 25, 2009 3:25 PM
To: Karen Xie
Cc: [email protected]; [email protected]; Steve
Wise; Divy Le Ray; Rakesh Ranjan; [email protected];
[email protected]; [email protected];
[email protected]
Subject: Re: [PATCH v2 0/2] cxgb3/cxgb3i: added support of private MAC
address and provisioning packet handler for iSCSI

From: [email protected]
Date: Fri, 25 Sep 2009 15:25:36 -0700

> From: Karen Xie <[email protected]>
>
> The two patches added support of private MAC address, which is derived
from
> the network (i.e., public) MAC address, and provisioning packet
handler for
> iSCSI traffic only.

I think I'm going to ask you to resubmit the drivers/net patch
after the necessary drivers/scsi changes get merged by the scsi
folks.

Thank you.

2009-09-26 07:16:29

by David Miller

[permalink] [raw]
Subject: Re: [PATCH v2 0/2] cxgb3/cxgb3i: added support of private MAC address and provisioning packet handler for iSCSI

From: "Karen Xie" <[email protected]>
Date: Fri, 25 Sep 2009 15:34:22 -0700

> Hmm, I am wondering how could this merge activity to be coordinated? If
> only the driver/scsi change is merged, then it won't compile either,
> since it requires the driver/net change.

That's rediculious, frankly.

Since they are two seperate changes you are knowingly creating
a bisection point that will not work. That's wrong.

You need to split up the changes so that each and every one of them
are independant and the tree can be checked out at either of them and
everything can be expected to work.

2009-09-26 18:49:50

by Karen Xie

[permalink] [raw]
Subject: RE: [PATCH v2 0/2] cxgb3/cxgb3i: added support of private MAC address and provisioning packet handler for iSCSI

Thanks, understood.

We are submitting a new version that would be isolated to changes to the
net driver only and would not affect scsi driver.

Thanks a lot.
Karen

-----Original Message-----
From: David Miller [mailto:[email protected]]
Sent: Saturday, September 26, 2009 12:17 AM
To: Karen Xie
Cc: [email protected]; [email protected]; Steve
Wise; Divy Le Ray; Rakesh Ranjan; [email protected];
[email protected]; [email protected];
[email protected]
Subject: Re: [PATCH v2 0/2] cxgb3/cxgb3i: added support of private MAC
address and provisioning packet handler for iSCSI

From: "Karen Xie" <[email protected]>
Date: Fri, 25 Sep 2009 15:34:22 -0700

> Hmm, I am wondering how could this merge activity to be coordinated?
If
> only the driver/scsi change is merged, then it won't compile either,
> since it requires the driver/net change.

That's rediculious, frankly.

Since they are two seperate changes you are knowingly creating
a bisection point that will not work. That's wrong.

You need to split up the changes so that each and every one of them
are independant and the tree can be checked out at either of them and
everything can be expected to work.