On Wed, 22 Jan 2025 at 19:09, Adriano Cordova <adria...@gmail.com> wrote: > > The following symbols are exposed: > - efi_reinstall_protocol_interface > This is done so that the device path protocol interface > of the network device can be changed internally by u-boot > when a new bootfile gets downloaded. > - eth_set_dev > To support multiple network udevices > - efi_close_event > This comes in preparation to support unregistering > an EFI network device from the EFI network stack when > the underlying U-boot device gets removed > - efi_[dis]connect_controller > The EFI network driver uses ConnectController to add a > NIC to the EFI network stack. > - efi_uninstall_protocol_interface > connect_controler for the efi network driver can install > protocols, which need to be uninstalled in > disconnect_controller > - EFI_SIMPLE_NETWORK_PROTOCOL_GUID
All of these function are intended to be called from an EFI application. Because this make it cumbersome to use in U-Boot internal we usually split the function to efi_reinstall_protocol_interface_ext -> called in EFI context and is a wrapper that adds EFI_ENTRY/EXIT efi_reinstall_protocol_interface -> the actual implementation, that we can use internally Heinrich, I assume we want to keep this similar ? /Ilias > > Signed-off-by: Adriano Cordova <adriano.cord...@canonical.com> > --- > include/efi_loader.h | 22 ++++++++++++++++++++++ > include/net-common.h | 1 + > lib/efi_loader/efi_boottime.c | 23 +++++++++++------------ > lib/efi_loader/efi_net.c | 2 +- > lib/efi_selftest/efi_selftest_snp.c | 1 - > 5 files changed, 35 insertions(+), 14 deletions(-) > > diff --git a/include/efi_loader.h b/include/efi_loader.h > index 9afbec35eb..626d2bbb1a 100644 > --- a/include/efi_loader.h > +++ b/include/efi_loader.h > @@ -308,6 +308,8 @@ extern const efi_guid_t efi_guid_host_dev; > #endif > /* GUID of the EFI_BLOCK_IO_PROTOCOL */ > extern const efi_guid_t efi_block_io_guid; > +/* GUID of the EFI_SIMPLE_NETWORK_PROTOCOL */ > +extern const efi_guid_t efi_net_guid; > extern const efi_guid_t efi_global_variable_guid; > extern const efi_guid_t efi_guid_console_control; > extern const efi_guid_t efi_guid_device_path; > @@ -711,6 +713,15 @@ efi_status_t efi_search_protocol(const efi_handle_t > handle, > efi_status_t efi_add_protocol(const efi_handle_t handle, > const efi_guid_t *protocol, > void *protocol_interface); > +/* Uninstall new protocol on a handle */ > +efi_status_t efi_uninstall_protocol > + (efi_handle_t handle, const efi_guid_t *protocol, > + void *protocol_interface, bool preserve); > +/* Reinstall a protocol on a handle */ > +efi_status_t EFIAPI efi_reinstall_protocol_interface( > + efi_handle_t handle, > + const efi_guid_t *protocol, > + void *old_interface, void *new_interface); > /* Open protocol */ > efi_status_t efi_protocol_open(struct efi_handler *handler, > void **protocol_interface, void *agent_handle, > @@ -721,6 +732,15 @@ efi_status_t EFIAPI > efi_install_multiple_protocol_interfaces(efi_handle_t *handle, ...); > efi_status_t EFIAPI > efi_uninstall_multiple_protocol_interfaces(efi_handle_t handle, ...); > +/* Connect and disconnect controller */ > +efi_status_t EFIAPI efi_connect_controller(efi_handle_t controller_handle, > + efi_handle_t *driver_image_handle, > + struct efi_device_path > *remain_device_path, > + bool recursive); > +efi_status_t EFIAPI efi_disconnect_controller( > + efi_handle_t controller_handle, > + efi_handle_t driver_image_handle, > + efi_handle_t child_handle); > /* Get handles that support a given protocol */ > efi_status_t EFIAPI efi_locate_handle_buffer( > enum efi_locate_search_type search_type, > @@ -741,6 +761,8 @@ efi_status_t efi_create_event(uint32_t type, efi_uintn_t > notify_tpl, > void *context), > void *notify_context, const efi_guid_t *group, > struct efi_event **event); > +/* Call this to close an event */ > +efi_status_t EFIAPI efi_close_event(struct efi_event *event); > /* Call this to set a timer */ > efi_status_t efi_set_timer(struct efi_event *event, enum efi_timer_delay > type, > uint64_t trigger_time); > diff --git a/include/net-common.h b/include/net-common.h > index 29d31f3726..1d507b13b0 100644 > --- a/include/net-common.h > +++ b/include/net-common.h > @@ -291,6 +291,7 @@ struct eth_ops { > #define eth_get_ops(dev) ((struct eth_ops *)(dev)->driver->ops) > > struct udevice *eth_get_dev(void); /* get the current device */ > +void eth_set_dev(struct udevice *dev); /* set a device */ > unsigned char *eth_get_ethaddr(void); /* get the current device MAC */ > int eth_rx(void); /* Check for received packets */ > void eth_halt(void); /* stop SCC */ > diff --git a/lib/efi_loader/efi_boottime.c b/lib/efi_loader/efi_boottime.c > index 080e7f78ae..79ebe94889 100644 > --- a/lib/efi_loader/efi_boottime.c > +++ b/lib/efi_loader/efi_boottime.c > @@ -58,9 +58,9 @@ static efi_handle_t current_image; > static volatile gd_t *efi_gd, *app_gd; > #endif > > -static efi_status_t efi_uninstall_protocol > - (efi_handle_t handle, const efi_guid_t *protocol, > - void *protocol_interface, bool preserve); > +efi_status_t efi_uninstall_protocol > + (efi_handle_t handle, const efi_guid_t *protocol, > + void *protocol_interface, bool preserve); > > /* 1 if inside U-Boot code, 0 if inside EFI payload code */ > static int entry_count = 1; > @@ -98,12 +98,11 @@ const efi_guid_t efi_guid_load_file2_protocol = > EFI_LOAD_FILE2_PROTOCOL_GUID; > /* GUID of the SMBIOS table */ > const efi_guid_t smbios_guid = SMBIOS_TABLE_GUID; > > -static efi_status_t EFIAPI efi_disconnect_controller( > +efi_status_t EFIAPI efi_disconnect_controller( > efi_handle_t controller_handle, > efi_handle_t driver_image_handle, > efi_handle_t child_handle); > > -static > efi_status_t EFIAPI efi_connect_controller(efi_handle_t controller_handle, > efi_handle_t *driver_image_handle, > struct efi_device_path > *remain_device_path, > @@ -1037,7 +1036,7 @@ static efi_status_t EFIAPI efi_signal_event_ext(struct > efi_event *event) > * > * Return: status code > */ > -static efi_status_t EFIAPI efi_close_event(struct efi_event *event) > +efi_status_t EFIAPI efi_close_event(struct efi_event *event) > { > struct efi_register_notify_event *item, *next; > > @@ -1378,9 +1377,9 @@ static efi_status_t efi_disconnect_all_drivers > * > * Return: status code > */ > -static efi_status_t efi_uninstall_protocol > - (efi_handle_t handle, const efi_guid_t *protocol, > - void *protocol_interface, bool preserve) > +efi_status_t efi_uninstall_protocol > + (efi_handle_t handle, const efi_guid_t *protocol, > + void *protocol_interface, bool preserve) > { > struct efi_handler *handler; > struct efi_open_protocol_info_item *item; > @@ -3665,7 +3664,7 @@ static efi_status_t efi_connect_single_controller( > * > * Return: status code > */ > -static efi_status_t EFIAPI efi_connect_controller( > +efi_status_t EFIAPI efi_connect_controller( > efi_handle_t controller_handle, > efi_handle_t *driver_image_handle, > struct efi_device_path *remain_device_path, > @@ -3733,7 +3732,7 @@ out: > * > * Return: status code > */ > -static efi_status_t EFIAPI efi_reinstall_protocol_interface( > +efi_status_t EFIAPI efi_reinstall_protocol_interface( > efi_handle_t handle, const efi_guid_t *protocol, > void *old_interface, void *new_interface) > { > @@ -3844,7 +3843,7 @@ static efi_status_t efi_get_child_controllers( > * > * Return: status code > */ > -static efi_status_t EFIAPI efi_disconnect_controller( > +efi_status_t EFIAPI efi_disconnect_controller( > efi_handle_t controller_handle, > efi_handle_t driver_image_handle, > efi_handle_t child_handle) > diff --git a/lib/efi_loader/efi_net.c b/lib/efi_loader/efi_net.c > index 67593ef50c..918d43c70f 100644 > --- a/lib/efi_loader/efi_net.c > +++ b/lib/efi_loader/efi_net.c > @@ -22,7 +22,7 @@ > #include <vsprintf.h> > #include <net.h> > > -static const efi_guid_t efi_net_guid = EFI_SIMPLE_NETWORK_PROTOCOL_GUID; > +const efi_guid_t efi_net_guid = EFI_SIMPLE_NETWORK_PROTOCOL_GUID; > static const efi_guid_t efi_pxe_base_code_protocol_guid = > EFI_PXE_BASE_CODE_PROTOCOL_GUID; > static struct efi_pxe_packet *dhcp_ack; > diff --git a/lib/efi_selftest/efi_selftest_snp.c > b/lib/efi_selftest/efi_selftest_snp.c > index 15af8d3e18..b00c76c2f1 100644 > --- a/lib/efi_selftest/efi_selftest_snp.c > +++ b/lib/efi_selftest/efi_selftest_snp.c > @@ -67,7 +67,6 @@ struct dhcp { > static struct efi_boot_services *boottime; > static struct efi_simple_network *net; > static struct efi_event *timer; > -static const efi_guid_t efi_net_guid = EFI_SIMPLE_NETWORK_PROTOCOL_GUID; > /* IP packet ID */ > static unsigned int net_ip_id; > > -- > 2.43.0 >