Hello All,
I have a question on the sys/pci/pci.c code, and its use of the
newbus architecture. An example of the code in question is:
static struct resource *
pci_alloc_resource(device_t dev, device_t child, int type, int *rid,
u_long start, u_long end, u_long count, u_int flags)
{
struct pci_devinfo *dinfo = device_get_ivars(child);
^^^^^ Looks wrong
struct resource_list *rl = &dinfo->resources;
return resource_list_alloc(rl, dev, child, type, rid,
start, end, count, flags);
}
I don't understand the line that extracts the ivars from the child
device. Isn't it the case that the ivars are a property of the bus
device (dev)? In any case, the child device might not be directly
descended from the pci bus (e.g. it could be attached thru' the
bridge isab0: <VIA 82C586 PCI-ISA bridge>).
It strikes me that the assignment is unsafe. I would have thought
that the code should be:
static struct resource *
pci_alloc_resource(device_t dev, device_t child, int type, int *rid,
u_long start, u_long end, u_long count, u_int flags)
{
struct pci_devinfo *dinfo = device_get_ivars(dev);
^^^ CHANGED
struct resource_list *rl = &dinfo->resources;
return resource_list_alloc(rl, dev, child, type, rid,
start, end, count, flags);
}
Why are the ivars extracted from the child, and not the bus device?
Kind Regards,
Mike Kennett
([EMAIL PROTECTED])
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-hackers" in the body of the message
