linux-kernel - Re: [PATCH v10 3/7] edac,soc: thunderx: Add wrapper for EDAC LMC PCI device

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20171004103426.g6ivm6w7dfhcoluh@pd.tnic>
Date:   Wed, 4 Oct 2017 12:34:26 +0200
From:   Borislav Petkov <bp@...en8.de>
To:     Jan Glauber <jan.glauber@...iumnetworks.com>
Cc:     Mark Rutland <mark.rutland@....com>,
        Will Deacon <will.deacon@....com>,
        linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
        Suzuki K Poulose <Suzuki.Poulose@....com>,
        David Daney <david.daney@...ium.com>,
        Zhangshaokun <zhangshaokun@...ilicon.com>
Subject: Re: [PATCH v10 3/7] edac,soc: thunderx: Add wrapper for EDAC LMC PCI
 device

On Mon, Oct 02, 2017 at 05:17:56PM +0200, Jan Glauber wrote:
> I went for this as the simplest solution, the probing is completely
> synchronous and no state needs to be stored in the wrapper.

What state would you need to store? The wrapper simply gives out the
struct pci_dev * to the callers or NULL if not present.

> 1. What will trigger probing the edac (or perf) driver part?
>    Right now the trigger is the PCI device ID. If the wrapper
>    does not call into edac how should we load the ThunderX edac/perf drivers?
>    The only option I see is a initcall in edac/perf to look for their devices.

The wrapper loads on the PCI dev ID. EDAC loads later and calls the
wrapper function to get the struct pci_dev *. Simple.

> 2. The probe & register is _very_ specific to perf/edac and very different.
>    The only part that would fit in the wrapper is pci_enable_device().
>    So is that what you have in mind?

No, see above. Instead of getting the PCI device IDs from the PCI core,
you use the wrapper, which gets those from the PCI core. Thus it is
called a "wrapper". :)

-- 
Regards/Gruss,
    Boris.

Good mailing practices for 400: avoid top-posting and trim the reply.