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
| ||
|
Message-ID: <47130F16.3000706@hhs.nl> Date: Mon, 15 Oct 2007 08:56:22 +0200 From: Hans de Goede <j.w.r.degoede@....nl> To: Jim Cromie <jim.cromie@...il.com> CC: Linux kernel <linux-kernel@...r.kernel.org>, LM Sensors <lm-sensors@...sensors.org> Subject: Re: [lm-sensors] [ patch .24-rc0 0/5 ] SuperIO locks coordinator Jim Cromie wrote: > this patchset (on hwmon-git) re-introduces superio_locks module, > previously RFC'd here, where I 'borrowed' another thread.. > > http://marc.info/?l=linux-kernel&m=115821759424601&w=2 > > The module shares out slots/shared-reservations containing > a mutex, so that multiple modules can coordinate access to > the sio-port. > > Im crossposting - LKML for more reviewers, lm-sensors for > folks with the hardware and (perhaps) more interest. > > If its not too late, please consider for 2.6.24-rc0 > > 01 - adds superio_locks module > > User-drivers specify the sio-port characteristics they can support > device-ids, sio-port-addrs, enter & exit sequences, etc in > a struct superio_search (in __devinit, preferably). > > superio_find() then searches existing slots/shared-reservations > for a matching sio-port, and returns it if found. > Otherwize it probes port-addrs, specified by find() user, > and makes and returns a new reservation. > > superio_find() finds and reserves the slot, > returned as ptr or null > superio_release() relinguishes the slot (ref-counted) > > Once theyve got the reservation in struct superio * gate > (as named in patches 2-5) they *may* use > > superio_lock(gate) > superio_enter/exit(gate) > superio_inb/w(gate, regaddr), > superio_outb/w(gate, regaddr, val) > > or they can do it themselves with inb/outb, by using gate->sioaddr, etc. > <snip> > > - Hans, I think Ive added all your suggestions, thanks. > Your welcome, at a first look over things look pretty sane now. I'm willing to do a full review of this, but first I would like to see some more input from others. Regards, Hans - 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/
Powered by blists - more mailing lists