linux-kernel - [PATCH v2 0/8] module params: few simplifications

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

Date:	Wed, 22 Apr 2015 14:55:03 -0700
From:	"Luis R. Rodriguez" <mcgrof@...not-panic.com>
To:	rusty@...tcorp.com.au
Cc:	akpm@...ux-foundation.org, mingo@...nel.org, tj@...nel.org,
	linux-kernel@...r.kernel.org, linux-wireless@...r.kernel.org,
	keescook@...omium.org, casey@...aufler-ca.com,
	cocci@...teme.lip6.fr, "Luis R. Rodriguez" <mcgrof@...e.com>
Subject: [PATCH v2 0/8] module params: few simplifications

From: "Luis R. Rodriguez" <mcgrof@...e.com>

This v2 splits out the #ifdef'ery eyesores out into separate
patches to make it clear this is a separate atomic operation
which requires review in and of itself. It also makes some
clarifications about how the module_param_config_*() can help
with eyesores, and also documents our paranoia over avoiding
bool_enable_only for now on CONFIG_MODULE_SIG_FORCE kernels.

It could be that we are comfortable with bool_enable_only's
implementation even for CONFIG_MODULE_SIG_FORCE kernels -- but
I'd rather let someone dealing with policy decide that.

Luis R. Rodriguez (8):
  kernel/params: constify struct kernel_param_ops uses
  kernel/module.c: use generic module param operaters for sig_enforce
  kernel/params.c: generalize bool_enable_only
  moduleparam.h: add module_param_config_*() helpers
  kernel/workqueue.c: remove ifdefs over wq_power_efficient
  kernel/workqueue.c: use module_param_config_on_off() for
    power_efficient
  kernel/module.c: avoid ifdefs for sig_enforce declaration
  kernel/module.c: use module_param_config_on() for sig_enforce

 arch/s390/kernel/perf_cpum_sf.c         |  2 +-
 arch/x86/kvm/mmu_audit.c                |  2 +-
 arch/x86/platform/uv/uv_nmi.c           |  2 +-
 drivers/block/null_blk.c                |  4 +-
 drivers/char/ipmi/ipmi_watchdog.c       |  6 +--
 drivers/dma/dmatest.c                   |  4 +-
 drivers/ide/ide.c                       |  2 +-
 drivers/infiniband/ulp/srp/ib_srp.c     |  4 +-
 drivers/input/misc/ati_remote2.c        |  4 +-
 drivers/input/mouse/psmouse-base.c      |  2 +-
 drivers/misc/lis3lv02d/lis3lv02d.c      |  2 +-
 drivers/mtd/ubi/block.c                 |  2 +-
 drivers/net/wireless/ath/wil6210/main.c |  4 +-
 drivers/power/test_power.c              | 16 ++++----
 drivers/thermal/intel_powerclamp.c      |  4 +-
 drivers/tty/hvc/hvc_iucv.c              |  2 +-
 drivers/tty/sysrq.c                     |  2 +-
 drivers/video/fbdev/uvesafb.c           |  2 +-
 drivers/virtio/virtio_mmio.c            |  2 +-
 fs/nfs/super.c                          |  2 +-
 include/linux/moduleparam.h             | 73 ++++++++++++++++++++++++++-------
 kernel/module.c                         | 40 ++----------------
 kernel/params.c                         | 44 ++++++++++++++++----
 kernel/workqueue.c                      |  8 +---
 net/sunrpc/auth.c                       |  2 +-
 net/sunrpc/xprtsock.c                   |  6 +--
 security/apparmor/lsm.c                 |  6 +--
 security/integrity/ima/ima_crypto.c     |  2 +-
 sound/pci/hda/hda_intel.c               |  2 +-
 29 files changed, 143 insertions(+), 110 deletions(-)

-- 
2.3.2.209.gd67f9d5.dirty

--
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/