| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <202203101947.WZi4vEux-lkp@intel.com>
Date: Thu, 10 Mar 2022 19:18:39 +0800
From: kernel test robot <lkp@...el.com>
To: Howard Yen <howardyen@...gle.com>
Cc: llvm@...ts.linux.dev, kbuild-all@...ts.01.org,
GNU/Weeb Mailing List <gwml@...r.gnuweeb.org>,
linux-kernel@...r.kernel.org,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Subject: [ammarfaizi2-block:google/android/kernel/common/android12-5.10-2021-12
1512/9999] drivers/usb/host/xhci-ring.c:2900:5: warning: no previous
prototype for function 'xhci_handle_event'
Hi Howard,
FYI, the error/warning still remains.
tree: https://github.com/ammarfaizi2/linux-block google/android/kernel/common/android12-5.10-2021-12
head: 66c74c58ab38fb1f5edb12b8bace4cf5bbad2cfd
commit: 26afe6712de0fad3a5de2b372286d029f8ff3231 [1512/9999] FROMLIST: usb: host: export symbols for xhci hooks usage
config: x86_64-randconfig-a012 (https://download.01.org/0day-ci/archive/20220310/202203101947.WZi4vEux-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 276ca87382b8f16a65bddac700202924228982f6)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/ammarfaizi2/linux-block/commit/26afe6712de0fad3a5de2b372286d029f8ff3231
git remote add ammarfaizi2-block https://github.com/ammarfaizi2/linux-block
git fetch --no-tags ammarfaizi2-block google/android/kernel/common/android12-5.10-2021-12
git checkout 26afe6712de0fad3a5de2b372286d029f8ff3231
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/usb/host/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>
All warnings (new ones prefixed by >>):
>> drivers/usb/host/xhci-ring.c:2900:5: warning: no previous prototype for function 'xhci_handle_event' [-Wmissing-prototypes]
int xhci_handle_event(struct xhci_hcd *xhci)
^
drivers/usb/host/xhci-ring.c:2900:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int xhci_handle_event(struct xhci_hcd *xhci)
^
static
>> drivers/usb/host/xhci-ring.c:2976:6: warning: no previous prototype for function 'xhci_update_erst_dequeue' [-Wmissing-prototypes]
void xhci_update_erst_dequeue(struct xhci_hcd *xhci,
^
drivers/usb/host/xhci-ring.c:2976:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void xhci_update_erst_dequeue(struct xhci_hcd *xhci,
^
static
2 warnings generated.
vim +/xhci_handle_event +2900 drivers/usb/host/xhci-ring.c
2893
2894 /*
2895 * This function handles all OS-owned events on the event ring. It may drop
2896 * xhci->lock between event processing (e.g. to pass up port status changes).
2897 * Returns >0 for "possibly more events to process" (caller should call again),
2898 * otherwise 0 if done. In future, <0 returns should indicate error code.
2899 */
> 2900 int xhci_handle_event(struct xhci_hcd *xhci)
2901 {
2902 union xhci_trb *event;
2903 int update_ptrs = 1;
2904 u32 trb_type;
2905 int ret;
2906
2907 /* Event ring hasn't been allocated yet. */
2908 if (!xhci->event_ring || !xhci->event_ring->dequeue) {
2909 xhci_err(xhci, "ERROR event ring not ready\n");
2910 return -ENOMEM;
2911 }
2912
2913 event = xhci->event_ring->dequeue;
2914 /* Does the HC or OS own the TRB? */
2915 if ((le32_to_cpu(event->event_cmd.flags) & TRB_CYCLE) !=
2916 xhci->event_ring->cycle_state)
2917 return 0;
2918
2919 trace_xhci_handle_event(xhci->event_ring, &event->generic);
2920
2921 /*
2922 * Barrier between reading the TRB_CYCLE (valid) flag above and any
2923 * speculative reads of the event's flags/data below.
2924 */
2925 rmb();
2926 trb_type = TRB_FIELD_TO_TYPE(le32_to_cpu(event->event_cmd.flags));
2927 /* FIXME: Handle more event types. */
2928
2929 switch (trb_type) {
2930 case TRB_COMPLETION:
2931 handle_cmd_completion(xhci, &event->event_cmd);
2932 break;
2933 case TRB_PORT_STATUS:
2934 handle_port_status(xhci, event);
2935 update_ptrs = 0;
2936 break;
2937 case TRB_TRANSFER:
2938 ret = handle_tx_event(xhci, &event->trans_event);
2939 if (ret >= 0)
2940 update_ptrs = 0;
2941 break;
2942 case TRB_DEV_NOTE:
2943 handle_device_notification(xhci, event);
2944 break;
2945 default:
2946 if (trb_type >= TRB_VENDOR_DEFINED_LOW)
2947 handle_vendor_event(xhci, event, trb_type);
2948 else
2949 xhci_warn(xhci, "ERROR unknown event type %d\n", trb_type);
2950 }
2951 /* Any of the above functions may drop and re-acquire the lock, so check
2952 * to make sure a watchdog timer didn't mark the host as non-responsive.
2953 */
2954 if (xhci->xhc_state & XHCI_STATE_DYING) {
2955 xhci_dbg(xhci, "xHCI host dying, returning from "
2956 "event handler.\n");
2957 return 0;
2958 }
2959
2960 if (update_ptrs)
2961 /* Update SW event ring dequeue pointer */
2962 inc_deq(xhci, xhci->event_ring);
2963
2964 /* Are there more items on the event ring? Caller will call us again to
2965 * check.
2966 */
2967 return 1;
2968 }
2969 EXPORT_SYMBOL_GPL(xhci_handle_event);
2970
2971 /*
2972 * Update Event Ring Dequeue Pointer:
2973 * - When all events have finished
2974 * - To avoid "Event Ring Full Error" condition
2975 */
> 2976 void xhci_update_erst_dequeue(struct xhci_hcd *xhci,
2977 union xhci_trb *event_ring_deq)
2978 {
2979 u64 temp_64;
2980 dma_addr_t deq;
2981
2982 temp_64 = xhci_read_64(xhci, &xhci->ir_set->erst_dequeue);
2983 /* If necessary, update the HW's version of the event ring deq ptr. */
2984 if (event_ring_deq != xhci->event_ring->dequeue) {
2985 deq = xhci_trb_virt_to_dma(xhci->event_ring->deq_seg,
2986 xhci->event_ring->dequeue);
2987 if (deq == 0)
2988 xhci_warn(xhci, "WARN something wrong with SW event ring dequeue ptr\n");
2989 /*
2990 * Per 4.9.4, Software writes to the ERDP register shall
2991 * always advance the Event Ring Dequeue Pointer value.
2992 */
2993 if ((temp_64 & (u64) ~ERST_PTR_MASK) ==
2994 ((u64) deq & (u64) ~ERST_PTR_MASK))
2995 return;
2996
2997 /* Update HC event ring dequeue pointer */
2998 temp_64 &= ERST_PTR_MASK;
2999 temp_64 |= ((u64) deq & (u64) ~ERST_PTR_MASK);
3000 }
3001
3002 /* Clear the event handler busy flag (RW1C) */
3003 temp_64 |= ERST_EHB;
3004 xhci_write_64(xhci, temp_64, &xhci->ir_set->erst_dequeue);
3005 }
3006 EXPORT_SYMBOL_GPL(xhci_update_erst_dequeue);
3007
---
0-DAY CI Kernel Test Service
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
Powered by blists - more mailing lists