Skip to content

Commit c16af01

Browse files
committed
resource: Introduce resource_assigned() for discerning active resources
A PCI bridge resource lifecycle involves both a "request" and "assign" phase. At any point in time that resource may not yet be assigned, or may have failed to assign (because it does not fit). There are multiple conventions to determine when assignment has not completed: IORESOURCE_UNSET, IORESOURCE_DISABLED, and checking whether the resource is parented. In code paths that are known to not be racing assignment, e.g. post subsys_initcall(), the most reliable method to judge that a bridge resource is assigned is to check the resource is parented [1]. Introduce a resource_assigned() helper for this purpose. Link: http://lore.kernel.org/2b9f7f7b-d6a4-be59-14d4-7b4ffccfe373@linux.intel.com [1] Suggested-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com> Cc: Bjorn Helgaas <bhelgaas@google.com> Reviewed-by: Jonathan Cameron <jonathan.cameron@huawei.com> Link: https://patch.msgid.link/20251113021446.436830-4-dan.j.williams@intel.com Signed-off-by: Dan Williams <dan.j.williams@intel.com>
1 parent e5b5f8b commit c16af01

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

include/linux/ioport.h

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -334,6 +334,15 @@ static inline bool resource_union(const struct resource *r1, const struct resour
334334
return true;
335335
}
336336

337+
/*
338+
* Check if this resource is added to a resource tree or detached. Caller is
339+
* responsible for not racing assignment.
340+
*/
341+
static inline bool resource_assigned(struct resource *res)
342+
{
343+
return res->parent;
344+
}
345+
337346
int find_resource_space(struct resource *root, struct resource *new,
338347
resource_size_t size, struct resource_constraint *constraint);
339348

0 commit comments

Comments
 (0)