Messages in this thread | | | From | Oded Gabbay <> | Date | Wed, 27 Feb 2019 12:17:44 +0200 | Subject | Re: [PATCH -next] habanalabs: goya: Make some functions static |
| |
On Wed, Feb 27, 2019 at 10:39 AM Greg Kroah-Hartman <gregkh@linuxfoundation.org> wrote: > > On Wed, Feb 27, 2019 at 10:31:48AM +0200, Oded Gabbay wrote: > > On Wed, Feb 27, 2019 at 9:05 AM Yue Haibing <yuehaibing@huawei.com> wrote: > > > > > > From: YueHaibing <yuehaibing@huawei.com> > > > > > > Fixes the following sparse warnings: > > > > > > drivers/misc/habanalabs/goya/goya.c:1233:5: warning: symbol 'goya_init_cpu_queues' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:2914:5: warning: symbol 'goya_suspend' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:2939:5: warning: symbol 'goya_resume' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:2952:5: warning: symbol 'goya_mmap' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:2957:5: warning: symbol 'goya_cb_mmap' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:2973:6: warning: symbol 'goya_ring_doorbell' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3063:6: warning: symbol 'goya_flush_pq_write' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3068:6: warning: symbol 'goya_dma_alloc_coherent' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3074:6: warning: symbol 'goya_dma_free_coherent' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3080:6: warning: symbol 'goya_get_int_queue_base' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3138:5: warning: symbol 'goya_send_job_on_qman0' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3295:5: warning: symbol 'goya_test_queue' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3411:6: warning: symbol 'goya_dma_pool_zalloc' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3420:6: warning: symbol 'goya_dma_pool_free' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3426:6: warning: symbol 'goya_cpu_accessible_dma_pool_alloc' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3442:6: warning: symbol 'goya_cpu_accessible_dma_pool_free' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3452:5: warning: symbol 'goya_dma_map_sg' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3461:6: warning: symbol 'goya_dma_unmap_sg' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:3467:5: warning: symbol 'goya_get_dma_desc_list_size' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:396:5: warning: symbol 'goya_send_pci_access_msg' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:417:5: warning: symbol 'goya_pci_bars_map' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:4202:5: warning: symbol 'goya_parse_cb_no_mmu' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:4253:5: warning: symbol 'goya_parse_cb_no_ext_quque' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:4286:5: warning: symbol 'goya_cs_parser' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:4299:6: warning: symbol 'goya_add_end_of_cb_packets' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:557:6: warning: symbol 'goya_reset_link_through_bridge' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:774:5: warning: symbol 'goya_early_fini' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:857:6: warning: symbol 'goya_late_fini' was not declared. Should it be static? > > > drivers/misc/habanalabs/goya/goya.c:971:5: warning: symbol 'goya_sw_fini' was not declared. Should it be static? > > > > > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > > > --- > > > drivers/misc/habanalabs/goya/goya.c | 86 +++++++++++++++++++------------------ > > > 1 file changed, 44 insertions(+), 42 deletions(-) > > > > > > diff --git a/drivers/misc/habanalabs/goya/goya.c b/drivers/misc/habanalabs/goya/goya.c > > > index fc53cdb..e0f1ba6 100644 > > > --- a/drivers/misc/habanalabs/goya/goya.c > > > +++ b/drivers/misc/habanalabs/goya/goya.c > > > @@ -393,7 +393,7 @@ static void goya_get_fixed_properties(struct hl_device *hdev) > > > prop->high_pll = PLL_HIGH_DEFAULT; > > > } > > > > > > -int goya_send_pci_access_msg(struct hl_device *hdev, u32 opcode) > > > +static int goya_send_pci_access_msg(struct hl_device *hdev, u32 opcode) > > > { > > > struct armcp_packet pkt; > > > > > > @@ -414,7 +414,7 @@ int goya_send_pci_access_msg(struct hl_device *hdev, u32 opcode) > > > * Returns 0 on success > > > * > > > */ > > > -int goya_pci_bars_map(struct hl_device *hdev) > > > +static int goya_pci_bars_map(struct hl_device *hdev) > > > { > > > struct pci_dev *pdev = hdev->pdev; > > > int rc; > > > @@ -554,7 +554,7 @@ static int goya_iatu_write(struct hl_device *hdev, u32 addr, u32 data) > > > return 0; > > > } > > > > > > -void goya_reset_link_through_bridge(struct hl_device *hdev) > > > +static void goya_reset_link_through_bridge(struct hl_device *hdev) > > > { > > > struct pci_dev *pdev = hdev->pdev; > > > struct pci_dev *parent_port; > > > @@ -771,7 +771,7 @@ static int goya_early_init(struct hl_device *hdev) > > > * Unmap PCI bars > > > * > > > */ > > > -int goya_early_fini(struct hl_device *hdev) > > > +static int goya_early_fini(struct hl_device *hdev) > > > { > > > goya_pci_bars_unmap(hdev); > > > > > > @@ -854,7 +854,7 @@ static int goya_late_init(struct hl_device *hdev) > > > * > > > * Free sensors allocated structures > > > */ > > > -void goya_late_fini(struct hl_device *hdev) > > > +static void goya_late_fini(struct hl_device *hdev) > > > { > > > const struct hwmon_channel_info **channel_info_arr; > > > int i = 0; > > > @@ -968,7 +968,7 @@ static int goya_sw_init(struct hl_device *hdev) > > > * @hdev: pointer to hl_device structure > > > * > > > */ > > > -int goya_sw_fini(struct hl_device *hdev) > > > +static int goya_sw_fini(struct hl_device *hdev) > > > { > > > struct goya_device *goya = hdev->asic_specific; > > > > > > @@ -1230,7 +1230,7 @@ static void goya_resume_external_queues(struct hl_device *hdev) > > > * Returns 0 on success > > > * > > > */ > > > -int goya_init_cpu_queues(struct hl_device *hdev) > > > +static int goya_init_cpu_queues(struct hl_device *hdev) > > > { > > > struct goya_device *goya = hdev->asic_specific; > > > struct hl_eq *eq; > > > @@ -2911,7 +2911,7 @@ static void goya_hw_fini(struct hl_device *hdev, bool hard_reset) > > > } > > > } > > > > > > -int goya_suspend(struct hl_device *hdev) > > > +static int goya_suspend(struct hl_device *hdev) > > > { > > > int rc; > > > > > > @@ -2936,7 +2936,7 @@ int goya_suspend(struct hl_device *hdev) > > > return rc; > > > } > > > > > > -int goya_resume(struct hl_device *hdev) > > > +static int goya_resume(struct hl_device *hdev) > > > { > > > int rc; > > > > > > @@ -2949,13 +2949,13 @@ int goya_resume(struct hl_device *hdev) > > > return rc; > > > } > > > > > > -int goya_mmap(struct hl_fpriv *hpriv, struct vm_area_struct *vma) > > > +static int goya_mmap(struct hl_fpriv *hpriv, struct vm_area_struct *vma) > > > { > > > return -EINVAL; > > > } > > > > > > -int goya_cb_mmap(struct hl_device *hdev, struct vm_area_struct *vma, > > > - u64 kaddress, phys_addr_t paddress, u32 size) > > > +static int goya_cb_mmap(struct hl_device *hdev, struct vm_area_struct *vma, > > > + u64 kaddress, phys_addr_t paddress, u32 size) > > > { > > > int rc; > > > > > > @@ -2970,7 +2970,7 @@ int goya_cb_mmap(struct hl_device *hdev, struct vm_area_struct *vma, > > > return rc; > > > } > > > > > > -void goya_ring_doorbell(struct hl_device *hdev, u32 hw_queue_id, u32 pi) > > > +static void goya_ring_doorbell(struct hl_device *hdev, u32 hw_queue_id, u32 pi) > > > { > > > u32 db_reg_offset, db_value; > > > bool invalid_queue = false; > > > @@ -3060,25 +3060,25 @@ void goya_ring_doorbell(struct hl_device *hdev, u32 hw_queue_id, u32 pi) > > > GOYA_ASYNC_EVENT_ID_PI_UPDATE); > > > } > > > > > > -void goya_flush_pq_write(struct hl_device *hdev, u64 *pq, u64 exp_val) > > > +static void goya_flush_pq_write(struct hl_device *hdev, u64 *pq, u64 exp_val) > > > { > > > /* Not needed in Goya */ > > > } > > > > > > -void *goya_dma_alloc_coherent(struct hl_device *hdev, size_t size, > > > - dma_addr_t *dma_handle, gfp_t flags) > > > +static void *goya_dma_alloc_coherent(struct hl_device *hdev, size_t size, > > > + dma_addr_t *dma_handle, gfp_t flags) > > > { > > > return dma_alloc_coherent(&hdev->pdev->dev, size, dma_handle, flags); > > > } > > > > > > -void goya_dma_free_coherent(struct hl_device *hdev, size_t size, void *cpu_addr, > > > - dma_addr_t dma_handle) > > > +static void goya_dma_free_coherent(struct hl_device *hdev, size_t size, > > > + void *cpu_addr, dma_addr_t dma_handle) > > > { > > > dma_free_coherent(&hdev->pdev->dev, size, cpu_addr, dma_handle); > > > } > > > > > > -void *goya_get_int_queue_base(struct hl_device *hdev, u32 queue_id, > > > - dma_addr_t *dma_handle, u16 *queue_len) > > > +static void *goya_get_int_queue_base(struct hl_device *hdev, u32 queue_id, > > > + dma_addr_t *dma_handle, u16 *queue_len) > > > { > > > void *base; > > > u32 offset; > > > @@ -3135,7 +3135,7 @@ void *goya_get_int_queue_base(struct hl_device *hdev, u32 queue_id, > > > return base; > > > } > > > > > > -int goya_send_job_on_qman0(struct hl_device *hdev, struct hl_cs_job *job) > > > +static int goya_send_job_on_qman0(struct hl_device *hdev, struct hl_cs_job *job) > > > { > > > struct goya_device *goya = hdev->asic_specific; > > > struct packet_msg_prot *fence_pkt; > > > @@ -3214,7 +3214,7 @@ int goya_send_job_on_qman0(struct hl_device *hdev, struct hl_cs_job *job) > > > } > > > > > > int goya_send_cpu_message(struct hl_device *hdev, u32 *msg, u16 len, > > > - u32 timeout, long *result) > > > + u32 timeout, long *result) > > > { > > > struct goya_device *goya = hdev->asic_specific; > > > struct armcp_packet *pkt; > > > @@ -3292,7 +3292,7 @@ int goya_send_cpu_message(struct hl_device *hdev, u32 *msg, u16 len, > > > return rc; > > > } > > > > > > -int goya_test_queue(struct hl_device *hdev, u32 hw_queue_id) > > > +static int goya_test_queue(struct hl_device *hdev, u32 hw_queue_id) > > > { > > > struct packet_msg_prot *fence_pkt; > > > dma_addr_t pkt_dma_addr; > > > @@ -3414,8 +3414,8 @@ static int goya_test_queues(struct hl_device *hdev) > > > return ret_val; > > > } > > > > > > -void *goya_dma_pool_zalloc(struct hl_device *hdev, size_t size, gfp_t mem_flags, > > > - dma_addr_t *dma_handle) > > > +static void *goya_dma_pool_zalloc(struct hl_device *hdev, size_t size, > > > + gfp_t mem_flags, dma_addr_t *dma_handle) > > > { > > > if (size > GOYA_DMA_POOL_BLK_SIZE) > > > return NULL; > > > @@ -3423,14 +3423,15 @@ void *goya_dma_pool_zalloc(struct hl_device *hdev, size_t size, gfp_t mem_flags, > > > return dma_pool_zalloc(hdev->dma_pool, mem_flags, dma_handle); > > > } > > > > > > -void goya_dma_pool_free(struct hl_device *hdev, void *vaddr, > > > - dma_addr_t dma_addr) > > > +static void goya_dma_pool_free(struct hl_device *hdev, void *vaddr, > > > + dma_addr_t dma_addr) > > > { > > > dma_pool_free(hdev->dma_pool, vaddr, dma_addr); > > > } > > > > > > -void *goya_cpu_accessible_dma_pool_alloc(struct hl_device *hdev, size_t size, > > > - dma_addr_t *dma_handle) > > > +static void *goya_cpu_accessible_dma_pool_alloc(struct hl_device *hdev, > > > + size_t size, > > > + dma_addr_t *dma_handle) > > > { > > > u64 kernel_addr; > > > > > > @@ -3445,8 +3446,8 @@ void *goya_cpu_accessible_dma_pool_alloc(struct hl_device *hdev, size_t size, > > > return (void *) (uintptr_t) kernel_addr; > > > } > > > > > > -void goya_cpu_accessible_dma_pool_free(struct hl_device *hdev, size_t size, > > > - void *vaddr) > > > +static void goya_cpu_accessible_dma_pool_free(struct hl_device *hdev, > > > + size_t size, void *vaddr) > > > { > > > /* roundup to CPU_PKT_SIZE */ > > > size = (size + (CPU_PKT_SIZE - 1)) & CPU_PKT_MASK; > > > @@ -3455,8 +3456,8 @@ void goya_cpu_accessible_dma_pool_free(struct hl_device *hdev, size_t size, > > > size); > > > } > > > > > > -int goya_dma_map_sg(struct hl_device *hdev, struct scatterlist *sg, int nents, > > > - enum dma_data_direction dir) > > > +static int goya_dma_map_sg(struct hl_device *hdev, struct scatterlist *sg, > > > + int nents, enum dma_data_direction dir) > > > { > > > if (!dma_map_sg(&hdev->pdev->dev, sg, nents, dir)) > > > return -ENOMEM; > > > @@ -3464,14 +3465,14 @@ int goya_dma_map_sg(struct hl_device *hdev, struct scatterlist *sg, int nents, > > > return 0; > > > } > > > > > > -void goya_dma_unmap_sg(struct hl_device *hdev, struct scatterlist *sg, > > > - int nents, enum dma_data_direction dir) > > > +static void goya_dma_unmap_sg(struct hl_device *hdev, struct scatterlist *sg, > > > + int nents, enum dma_data_direction dir) > > > { > > > dma_unmap_sg(&hdev->pdev->dev, sg, nents, dir); > > > } > > > > > > -u32 goya_get_dma_desc_list_size(struct hl_device *hdev, > > > - struct sg_table *sgt) > > > +static u32 goya_get_dma_desc_list_size(struct hl_device *hdev, > > > + struct sg_table *sgt) > > > { > > > struct scatterlist *sg, *sg_next_iter; > > > u32 count, dma_desc_cnt; > > > @@ -4207,7 +4208,8 @@ static int goya_parse_cb_mmu(struct hl_device *hdev, > > > return rc; > > > } > > > > > > -int goya_parse_cb_no_mmu(struct hl_device *hdev, struct hl_cs_parser *parser) > > > +static int goya_parse_cb_no_mmu(struct hl_device *hdev, > > > + struct hl_cs_parser *parser) > > > { > > > u64 patched_cb_handle; > > > int rc; > > > @@ -4258,7 +4260,7 @@ int goya_parse_cb_no_mmu(struct hl_device *hdev, struct hl_cs_parser *parser) > > > return rc; > > > } > > > > > > -int goya_parse_cb_no_ext_quque(struct hl_device *hdev, > > > +static int goya_parse_cb_no_ext_quque(struct hl_device *hdev, > > > struct hl_cs_parser *parser) > > > { > > > struct asic_fixed_properties *asic_prop = &hdev->asic_prop; > > > @@ -4291,7 +4293,7 @@ int goya_parse_cb_no_ext_quque(struct hl_device *hdev, > > > return 0; > > > } > > > > > > -int goya_cs_parser(struct hl_device *hdev, struct hl_cs_parser *parser) > > > +static int goya_cs_parser(struct hl_device *hdev, struct hl_cs_parser *parser) > > > { > > > struct goya_device *goya = hdev->asic_specific; > > > > > > @@ -4304,8 +4306,8 @@ int goya_cs_parser(struct hl_device *hdev, struct hl_cs_parser *parser) > > > return goya_parse_cb_no_mmu(hdev, parser); > > > } > > > > > > -void goya_add_end_of_cb_packets(u64 kernel_address, u32 len, u64 cq_addr, > > > - u32 cq_val, u32 msix_vec) > > > +static void goya_add_end_of_cb_packets(u64 kernel_address, u32 len, u64 cq_addr, > > > + u32 cq_val, u32 msix_vec) > > > { > > > struct packet_msg_prot *cq_pkt; > > > > > > -- > > > 2.7.0 > > > > > > > > > > Hi, > > Thanks for this but I prefer not to merge this. > > I prefer to eliminate the warnings by adding declarations in goyaP.h > > for the above functions. > > These functions are called from different files who belong to the goya > > platform. That code has not yet been upstreamed,so when that happens, > > these functions will have to be not static. > > Therefore, no point in adding this patch now and then reverting it later. > > One hint, if you do not take this patch now, people will keep sending it > over and over again :) > > We don't know that you will be using these functions later, as that > patch hasn't even been posted yet. So it's good to take this for now, > and when/if you really do need to use these functions in a future patch, > it's trivial for you to just make these non-static. > > So I recommend taking this patch. > > thanks, > > greg k-h
I sent a somewhat revised patch to fix these issues. I've gone over all the functions and indeed half of them could be made static, but I added declarations for the other half. I also fixed 3 such warnings in other files.
Oded
| |