linux-kernel - Re: [PATCH v8] add support for pci in the pvpanic driver

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: <YGB9Q4bejSnNlxkv@kroah.com>
Date:   Sun, 28 Mar 2021 14:57:39 +0200
From:   Greg KH <gregkh@...uxfoundation.org>
To:     Mihai Carabas <mihai.carabas@...cle.com>
Cc:     linux-kernel@...r.kernel.org, arnd@...db.de,
        bobo.shaobowang@...wei.com, rdunlap@...radead.org
Subject: Re: [PATCH v8] add support for pci in the pvpanic driver

On Wed, Mar 24, 2021 at 04:49:13PM +0200, Mihai Carabas wrote:
> This patchset adds support for PCI in the pvpanic driver. The device already
> got in qemu [1].
> 
> v2:
> - mmio -> MMIO, pci -> PCI suggested by Randy Dunlap.
> - group pvpanic-common.c and mmio.c in the same module. The intention was to
>   have only one module and the common code splitted up to be re-used by the
>   pvpanic-pci module in the next patch.
> - add a new patch where add the licenses for the new files (when I moved the
>   code to mmio.c I preserved the original licenses there)
> - properly clean-up PCI device when driver removed.
> 
> v3:
> - drop patch 3 with licenses and add them when creating files
> - create a new patch (2/3) which allowes multiple pvpanic instances
> 
> v4:
> - fix Makefile in patch 1/3 and 3/3 for pvpanic.o as suggested by Arnd
> 
> v5:
> - rebase on 5.12
> - fix a warning caused by one of the patches
> 
> v6:
> - remove pr_fmt as were not used
> - "remove" functions of type void
> - moved events/capability per-device structure
> 
> v7:
> - rebased on char-misc-next
> 
> v8:
> - fixed build error while compiling only the first patch (accept
>   my apologizes Greg)
> - build and tested pvpanic functionality after each applied patch

Much better, now queued up, thanks for sticking with it.

greg k-h