From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jun Kamada Subject: Re: [PATCH 0/5] pvSCSI (SCSI pass through) driver Date: Tue, 23 Oct 2007 16:09:12 +0900 Message-ID: <20071023154955.FD8A.KAMA@jp.fujitsu.com> References: <20071019132525.E764.KAMA@jp.fujitsu.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="------_471D999300000000FE59_MULTIPART_MIXED_" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Keir Fraser Cc: kama@jp.fujitsu.com, xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org --------_471D999300000000FE59_MULTIPART_MIXED_ Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Dear Keir-san, On Mon, 22 Oct 2007 09:58:17 +0100 Keir Fraser wrote: > What's the CONFIG_XEN_FC stuff all about? It doesn't appear to be > configurable (so it's always off), it's ifdef'ed all over the place, and > there are dead files (e.g., fcfront.c) that presumably would depend on that > option. > > Either it needs to be plumbed in properly, or the XEN_FC stuff needs to be > stripped out entirely from this patchset. The CONFIG_XEN_FC stuff is used for activating FC(Fibre Channel) transport layer functionality. I will attach additional patch for the stuff. Please note: The FC transport functionality for PV driver on HVM domain is not activated even if this stuff is set. It is only available for 2.6.18 kernel for PV domain. This is why definition of "struct fc_function_template" used for the FC transport functionality are differrent for each kernel version. Therefore "#ifdef" switch is needed for each kernel version. We didn't implement the switch on current implementation. Best regards, ----- Jun Kamada --------_471D999300000000FE59_MULTIPART_MIXED_ Content-Type: application/octet-stream; name="modify_Make_for_CONFIG_XEN_FC.patch" Content-Disposition: attachment; filename="modify_Make_for_CONFIG_XEN_FC.patch" Content-Transfer-Encoding: base64 IyBIRyBjaGFuZ2VzZXQgcGF0Y2gKIyBVc2VyIHQuaG9yaWtvc2hpQGpwLmZ1aml0c3UuY29tCiMg RGF0ZSAxMTkzMTE0NzI1IC0zMjQwMAojIE5vZGUgSUQgOWNhNDJlNDk2ZWUzNTU1ZTA1NzZmY2Fi ZDc1OTQyZGNhOGIzODI1ZgojIFBhcmVudCAgYjhhNGVmN2MwMWFhMzQ1ZDgyMDE2OTI2NmE2YWE4 OGM0NDhmN2FhNQpbTElOVVhdIG1vZGlmeSBNYWtlZmlsZSBmb3IgQ09ORklHX1hFTl9GQwoKU2ln bmVkLW9mZi1ieTogVG9tb25hcmkgSG9yaWtvc2hpIDx0Lmhvcmlrb3NoaUBqcC5mdWppdHN1LmNv bT4KU2lnbmVkLW9mZi1ieTogVHN1bmVoaXNhIERvaSA8RG9pLlRzdW5laGlzYUBqcC5mdWppdHN1 LmNvbT4KU2lnbmVkLW9mZi1ieTogSnVuIEthbWFkYSA8a2FtYUBqcC5mdWppdHN1LmNvbT4KU2ln bmVkLW9mZi1ieTogQWtpcmEgSGF5YWthd2EgPGhheWFrYXdhLmFraXJhQGpwLmZ1aml0c3UuY29t PgoKZGlmZiAtciBiOGE0ZWY3YzAxYWEgLXIgOWNhNDJlNDk2ZWUzIGRyaXZlcnMveGVuL0tjb25m aWcKLS0tIGEvZHJpdmVycy94ZW4vS2NvbmZpZwlGcmkgT2N0IDE5IDA4OjU5OjAzIDIwMDcgKzA5 MDAKKysrIGIvZHJpdmVycy94ZW4vS2NvbmZpZwlUdWUgT2N0IDIzIDEzOjQ1OjI1IDIwMDcgKzA5 MDAKQEAgLTYxLDYgKzYxLDIxIEBAIGNvbmZpZyBYRU5fU0NTSV9CQUNLRU5ECiAJaGVscAogCSAg VGhlIFNDU0kgYmFja2VuZCBkcml2ZXIgYWxsb3dzIHRoZSBrZXJuZWwgdG8gZXhwb3J0IGl0cyBT Q1NJIEhCQXMKIAkgIHRvIG90aGVyIGd1ZXN0cyB2aWEgYSBoaWdoLXBlcmZvcm1hbmNlIHNoYXJl ZC1tZW1vcnkgaW50ZXJmYWNlLgorCitjaG9pY2UKKwlwcm9tcHQgIlNDU0kgQmFja2VuZCBNb2Rl IgorCWRlcGVuZHMgb24gWEVOX1NDU0lfQkFDS0VORAorCWRlZmF1bHQgWEVOX1NDU0lfQkFDS0VO RF9TQ1NJCisKK2NvbmZpZyBYRU5fU0NTSV9CQUNLRU5EX1NDU0kKKwlib29sICJOb3QgRkMiCisK K2NvbmZpZyBYRU5fRkMKKwlib29sICJGQyBkcml2ZXIgc3VwcG9ydCIKKwktLS1oZWxwLS0tCisg ICAgICAgICAgIFRoZSBmdW5jdGlvbiBvZiBzY3NpX3RyYW5zcG9ydF9mYyBpcyBhZGQuCisKK2Vu ZGNob2ljZQogCiBjb25maWcgWEVOX0JMS0RFVl9UQVAKIAl0cmlzdGF0ZSAiQmxvY2stZGV2aWNl IHRhcCBiYWNrZW5kIGRyaXZlciIKZGlmZiAtciBiOGE0ZWY3YzAxYWEgLXIgOWNhNDJlNDk2ZWUz IGRyaXZlcnMveGVuL3Njc2liYWNrL01ha2VmaWxlCi0tLSBhL2RyaXZlcnMveGVuL3Njc2liYWNr L01ha2VmaWxlCUZyaSBPY3QgMTkgMDg6NTk6MDMgMjAwNyArMDkwMAorKysgYi9kcml2ZXJzL3hl bi9zY3NpYmFjay9NYWtlZmlsZQlUdWUgT2N0IDIzIDEzOjQ1OjI1IDIwMDcgKzA5MDAKQEAgLTEs MiArMSw4IEBAIG9iai0kKENPTkZJR19YRU5fU0NTSV9CQUNLRU5EKQkrPSB4ZW4tc2MKLW9iai0k KENPTkZJR19YRU5fU0NTSV9CQUNLRU5EKQkrPSB4ZW4tc2NzaWJrLm8KLXhlbi1zY3NpYmsteQkJ CSs9IGludGVyZmFjZS5vIHNjc2liYWNrLm8geGVuYnVzLm8gY29tYmFjay5vIHRyYWNlYmFjay5v CitpZmVxICgkKENPTkZJR19YRU5fRkMpLHkpCisgIEVYVFJBX0NGTEFHUyArPSAtRENPTkZJR19Y RU5fRkMKKyAgb2JqLSQoQ09ORklHX1hFTl9TQ1NJX0JBQ0tFTkQpCSs9IHhlbi1mY3Njc2liay5v CisgIHhlbi1mY3Njc2liay15CQkJKz0gaW50ZXJmYWNlLm8gc2NzaWJhY2subyB4ZW5idXMubyBm Y2JhY2subyBjb21iYWNrLm8gdHJhY2ViYWNrLm8KK2Vsc2UKKyAgb2JqLSQoQ09ORklHX1hFTl9T Q1NJX0JBQ0tFTkQpCSs9IHhlbi1zY3NpYmsubworICB4ZW4tc2NzaWJrLXkJCQkJKz0gaW50ZXJm YWNlLm8gc2NzaWJhY2subyB4ZW5idXMubyBjb21iYWNrLm8gdHJhY2ViYWNrLm8KK2VuZGlmCmRp ZmYgLXIgYjhhNGVmN2MwMWFhIC1yIDljYTQyZTQ5NmVlMyBkcml2ZXJzL3hlbi9zY3NpZnJvbnQv TWFrZWZpbGUKLS0tIGEvZHJpdmVycy94ZW4vc2NzaWZyb250L01ha2VmaWxlCUZyaSBPY3QgMTkg MDg6NTk6MDMgMjAwNyArMDkwMAorKysgYi9kcml2ZXJzL3hlbi9zY3NpZnJvbnQvTWFrZWZpbGUJ VHVlIE9jdCAyMyAxMzo0NToyNSAyMDA3ICswOTAwCkBAIC0xLDIgKzEsOCBAQAoraWZlcSAoJChD T05GSUdfWEVOX0ZDKSx5KQorICBFWFRSQV9DRkxBR1MgKz0gLURDT05GSUdfWEVOX0ZDCisgIG9i ai0kKENPTkZJR19YRU5fU0NTSV9GUk9OVEVORCkJOj0geGVuZmNzY3NpLm8KKyAgeGVuZmNzY3Np LW9ianMgOj0gc2NzaWZyb250Lm8gZmNmcm9udC5vIGNvbWZyb250Lm8gdHJhY2Vmcm9udC5vCitl bHNlCiAgIG9iai0kKENPTkZJR19YRU5fU0NTSV9GUk9OVEVORCkJOj0geGVuc2NzaS5vCiAgIHhl bnNjc2ktb2JqcyA6PSBzY3NpZnJvbnQubyBjb21mcm9udC5vIHRyYWNlZnJvbnQubworZW5kaWYK --------_471D999300000000FE59_MULTIPART_MIXED_ Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel --------_471D999300000000FE59_MULTIPART_MIXED_--