linux-kernel - Re: PCI: the overdue removal of pci_module

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <4620D325.8050903@student.ltu.se>
Date:	Sat, 14 Apr 2007 15:12:05 +0200
From:	Richard Knutsson <ricknu-0@...dent.ltu.se>
To:	Adrian Bunk <bunk@...sta.de>
CC:	Greg Kroah-Hartman <gregkh@...e.de>, linux-kernel@...r.kernel.org,
	linux-pci@...ey.karlin.mff.cuni.cz
Subject: Re: PCI: the overdue removal of pci_module_init()

Adrian Bunk wrote:
> Unless we finally completely remove it, people will always add new users.
>
> Signed-off-by: Adrian Bunk <bunk@...sta.de>
> Signed-off-by: Greg Kroah-Hartman <gregkh@...e.de>
>
>   
If this is really it, then ACK!
> ---
>  Documentation/feature-removal-schedule.txt |    7 -------
>  Documentation/pci.txt                      |    2 --
>  include/linux/pci.h                        |    6 ------
>  3 files changed, 15 deletions(-)
>
> --- a/Documentation/feature-removal-schedule.txt
> +++ b/Documentation/feature-removal-schedule.txt
> @@ -105,13 +105,6 @@ Who:   Adrian Bunk <bunk@...sta.de>
>  
>  ---------------------------
>  
> -What:	pci_module_init(driver)
> -When:	January 2007
> -Why:	Is replaced by pci_register_driver(pci_driver).
> -Who:	Richard Knutsson <ricknu-0@...dent.ltu.se> and Greg Kroah-Hartman <gregkh@...e.de>
> -
> ----------------------------
> -
>  What:	Usage of invalid timevals in setitimer
>  When:	March 2007
>  Why:	POSIX requires to validate timevals in the setitimer call. This
> --- a/Documentation/pci.txt
> +++ b/Documentation/pci.txt
> @@ -545,8 +545,6 @@ pci_find_slot()			Find pci_dev correspon
>  pci_set_power_state()		Set PCI Power Management state (0=D0 ... 3=D3)
>  pci_find_capability()		Find specified capability in device's capability
>  				list.
> -pci_module_init()		Inline helper function for ensuring correct
> -				pci_driver initialization and error handling.
>  pci_resource_start()		Returns bus start address for a given PCI region
>  pci_resource_end()		Returns bus end address for a given PCI region
>  pci_resource_len()		Returns the byte length of a PCI region
> --- a/include/linux/pci.h
> +++ b/include/linux/pci.h
> @@ -405,12 +405,6 @@ struct pci_driver {
>  	.vendor = PCI_ANY_ID, .device = PCI_ANY_ID, \
>  	.subvendor = PCI_ANY_ID, .subdevice = PCI_ANY_ID
>  
> -/*
> - * pci_module_init is obsolete, this stays here till we fix up all usages of it
> - * in the tree.
> - */
> -#define pci_module_init	pci_register_driver
> -
>  /**
>   * PCI_VDEVICE - macro used to describe a specific pci device in short form
>   * @vend: the vendor name
>
>
> Patches currently in gregkh-2.6 which might be from bunk@...sta.de are
>
> bad/sysfs-crash-debugging.patch
> driver/the-overdue-removal-of-the-mount-umount-uevents.patch
> pci/pci-fix-multiple-definition-of-queue_pushbutton_work.patch
> pci/pci-remove-the-broken-pci_multithread_probe-option.patch
> pci/pci-define-pci_request-release_regions-for-config_pci-n.patch
> pci/pci-the-overdue-removal-of-pci_module_init.patch
> usb/usb-gtco.c-fix-a-use-before-check.patch
> usb/usb-iowarrior.c-fix-null-dereference.patch
>   

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/