iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

hwtracing: hisi_ptt: Factor out filter allocation and release operation

Browse Source 
Factor out the allocation and release of filters. This will make it easier
to extend and manage the function of the filter.

Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Yicong Yang <yangyicong@hisilicon.com>
Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com>
Link: https://lore.kernel.org/r/20230621092804.15120-2-yangyicong@huawei.com
This commit is contained in:
Yicong Yang
2023-06-21 17:28:00 +08:00
committed by Suzuki K Poulose
parent 185891f03f
commit a3ecaba701
1 changed files with 39 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

63
drivers/hwtracing/ptt/hisi_ptt.c
View File

@@ -354,6 +354,39 @@ static int hisi_ptt_register_irq(struct hisi_ptt *hisi_ptt)
return 0; return 0;
} }
static void hisi_ptt_del_free_filter(struct hisi_ptt *hisi_ptt,
struct hisi_ptt_filter_desc *filter)
{
list_del(&filter->list);
kfree(filter);
}
static struct hisi_ptt_filter_desc *
hisi_ptt_alloc_add_filter(struct hisi_ptt *hisi_ptt, struct pci_dev *pdev)
{
struct hisi_ptt_filter_desc *filter;
filter = kzalloc(sizeof(*filter), GFP_KERNEL);
if (!filter) {
pci_err(hisi_ptt->pdev, "failed to add filter for %s\n",
pci_name(pdev));
return NULL;
}
filter->devid = PCI_DEVID(pdev->bus->number, pdev->devfn);
filter->is_port = pci_pcie_type(pdev) == PCI_EXP_TYPE_ROOT_PORT;
if (filter->is_port) {
list_add_tail(&filter->list, &hisi_ptt->port_filters);
/* Update the available port mask */
hisi_ptt->port_mask |= hisi_ptt_get_filter_val(filter->devid, true);
} else {
list_add_tail(&filter->list, &hisi_ptt->req_filters);
}
return filter;
}
static int hisi_ptt_init_filters(struct pci_dev *pdev, void *data) static int hisi_ptt_init_filters(struct pci_dev *pdev, void *data)
{ {
struct pci_dev *root_port = pcie_find_root_port(pdev); struct pci_dev *root_port = pcie_find_root_port(pdev);
@@ -374,23 +407,9 @@ static int hisi_ptt_init_filters(struct pci_dev *pdev, void *data)
* should be partial initialized and users would know which filter fails * should be partial initialized and users would know which filter fails
* through the log. Other functions of PTT device are still available. * through the log. Other functions of PTT device are still available.
*/ */
filter = kzalloc(sizeof(*filter), GFP_KERNEL); filter = hisi_ptt_alloc_add_filter(hisi_ptt, pdev);
if (!filter) { if (!filter)
pci_err(hisi_ptt->pdev, "failed to add filter %s\n", pci_name(pdev));
return -ENOMEM; return -ENOMEM;
}
filter->devid = PCI_DEVID(pdev->bus->number, pdev->devfn);
if (pci_pcie_type(pdev) == PCI_EXP_TYPE_ROOT_PORT) {
filter->is_port = true;
list_add_tail(&filter->list, &hisi_ptt->port_filters);
/* Update the available port mask */
hisi_ptt->port_mask |= hisi_ptt_get_filter_val(filter->devid, true);
} else {
list_add_tail(&filter->list, &hisi_ptt->req_filters);
}
return 0; return 0;
} }
@@ -400,15 +419,11 @@ static void hisi_ptt_release_filters(void *data)
struct hisi_ptt_filter_desc *filter, *tmp; struct hisi_ptt_filter_desc *filter, *tmp;
struct hisi_ptt *hisi_ptt = data; struct hisi_ptt *hisi_ptt = data;
list_for_each_entry_safe(filter, tmp, &hisi_ptt->req_filters, list) { list_for_each_entry_safe(filter, tmp, &hisi_ptt->req_filters, list)
list_del(&filter->list); hisi_ptt_del_free_filter(hisi_ptt, filter);
kfree(filter);
}
list_for_each_entry_safe(filter, tmp, &hisi_ptt->port_filters, list) { list_for_each_entry_safe(filter, tmp, &hisi_ptt->port_filters, list)
list_del(&filter->list); hisi_ptt_del_free_filter(hisi_ptt, filter);
kfree(filter);
}
} }
static int hisi_ptt_config_trace_buf(struct hisi_ptt *hisi_ptt) static int hisi_ptt_config_trace_buf(struct hisi_ptt *hisi_ptt)