Pc_floppy does not work

CodeforEvolution · August 5, 2021, 9:55pm

After looking over the pc_floppy driver’s code, it looks like the reason this driver isn’t functional is that the “Driver Configuration Manager” (a system module that was used for keeping track of internally connected devices on BeOS) is currently unimplemented.

The pc_floppy driver attempts to scan on the ISA bus for floppy drives using the Driver Configuration Manager here:

github.com

haiku/haiku/blob/fe3ac93cdf1d5da37b3edc52a23fcfcd214ce89b/src/add-ons/kernel/drivers/disk/floppy/pc_floppy/floppy.c#L138

    
      
          	}
          	if ((err = get_module(B_CONFIG_MANAGER_FOR_DRIVER_MODULE_NAME, (module_info **)&config_man)) < B_OK) {
          		dprintf(FLO "no config_manager module!\n");
          		put_module(B_ISA_MODULE_NAME);
          		return err;
          	}
          	for (i = 0; i < MAX_FLOPPIES; i++)
          		memset(&(floppies[i]), 0, sizeof(floppy_t));
          	current = 0;
          	register_kernel_daemon(motor_off_daemon, floppies, 5);
          	while (config_man->get_next_device_info(B_ISA_BUS, &cmcookie, &info, sizeof(struct device_info)) == B_OK) {
          		struct device_configuration *conf;
          		struct floppy *master;
          		struct floppy *last = NULL;
          		int flops;
          		if ((info.config_status != B_OK) || 
          				((info.flags & B_DEVICE_INFO_ENABLED) == 0) || 
          				((info.flags & B_DEVICE_INFO_CONFIGURED) == 0) || 
          				(info.devtype.base != PCI_mass_storage) || 
          				(info.devtype.subtype != PCI_floppy) || 
          				(info.devtype.interface != 0) /* XXX: needed ?? */)

This is the code attempting to be called:

github.com

haiku/haiku/blob/fe3ac93cdf1d5da37b3edc52a23fcfcd214ce89b/src/add-ons/kernel/bus_managers/config_manager/config_manager.c#L30

    
      
          #define B_CONFIG_MANAGER_FOR_BUS_MODULE_NAME "bus_managers/config_manager/bus/v1"
          
          
#define FUNCTION(x, y...) dprintf("%s" x, __FUNCTION__, y)
          #define TRACE(x) dprintf x
          
          

          
//	Driver module API
          
          

          
static status_t
          driver_get_next_device_info(bus_type bus, uint64 *cookie, struct device_info *info, uint32 size)
          {
          	FUNCTION("(bus = %d, cookie = %" B_PRId64 ")\n", bus, *cookie);
          	return B_ENTRY_NOT_FOUND;
          }
          
          

          
static status_t
          driver_get_device_info_for(uint64 id, struct device_info *info, uint32 size)
          {
          	FUNCTION("(id = %" B_PRId64 ")\n", id);

I think the solution here would involve wrapping the Driver Configuration Manager around the newer Device Manager framework. However, I don’t think the Device Manager framework has any ISA PNP scanning capabilities… Furthermore, Haiku’s ISA Bus Manager has holes when it comes to DMA transfers. (Though that wouldn’t be a problem for smaller transfers I would think? )