User Tools

Site Tools


lkmp:lkmp_task_list

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revision Previous revision
Next revision
Previous revision
lkmp:lkmp_task_list [2019/06/15 19:23]
ShuahKhanLF
lkmp:lkmp_task_list [2020/01/07 20:05] (current)
BjornHelgaas
Line 1: Line 1:
 ====== Linux Kernel Mentorship Task List ====== ====== Linux Kernel Mentorship Task List ======
  
-==== hrtimer_forward_now() return value is u64 ====+==== Documentation conversion and updates ​====
  
-Callers of hrtimer_forward_now() should save the return value in u64A few callers don'tFix them.+The following directories have several ​.txt filesPlease pick two files and convert to ReSTThere are a few exceptions that should be kept as plain text files:
  
-  ​^   ^ +  ​* a configuration file whose extension is .txt; 
-| **Name** ​ **Description** | **File** | **Status** | +  * a dump of chapter extracted from an old arch-specific manual which seems to be provided by its manufacturer.
-| cx88_ir_work() | missed is unsigned long | drivers/​media/​pci/​cx88/​cx88-input.c | Open | +
-| rtc_pie_update_irq() | count is int | drivers/​rtc/​interface.c | Open | +
-| sched_cfs_period_timer() | overrun is int | kernel/​sched/​fair.c | Open | +
-| sched_rt_period_timer() | overrun is int | kernel/​sched/​rt.c | Open |+
  
-==== Cleanup cppcheck errors ====+**Preparing to build the documentation:​**
  
-**Shifting signed 32-bit value by 31 bits is undefined behaviour errors**+There'​s a script that checks if you have all needed dependencies to build the documentation. It is called automatically when you run ''​make htmldocs''​. Alternatively,​ you can call the script directly with:
  
-These errors aren't real problems for us as compiler takes care of theseHowever, cleaning these up doesn'​t hurt, especially in public API headers. Some maintainers might like drivers fixed, on the other hand fixing public API headers first is a good idea. There are two ways to fix this:+<​code>​ 
 +./​scripts/​sphinx-pre-install  
 +</​code>​
  
-  ​Change #define FOO    (1 << 31) to #define FOO    BIT(31) or +**Building docs and looking for warnings:**
-  ​Adding a "​U"​ cast - changing (1 << 31) to (1U << 31)+
  
-cd git_repo_dir+Once you have all the requirements,​ you can do the building with:
  
-cppcheck --force ./ cppcheck.out 2>&1+<code> 
 +make htmldocs > doc_make.log 2>&1 
 +</​code>​
  
-You can run cppcheck on one or more files by passing a list of files: +And check the warnings with the following script.
-cppcheck --file-list=files+
  
-Grep for "​Shifting signed 32-bit value by 31 bits is undefined behaviour"​ errors+<​code>​ 
 +#​!/​bin/​bash 
 +
 +# SPDX-License-Identifier:​ GPL-2.0 
 +
 +# Copyright(c) Shuah Khan <​skhan@linuxfoundation.org>​ 
 +# License: GPLv2 
 +
 +# Generates wiki format table of task list from Document 
 +# build log 
 +# Usage: gen_doc_task_list.sh <​doc_build_log_file>​
  
-You can refer to the one I generated on [[https://drive.google.com/file/d/19Xu7UqBGJ7BpzxEp92ZQYb6F8UPrk3z3/view?​usp=sharing|Linux 5.2-rc3 cppcheck 32-bit log]]+# Print Table header 
 +echo "| **Task** | **Description** | **Owner** | **Status** | **Commit ID** |" 
 +for f in `egrep "​warn|WARN|error|ERROR"​ $1 | \ 
 +        egrep ".c:|.h:|.rst:" | \ 
 +        awk -F: '​{print $1}' | awk '​!a[$0]++'​ | \ 
 +        sed -e "s/^.\///" ​sed -e "​s/​^\/​mnt\/​data\/​lkml\/​linux_5.3\///"​` 
 +do 
 +        echo "| $f | Fix warns | | | |" 
 +done 
 +</​code>​
  
-[[https://​drive.google.com/​file/​d/​1EbBsyLNh9nQuB_z4JbkyxDsXqItSpNo4/​view?​usp=sharing|Linux 5.2-rc3 cppcheck full log]]+==== Documentation build warn tasks (generated on Linux 5.3.0) ====
  
-**Defines ​in public headers ​include/** +**Before starting work on a task, please check the archive first to see if a patch is already ​in progress. Please avoid duplicated efforts. [[https://​lists.linuxfoundation.org/​pipermail/​linux-kernel-mentees/|Liunx Kernel Mentees Archive]]. Send email to the list informing others that you are working on the task.**
-^   ​^ ​  ^ +
-| **File** ​ | **Define** | **Status** |+
  
-uapi/linux/cramfs_fs.h | CRAMFS_BLK_FLAG_UNCOMPRESSED ​Open +  
-uapi/linux/ethtool.h | RXH_DISCARD ​Open +**Task** | **Description** | **Owner** | **Status** | **Commit ID** | 
-uapi/linux/if_packet.h | TP_STATUS_TS_RAW_HARDWARE ​Open +| lib/​genalloc.c | Fix warns | | | | 
-uapi/linux/kfd_ioctl.h | KFD_IOC_ALLOC_MEM_FLAGS_WRITABLE ​Open +| include/linux/i2c.h | Fix warns | | | | 
-uapi/linux/media.h | MEDIA_ID_FLAG_NEXT ​[[https://lore.kernel.org/linux-media/20190613005620.7362-1-skhan@linuxfoundation.org/T/#u|Patch Sent - use as reference]] ​+include/linux/regulator/​machine.h | Fix warns | | | | 
-uapi/linux/nl80211.h | NL80211_FEATURE_ND_RANDOM_MAC_ADDR ​Open +include/linux/regulator/​driver.h | Fix warns | | | | 
-uapi/linux/vfio.h | VFIO_REGION_TYPE_PCI_VENDOR_TYPE ​Open +include/net/cfg80211.h | Fix warns | | | | 
-linux/videodev2.v4l2_fourcc_be ​[[https://lore.kernel.org/linux-media/20190613005652.7423-1-skhan@linuxfoundation.org/T/#u|Patch sent - use as reference]] ​+include/linux/input/​sparse-keymap.h | Fix warns | | | 
-uapi/drm/exynos_drm.h| G2D_BUF_USERPTR ​Open +| mm/util.c | Fix warns | | | | 
-uapi/rdma/vmw_pvrdma-abi.PVRDMA_UAR_QP_RECV ​Open +| mm/slab.c | Fix warns | | | | 
-uapi/rdma/vmw_pvrdma-abi.PVRDMA_UAR_CQ_POLL ​Open +| include/linux/spi/spi.h | Fix warns | | | | 
-linux/ata.ATA_PRD_EOT ​Open +| drivers/​usb/​typec/​bus.c | Fix warns | | | | 
-linux/cs5535.h | CS5536_GPIOM7_PME_FLAG ​Open +| drivers/usb/typec/​class.c | Fix warns | | | | 
-linux/cs5535.h | CS5536_GPIOM7_PME_EN ​Open +include/net/mac80211.h | Fix warns | | | | 
-linux/ide.h | IDE_DFLAG_NIEN_QUIRK ​Open +include/linux/w1.h | Fix warns | | | | 
-linux/ide.IDE_HFLAG_NO_UNMASK_IRQS ​Open +drivers/​gpu/​drm/​amd/​amdgpu/amdgpu_mn.Fix warns | | | | 
-| linux/pxa2xx_ssp.h | SSCR0_MOD ​Open +drivers/gpu/drm/​amd/​amdgpu/​amdgpu_vm.c | Fix warns | | | | 
-linux/pxa2xx_ssp.h | SSCR1_TTELP ​Open |  +| drivers/​gpu/​drm/​amd/​amdgpu/​amdgpu_irq.c | Fix warns | | | | 
-| linux/rtsx_pci.h | TRIG_DMA ​Open +| drivers/gpu/drm/​amd/​amdgpu/​amdgpu_xgmi.c | Fix warns | | | | 
-| linux/rtsx_pci.h | HAIMR_TRANS_START ​Open +| drivers/​gpu/​drm/​amd/​amdgpu/​amdgpu_ras.c | Fix warns | | | | 
-| linux/rtsx_pci.h | CMD_DONE_INT ​Open +| drivers/gpu/drm/​amd/​amdgpu/​amdgpu_pm.c | Fix warns | | | | 
-| linux/rtsx_pci.h | CMD_DONE_INT_EN ​Open +drivers/gpu/drm/amd/​display/​amdgpu_dm/​amdgpu_dm.h | Fix warns | | | | 
-linux/​serial_core.UPF_IOREMAP ​Open +fs/direct-io.c | Fix warns | | | | 
-media/dvb_frontend.DVBFE_ALGO_RECOVERY ​Open +fs/libfs.c | Fix warns | | | | 
-media/dvb_frontend.DVBFE_ALGO_SEARCH_ERROR ​Open +| drivers/gpu/​drm/​amd/​display/​amdgpu_dm/​amdgpu_dm.c | Fix warns | | | | 
-net/dst.DST_FEATURE_ECN_CA ​   (1 << 31) Open +fs/posix_acl.c | Fix warns | | | | 
-net/gue.GUE_PFLAG_REMCSUM ​Open +include/drm/drm_drv.h | Fix warns | | | | 
-rdma/ib_verbs.IB_QP_CREATE_RESERVED_END ​Open +include/drm/drm_modeset_helper_vtables.h | Fix warns | | | | 
-rdma/ib_verbs.h | IB_SEND_RESERVED_END ​Open +include/drm/drm_atomic_state_helper.h | Fix warns | | | | 
-video/atmel_lcdc.ATMEL_LCDC_MEMOR ​Open +drivers/gpu/​drm/​mcde/​mcde_drv.c | Fix warns | | | | 
-video/atmel_lcdc.ATMEL_LCDC_MEMOR_LITTLE ​Open +include/linux/skbuff.h | Fix warns | | | | 
-video/pm3fb.h | PM3PixelSize_INDIVIDUAL ​Open +include/net/sock.h | Fix warns | | | | 
-video/pm3fb.PM3Render2D_TextureEnable ​Open +include/linux/netdevice.h | Fix warns | | | | 
-video/radeon.RB2D_DC_BUSY ​Open +include/linux/lsm_hooks.h | Fix warns | | | | 
-video/radeon.h | HORZ_AUTO_RATIO_INC ​Open |+include/linux/phylink.h | Fix warns | | | | 
 +drivers/​net/​phy/​phylink.c | Fix warns | | | | 
 +| Documentation/​admin-guide/​xfs.rst | Fix warns | | | | 
 +| include/​uapi/​linux/firewire-cdev.h | Fix warns | | | | 
 +drivers/​firewire/​core-transaction.c | Fix warns | | | | 
 +| drivers/​ata/​libata-core.c | Fix warns | | | | 
 +| drivers/​message/​fusion/​mptbase.c | Fix warns | | | | 
 +| drivers/​tty/​serial/​serial_core.c | Fix warns | | | | 
 +fs/seq_file.c | Fix warns | | | | 
 +fs/debugfs/​inode.c | Fix warns | | | | 
 +fs/debugfs/​file.c | Fix warns | | | | 
 +Documentation/index.rst | Fix warns | | | | 
 +Documentation/​kbuild/makefiles.rst | Fix warns | | | | 
 +include/linux/​xarray.h | Fix warns | | | | 
 +Documentation/crypto/​crypto_engine.rst | Fix warns | | | | 
 +drivers/gpu/​drm/​amd/​amdgpu/​amdgpu_drv.c | Fix warns | | | | 
 +include/drm/​drm_connector.h | Fix warns | | | | 
 +Documentation/trace/​kprobetrace.rst | Fix warns | | | | 
 +net/core/dev.c | Fix warns | | | | 
 +Documentation/security/​keys/​core.rst | Fix warns | | | | 
 +| Documentation/​translations/​it_IT/​process/​maintainer-pgp-guide.rst | Fix warns | | | | 
 +| Documentation/​translations/​it_IT/​process/​programming-language.rst | Fix warns | | | | 
 +| Documentation/​virt/​index.rst | Fix warns | | | | 
 +| include/​linux/​slab.h | Fix warns | | | | 
 +| Documentation/​gpu/​drm-internals.rst | Fix warns | | | |
  
  
-**Uninitialized variable errors/​warns** 
- 
-Grep for Uninitialized variable errors and send fixes in. Make sure the error is real and not a false. Check to see if it is used before initializing. 
  
  
lkmp/lkmp_task_list.1560626594.txt.gz · Last modified: 2019/06/15 19:23 by ShuahKhanLF