[iscsi] Use the "Ethernet-compatible" MAC address in the iBFT
[people/peper/gpxe.git] / src / arch / i386 / interface / pcbios / ibft.c
index ffa6596..adf8e6b 100644 (file)
@@ -25,6 +25,8 @@
  *
  */
 
+FILE_LICENCE ( BSD2 );
+
 #include <stdint.h>
 #include <stdio.h>
 #include <string.h>
@@ -35,6 +37,7 @@
 #include <gpxe/acpi.h>
 #include <gpxe/in.h>
 #include <gpxe/netdevice.h>
+#include <gpxe/ethernet.h>
 #include <gpxe/dhcp.h>
 #include <gpxe/iscsi.h>
 #include <gpxe/ibft.h>
@@ -136,6 +139,17 @@ static void ibft_set_ipaddr_option ( struct ibft_ipaddr *ipaddr,
        ibft_set_ipaddr ( ipaddr, in );
 }
 
+/**
+ * Read IP address from iBFT (for debugging)
+ *
+ * @v strings          iBFT string block descriptor
+ * @v string           String field
+ * @ret ipaddr         IP address string
+ */
+static const char * ibft_ipaddr ( struct ibft_ipaddr *ipaddr ) {
+       return inet_ntoa ( ipaddr->in );
+}
+
 /**
  * Allocate a string within iBFT
  *
@@ -214,6 +228,19 @@ static int ibft_set_string_option ( struct ibft_string_block *strings,
        return 0;
 }
 
+/**
+ * Read string from iBFT (for debugging)
+ *
+ * @v strings          iBFT string block descriptor
+ * @v string           String field
+ * @ret data           String content (or "<empty>")
+ */
+static const char * ibft_string ( struct ibft_string_block *strings,
+                                 struct ibft_string *string ) {
+       return ( string->offset ?
+                ( ( ( char * ) strings->table ) + string->offset ) : NULL );
+}
+
 /**
  * Fill in NIC portion of iBFT
  *
@@ -225,17 +252,23 @@ static int ibft_set_string_option ( struct ibft_string_block *strings,
 static int ibft_fill_nic ( struct ibft_nic *nic,
                           struct ibft_string_block *strings,
                           struct net_device *netdev ) {
+       struct ll_protocol *ll_protocol = netdev->ll_protocol;
        struct in_addr netmask_addr = { 0 };
        unsigned int netmask_count = 0;
        int rc;
 
        /* Extract values from DHCP configuration */
        ibft_set_ipaddr_option ( &nic->ip_address, &ip_setting );
+       DBG ( "iBFT NIC IP = %s\n", ibft_ipaddr ( &nic->ip_address ) );
        ibft_set_ipaddr_option ( &nic->gateway, &gateway_setting );
+       DBG ( "iBFT NIC gateway = %s\n", ibft_ipaddr ( &nic->gateway ) );
        ibft_set_ipaddr_option ( &nic->dns[0], &dns_setting );
+       DBG ( "iBFT NIC DNS = %s\n", ibft_ipaddr ( &nic->dns[0] ) );
        if ( ( rc = ibft_set_string_option ( strings, &nic->hostname,
                                             &hostname_setting ) ) != 0 )
                return rc;
+       DBG ( "iBFT NIC hostname = %s\n",
+             ibft_string ( strings, &nic->hostname ) );
 
        /* Derive subnet mask prefix from subnet mask */
        fetch_ipv4_setting ( NULL, &netmask_setting, &netmask_addr );
@@ -245,11 +278,17 @@ static int ibft_fill_nic ( struct ibft_nic *nic,
                netmask_addr.s_addr >>= 1;
        }
        nic->subnet_mask_prefix = netmask_count;
+       DBG ( "iBFT NIC subnet = /%d\n", nic->subnet_mask_prefix );
 
        /* Extract values from net-device configuration */
-       memcpy ( nic->mac_address, netdev->ll_addr,
-                sizeof ( nic->mac_address ) );
+       if ( ( rc = ll_protocol->eth_addr ( netdev->ll_addr,
+                                           nic->mac_address ) ) != 0 ) {
+               DBG ( "Could not determine iBFT MAC: %s\n", strerror ( rc ) );
+               return rc;
+       }
+       DBG ( "iBFT NIC MAC = %s\n", eth_ntoa ( nic->mac_address ) );
        nic->pci_bus_dev_func = netdev->dev->desc.location;
+       DBG ( "iBFT NIC PCI = %04x\n", nic->pci_bus_dev_func );
 
        return 0;
 }
@@ -269,6 +308,8 @@ static int ibft_fill_initiator ( struct ibft_initiator *initiator,
        if ( ( rc = ibft_set_string ( strings, &initiator->initiator_name,
                                      initiator_iqn ) ) != 0 )
                return rc;
+       DBG ( "iBFT initiator hostname = %s\n",
+             ibft_string ( strings, &initiator->initiator_name ) );
 
        return 0;
 }
@@ -286,17 +327,23 @@ static int ibft_fill_target_chap ( struct ibft_target *target,
                                   struct iscsi_session *iscsi ) {
        int rc;
 
-       if ( ! iscsi->initiator_username )
+       if ( ! ( iscsi->status & ISCSI_STATUS_AUTH_FORWARD_REQUIRED ) )
                return 0;
+
+       assert ( iscsi->initiator_username );
        assert ( iscsi->initiator_password );
 
        target->chap_type = IBFT_CHAP_ONE_WAY;
        if ( ( rc = ibft_set_string ( strings, &target->chap_name,
                                      iscsi->initiator_username ) ) != 0 )
                return rc;
+       DBG ( "iBFT target username = %s\n",
+             ibft_string ( strings, &target->chap_name ) );
        if ( ( rc = ibft_set_string ( strings, &target->chap_secret,
                                      iscsi->initiator_password ) ) != 0 )
                return rc;
+       DBG ( "iBFT target password = <redacted>\n" );
+
        return 0;
 }
 
@@ -313,19 +360,25 @@ static int ibft_fill_target_reverse_chap ( struct ibft_target *target,
                                           struct iscsi_session *iscsi ) {
        int rc;
 
-       if ( ! iscsi->target_username )
+       if ( ! ( iscsi->status & ISCSI_STATUS_AUTH_REVERSE_REQUIRED ) )
                return 0;
-       assert ( iscsi->target_password );
+
        assert ( iscsi->initiator_username );
        assert ( iscsi->initiator_password );
+       assert ( iscsi->target_username );
+       assert ( iscsi->target_password );
 
        target->chap_type = IBFT_CHAP_MUTUAL;
        if ( ( rc = ibft_set_string ( strings, &target->reverse_chap_name,
                                      iscsi->target_username ) ) != 0 )
                return rc;
+       DBG ( "iBFT target reverse username = %s\n",
+             ibft_string ( strings, &target->chap_name ) );
        if ( ( rc = ibft_set_string ( strings, &target->reverse_chap_secret,
                                      iscsi->target_password ) ) != 0 )
                return rc;
+       DBG ( "iBFT target reverse password = <redacted>\n" );
+
        return 0;
 }
 
@@ -346,10 +399,14 @@ static int ibft_fill_target ( struct ibft_target *target,
 
        /* Fill in Target values */
        ibft_set_ipaddr ( &target->ip_address, sin_target->sin_addr );
+       DBG ( "iBFT target IP = %s\n", ibft_ipaddr ( &target->ip_address ) );
        target->socket = ntohs ( sin_target->sin_port );
+       DBG ( "iBFT target port = %d\n", target->socket );
        if ( ( rc = ibft_set_string ( strings, &target->target_name,
                                      iscsi->target_iqn ) ) != 0 )
                return rc;
+       DBG ( "iBFT target name = %s\n",
+             ibft_string ( strings, &target->target_name ) );
        if ( ( rc = ibft_fill_target_chap ( target, strings, iscsi ) ) != 0 )
                return rc;
        if ( ( rc = ibft_fill_target_reverse_chap ( target, strings,