linux-kernel - Re: [PATCH v5 0/4] iMC SMBUS and DIMM bus probing

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: <CALCETrX4-WHrCumnOxEoa53-2otNOPBFk_Q6yL3ufyhWbaJ1wQ@mail.gmail.com>
Date:	Fri, 8 Nov 2013 12:41:38 -0800
From:	Andy Lutomirski <luto@...capital.net>
To:	Wolfram Sang <wsa@...-dreams.de>, linux-i2c@...r.kernel.org
Cc:	Jean Delvare <khali@...ux-fr.org>,
	Guenter Roeck <linux@...ck-us.net>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Mauro Carvalho Chehab <m.chehab@...sung.com>,
	Rui Wang <ruiv.wang@...il.com>,
	Andy Lutomirski <luto@...capital.net>
Subject: Re: [PATCH v5 0/4] iMC SMBUS and DIMM bus probing

On Fri, Sep 27, 2013 at 6:02 PM, Andy Lutomirski <luto@...capital.net> wrote:
> Intel LGA2011 machines have dedicated SMBUS controllers for DIMM
> sockets.  Because they're dedicated, they can be safely and accurately
> probed, since all devices on them are known to be attached to DIMMs.
> The devices found are:
>  - SPD EEPROMs
>  - TSODs (Temperature Sensor on DIMMs)
>  - Other interesting things, with drivers hopefully to follow...
>
> This patch series adds a simple generic layer for probing for DIMMs over
> SMBUS and an i2c bus driver for the iMC controller found on Intel
> LGA2011 chips.
>
> It now uses only modern infrastructure -- new-style I2C probing and the
> at24 (instead of eeprom) driver.
>
> I've tested this on a Core i7 Extreme and on a Xeon E5 server.
>
> Patches 1-3 are useful even without patch 4.  I'm still hoping for feedback
> on patch 4.
>
> Changes from v4:
>  - Added the sb_edac changes -- i2c_imc and sb_edac can now coexist
>  - Added some paranoid race detection.
>  - The driver now confirms its ability to claim software control of the SMBUS
>    master.  This prevents unpleasant problems on systems that enable CLTT
>    (closed loop thermal throttling).
>  - Reordered the patches so that the DIMM bus code is last.
>
> Changes from v3:
>  - Dropped redundant "tsod" driver
>  - Dropped eeprom modalias
>  - Switched to probing for the "eeprom" and "jc42"
>
> Andy Lutomirski (4):
>   Move Intel SNB device ids from sb_edac to pci_ids.h
>   sb_edac: Claim a different PCI device
>   i2c_imc: New driver for Intel's iMC, found on LGA2011 chips
>   i2c, i2c_imc: Add DIMM bus code
>
>  drivers/edac/sb_edac.c        |  32 +--
>  drivers/i2c/busses/Kconfig    |  19 ++
>  drivers/i2c/busses/Makefile   |   5 +
>  drivers/i2c/busses/dimm-bus.c |  97 ++++++++
>  drivers/i2c/busses/i2c-imc.c  | 562 ++++++++++++++++++++++++++++++++++++++++++
>  include/linux/i2c/dimm-bus.h  |  24 ++
>  include/linux/pci_ids.h       |  15 ++
>  7 files changed, 723 insertions(+), 31 deletions(-)
>  create mode 100644 drivers/i2c/busses/dimm-bus.c
>  create mode 100644 drivers/i2c/busses/i2c-imc.c
>  create mode 100644 include/linux/i2c/dimm-bus.h

Any update on these?  It would be nice to get them reviewed and applied.

Thanks,
Andy
--
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/