drivers/irqchip/irq-apple-aic.c:980 aic_of_ic_init() error: uninitialized symbol 'start_off'.

From: Dan Carpenter
Date: Mon Jun 06 2022 - 07:25:40 EST

Next message: Amit Kumar Mahapatra: "[RFC PATCH 0/2] spi: Add support for stacked/parallel memories"
Previous message: Dmitry Rokosov: "[PATCH v2] iio: trigger: warn about non-registered iio trigger getting attempt"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 700170bf6b4d773e328fa54ebb70ba444007c702
commit: a801f0ee563b8180caf186493806a145a75b4a3c irqchip/apple-aic: Support multiple dies
config: arm64-randconfig-m031-20220530 (https://download.01.org/0day-ci/archive/20220601/202206011052.du9opGWF-lkp@xxxxxxxxx/config)
compiler: aarch64-linux-gcc (GCC) 11.3.0

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@xxxxxxxxx>
Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

New smatch warnings:
drivers/irqchip/irq-apple-aic.c:980 aic_of_ic_init() error: uninitialized symbol 'start_off'.

Old smatch warnings:
drivers/irqchip/irq-apple-aic.c:943 aic_of_ic_init() warn: possible memory leak of 'irqc'
drivers/irqchip/irq-apple-aic.c:965 aic_of_ic_init() error: uninitialized symbol 'off'.
drivers/irqchip/irq-apple-aic.c:1029 aic_of_ic_init() warn: 'regs' from of_iomap() not released on lines: 937,943.

vim +/start_off +980 drivers/irqchip/irq-apple-aic.c

76cde26394114f Hector Martin 2021-01-21 923 static int __init aic_of_ic_init(struct device_node *node, struct device_node *parent)
76cde26394114f Hector Martin 2021-01-21 924 {
a801f0ee563b81 Hector Martin 2022-03-10 925 int i, die;
a801f0ee563b81 Hector Martin 2022-03-10 926 u32 off, start_off;
76cde26394114f Hector Martin 2021-01-21 927 void __iomem *regs;
76cde26394114f Hector Martin 2021-01-21 928 struct aic_irq_chip *irqc;
2cf68211664acd Hector Martin 2022-03-10 929 const struct of_device_id *match;
76cde26394114f Hector Martin 2021-01-21 930
76cde26394114f Hector Martin 2021-01-21 931 regs = of_iomap(node, 0);
76cde26394114f Hector Martin 2021-01-21 932 if (WARN_ON(!regs))
76cde26394114f Hector Martin 2021-01-21 933 return -EIO;
76cde26394114f Hector Martin 2021-01-21 934
76cde26394114f Hector Martin 2021-01-21 935 irqc = kzalloc(sizeof(*irqc), GFP_KERNEL);
76cde26394114f Hector Martin 2021-01-21 936 if (!irqc)
76cde26394114f Hector Martin 2021-01-21 937 return -ENOMEM;
76cde26394114f Hector Martin 2021-01-21 938
76cde26394114f Hector Martin 2021-01-21 939 irqc->base = regs;
76cde26394114f Hector Martin 2021-01-21 940
2cf68211664acd Hector Martin 2022-03-10 941 match = of_match_node(aic_info_match, node);
2cf68211664acd Hector Martin 2022-03-10 942 if (!match)
2cf68211664acd Hector Martin 2022-03-10 943 return -ENODEV;

ret = -ENOMEM;
goto err_free_irqc;

2cf68211664acd Hector Martin 2022-03-10 944
2cf68211664acd Hector Martin 2022-03-10 945 irqc->info = *(struct aic_info *)match->data;
2cf68211664acd Hector Martin 2022-03-10 946
2cf68211664acd Hector Martin 2022-03-10 947 aic_irqc = irqc;
2cf68211664acd Hector Martin 2022-03-10 948
dc97fd6fec0099 Hector Martin 2022-03-10 949 switch (irqc->info.version) {
dc97fd6fec0099 Hector Martin 2022-03-10 950 case 1: {
dc97fd6fec0099 Hector Martin 2022-03-10 951 u32 info;
dc97fd6fec0099 Hector Martin 2022-03-10 952
76cde26394114f Hector Martin 2021-01-21 953 info = aic_ic_read(irqc, AIC_INFO);
7c841f5f6fa3f9 Hector Martin 2022-03-10 954 irqc->nr_irq = FIELD_GET(AIC_INFO_NR_IRQ, info);
dc97fd6fec0099 Hector Martin 2022-03-10 955 irqc->max_irq = AIC_MAX_IRQ;
a801f0ee563b81 Hector Martin 2022-03-10 956 irqc->nr_die = irqc->max_die = 1;
dc97fd6fec0099 Hector Martin 2022-03-10 957
a801f0ee563b81 Hector Martin 2022-03-10 958 off = start_off = irqc->info.target_cpu;
dc97fd6fec0099 Hector Martin 2022-03-10 959 off += sizeof(u32) * irqc->max_irq; /* TARGET_CPU */
dc97fd6fec0099 Hector Martin 2022-03-10 960
dc97fd6fec0099 Hector Martin 2022-03-10 961 break;
dc97fd6fec0099 Hector Martin 2022-03-10 962 }
dc97fd6fec0099 Hector Martin 2022-03-10 963 }
dc97fd6fec0099 Hector Martin 2022-03-10 964
dc97fd6fec0099 Hector Martin 2022-03-10 965 irqc->info.sw_set = off;
dc97fd6fec0099 Hector Martin 2022-03-10 966 off += sizeof(u32) * (irqc->max_irq >> 5); /* SW_SET */
dc97fd6fec0099 Hector Martin 2022-03-10 967 irqc->info.sw_clr = off;
dc97fd6fec0099 Hector Martin 2022-03-10 968 off += sizeof(u32) * (irqc->max_irq >> 5); /* SW_CLR */
dc97fd6fec0099 Hector Martin 2022-03-10 969 irqc->info.mask_set = off;
dc97fd6fec0099 Hector Martin 2022-03-10 970 off += sizeof(u32) * (irqc->max_irq >> 5); /* MASK_SET */
dc97fd6fec0099 Hector Martin 2022-03-10 971 irqc->info.mask_clr = off;
dc97fd6fec0099 Hector Martin 2022-03-10 972 off += sizeof(u32) * (irqc->max_irq >> 5); /* MASK_CLR */
dc97fd6fec0099 Hector Martin 2022-03-10 973 off += sizeof(u32) * (irqc->max_irq >> 5); /* HW_STATE */
76cde26394114f Hector Martin 2021-01-21 974
2cf68211664acd Hector Martin 2022-03-10 975 if (irqc->info.fast_ipi)
2cf68211664acd Hector Martin 2022-03-10 976 static_branch_enable(&use_fast_ipi);
2cf68211664acd Hector Martin 2022-03-10 977 else
2cf68211664acd Hector Martin 2022-03-10 978 static_branch_disable(&use_fast_ipi);
2cf68211664acd Hector Martin 2022-03-10 979
a801f0ee563b81 Hector Martin 2022-03-10 @980 irqc->info.die_stride = off - start_off;
a801f0ee563b81 Hector Martin 2022-03-10 981
7c841f5f6fa3f9 Hector Martin 2022-03-10 982 irqc->hw_domain = irq_domain_create_tree(of_node_to_fwnode(node),
76cde26394114f Hector Martin 2021-01-21 983 &aic_irq_domain_ops, irqc);
76cde26394114f Hector Martin 2021-01-21 984 if (WARN_ON(!irqc->hw_domain)) {
76cde26394114f Hector Martin 2021-01-21 985 iounmap(irqc->base);
76cde26394114f Hector Martin 2021-01-21 986 kfree(irqc);
76cde26394114f Hector Martin 2021-01-21 987 return -ENODEV;

ret = -ENODEV;
goto err_unmap;

76cde26394114f Hector Martin 2021-01-21 988 }
76cde26394114f Hector Martin 2021-01-21 989
76cde26394114f Hector Martin 2021-01-21 990 irq_domain_update_bus_token(irqc->hw_domain, DOMAIN_BUS_WIRED);
76cde26394114f Hector Martin 2021-01-21 991
76cde26394114f Hector Martin 2021-01-21 992 if (aic_init_smp(irqc, node)) {
76cde26394114f Hector Martin 2021-01-21 993 irq_domain_remove(irqc->hw_domain);
76cde26394114f Hector Martin 2021-01-21 994 iounmap(irqc->base);
76cde26394114f Hector Martin 2021-01-21 995 kfree(irqc);
76cde26394114f Hector Martin 2021-01-21 996 return -ENODEV;

ret = -ENODEV;
goto err_remove_domain;

76cde26394114f Hector Martin 2021-01-21 997 }
76cde26394114f Hector Martin 2021-01-21 998
76cde26394114f Hector Martin 2021-01-21 999 set_handle_irq(aic_handle_irq);
76cde26394114f Hector Martin 2021-01-21 1000 set_handle_fiq(aic_handle_fiq);
76cde26394114f Hector Martin 2021-01-21 1001
a801f0ee563b81 Hector Martin 2022-03-10 1002 off = 0;
a801f0ee563b81 Hector Martin 2022-03-10 1003 for (die = 0; die < irqc->nr_die; die++) {
7c841f5f6fa3f9 Hector Martin 2022-03-10 1004 for (i = 0; i < BITS_TO_U32(irqc->nr_irq); i++)
a801f0ee563b81 Hector Martin 2022-03-10 1005 aic_ic_write(irqc, irqc->info.mask_set + off + i * 4, U32_MAX);
7c841f5f6fa3f9 Hector Martin 2022-03-10 1006 for (i = 0; i < BITS_TO_U32(irqc->nr_irq); i++)
a801f0ee563b81 Hector Martin 2022-03-10 1007 aic_ic_write(irqc, irqc->info.sw_clr + off + i * 4, U32_MAX);
a801f0ee563b81 Hector Martin 2022-03-10 1008 if (irqc->info.target_cpu)
7c841f5f6fa3f9 Hector Martin 2022-03-10 1009 for (i = 0; i < irqc->nr_irq; i++)
a801f0ee563b81 Hector Martin 2022-03-10 1010 aic_ic_write(irqc, irqc->info.target_cpu + off + i * 4, 1);
a801f0ee563b81 Hector Martin 2022-03-10 1011 off += irqc->info.die_stride;
a801f0ee563b81 Hector Martin 2022-03-10 1012 }
76cde26394114f Hector Martin 2021-01-21 1013
76cde26394114f Hector Martin 2021-01-21 1014 if (!is_kernel_in_hyp_mode())
76cde26394114f Hector Martin 2021-01-21 1015 pr_info("Kernel running in EL1, mapping interrupts");
76cde26394114f Hector Martin 2021-01-21 1016
2cf68211664acd Hector Martin 2022-03-10 1017 if (static_branch_likely(&use_fast_ipi))
2cf68211664acd Hector Martin 2022-03-10 1018 pr_info("Using Fast IPIs");
2cf68211664acd Hector Martin 2022-03-10 1019
76cde26394114f Hector Martin 2021-01-21 1020 cpuhp_setup_state(CPUHP_AP_IRQ_APPLE_AIC_STARTING,
76cde26394114f Hector Martin 2021-01-21 1021 "irqchip/apple-aic/ipi:starting",
76cde26394114f Hector Martin 2021-01-21 1022 aic_init_cpu, NULL);
76cde26394114f Hector Martin 2021-01-21 1023
b6ca556c352979 Marc Zyngier 2021-02-28 1024 vgic_set_kvm_info(&vgic_info);
b6ca556c352979 Marc Zyngier 2021-02-28 1025
a801f0ee563b81 Hector Martin 2022-03-10 1026 pr_info("Initialized with %d/%d IRQs * %d/%d die(s), %d FIQs, %d vIPIs",
a801f0ee563b81 Hector Martin 2022-03-10 1027 irqc->nr_irq, irqc->max_irq, irqc->nr_die, irqc->max_die, AIC_NR_FIQ, AIC_NR_SWIPI);
76cde26394114f Hector Martin 2021-01-21 1028
76cde26394114f Hector Martin 2021-01-21 1029 return 0;

return 0;

err_remove_domain:
irq_domain_remove(irqc->hw_domain);
err_unmap:
iounmap(irqc->base);
err_free_irqc:
kfree(irqc);

return ret;

76cde26394114f Hector Martin 2021-01-21 1030 }
76cde26394114f Hector Martin 2021-01-21 1031
--
0-DAY CI Kernel Test Service
https://01.org/lkp

Next message: Amit Kumar Mahapatra: "[RFC PATCH 0/2] spi: Add support for stacked/parallel memories"
Previous message: Dmitry Rokosov: "[PATCH v2] iio: trigger: warn about non-registered iio trigger getting attempt"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]