From: "Bharadiya,Pankaj" <pankaj.laxminarayan.bharadiya@intel.com>
To: kernel-hardening@lists.openwall.com, keescook@chromium.org,
akpm@linux-foundation.org, mayhs11saini@gmail.com
Subject: Re: [PATCH 0/5] Add and use sizeof_member macro to bring uniformity
Date: Tue, 24 Sep 2019 18:30:46 +0530 [thread overview]
Message-ID: <20190924130046.GA112306@pktinlab> (raw)
In-Reply-To: <20190924081439.15219-1-pankaj.laxminarayan.bharadiya@intel.com>
I accidentally sent this patch series multiple times.
Please ignore this series. Sorry for spamming your mailbox :(.
Thanks,
Pankaj
On Tue, Sep 24, 2019 at 01:44:34PM +0530, Pankaj Bharadiya wrote:
> This topic has been discussed on the kernel hardening mailing list [1]
> few months back. Thanks to Shyam for initiating work on this.
>
> Its been concluded on how the patch series should be but there was
> no further progress and this work is still pending (Pardon me if I
> missed something here).
>
> At present, we have 3 different macros which serve the same purpose
> of finding the size of a member of a structure:
> - SIZEOF_FIELD
> - FIELD_SIZEOF
> - sizeof_field
>
> To bring uniformity in entire kernel source tree, this patch series -
> - adds the new sizeof_member macro
> - does scripted replacement of above 3 macros where used with newly
> introduced sizeof_member macro
> - removes definitions of SIZEOF_FIELD, FIELD_SIZEOF, sizeof_field
>
> This series aims to fix the issue based on given comments in [1].
>
> [1] Link: https://www.openwall.com/lists/kernel-hardening/2019/07/02/2
>
> Pankaj Bharadiya (5):
> linux/kernel.h: Add sizeof_member macro
> treewide: Use sizeof_member macro
> MIPS: OCTEON: use sizeof_member macro instead of SIZEOF_FIELD
> linux/kernel.h: Remove FIELD_SIZEOF macro
> stddef.h: Remove sizeof_field macro
>
> Documentation/process/coding-style.rst | 2 +-
> .../it_IT/process/coding-style.rst | 2 +-
> .../zh_CN/process/coding-style.rst | 2 +-
> arch/arc/kernel/unwind.c | 6 +-
> arch/arm64/include/asm/processor.h | 10 +-
> .../cavium-octeon/executive/cvmx-bootmem.c | 9 +-
> arch/powerpc/net/bpf_jit32.h | 4 +-
> arch/powerpc/net/bpf_jit_comp.c | 16 +-
> arch/sparc/net/bpf_jit_comp_32.c | 8 +-
> arch/x86/kernel/fpu/xstate.c | 2 +-
> block/blk-core.c | 4 +-
> crypto/adiantum.c | 4 +-
> drivers/firmware/efi/efi.c | 2 +-
> drivers/gpu/drm/i915/gvt/scheduler.c | 2 +-
> drivers/infiniband/hw/hfi1/sdma.c | 2 +-
> drivers/infiniband/hw/hfi1/verbs.h | 4 +-
> .../ulp/opa_vnic/opa_vnic_ethtool.c | 2 +-
> drivers/input/keyboard/applespi.c | 2 +-
> drivers/md/raid5-ppl.c | 2 +-
> drivers/media/platform/omap3isp/isppreview.c | 24 +--
> drivers/net/ethernet/amd/xgbe/xgbe-ethtool.c | 4 +-
> .../ethernet/cavium/liquidio/octeon_console.c | 16 +-
> .../net/ethernet/emulex/benet/be_ethtool.c | 2 +-
> .../ethernet/hisilicon/hns3/hns3pf/hclge_tm.c | 2 +-
> .../net/ethernet/huawei/hinic/hinic_ethtool.c | 8 +-
> .../net/ethernet/intel/fm10k/fm10k_ethtool.c | 2 +-
> .../net/ethernet/intel/i40e/i40e_ethtool.c | 2 +-
> .../net/ethernet/intel/i40e/i40e_lan_hmc.c | 2 +-
> .../net/ethernet/intel/iavf/iavf_ethtool.c | 2 +-
> drivers/net/ethernet/intel/ice/ice_ethtool.c | 10 +-
> .../net/ethernet/intel/ice/ice_lan_tx_rx.h | 2 +-
> drivers/net/ethernet/intel/igb/igb_ethtool.c | 4 +-
> drivers/net/ethernet/intel/igc/igc_ethtool.c | 4 +-
> .../net/ethernet/intel/ixgb/ixgb_ethtool.c | 4 +-
> drivers/net/ethernet/intel/ixgbevf/ethtool.c | 4 +-
> drivers/net/ethernet/marvell/mv643xx_eth.c | 4 +-
> .../net/ethernet/mellanox/mlx4/en_ethtool.c | 2 +-
> .../ethernet/mellanox/mlx5/core/fpga/ipsec.c | 6 +-
> .../net/ethernet/mellanox/mlx5/core/fs_core.c | 4 +-
> .../ethernet/mellanox/mlxsw/spectrum_fid.c | 4 +-
> .../ethernet/mellanox/mlxsw/spectrum_ptp.c | 2 +-
> drivers/net/ethernet/netronome/nfp/bpf/jit.c | 10 +-
> drivers/net/ethernet/netronome/nfp/bpf/main.c | 2 +-
> .../net/ethernet/netronome/nfp/bpf/offload.c | 2 +-
> .../net/ethernet/netronome/nfp/flower/main.h | 2 +-
> .../oki-semi/pch_gbe/pch_gbe_ethtool.c | 2 +-
> drivers/net/ethernet/qlogic/qede/qede.h | 2 +-
> .../ethernet/qlogic/qlcnic/qlcnic_ethtool.c | 2 +-
> .../net/ethernet/qlogic/qlge/qlge_ethtool.c | 2 +-
> .../ethernet/samsung/sxgbe/sxgbe_ethtool.c | 2 +-
> .../ethernet/stmicro/stmmac/stmmac_ethtool.c | 4 +-
> drivers/net/ethernet/ti/cpsw_ethtool.c | 6 +-
> drivers/net/ethernet/ti/netcp_ethss.c | 32 ++--
> drivers/net/fjes/fjes_ethtool.c | 2 +-
> drivers/net/geneve.c | 2 +-
> drivers/net/hyperv/netvsc_drv.c | 2 +-
> drivers/net/usb/sierra_net.c | 2 +-
> drivers/net/usb/usbnet.c | 2 +-
> drivers/net/vxlan.c | 4 +-
> .../net/wireless/marvell/libertas/debugfs.c | 2 +-
> drivers/net/wireless/marvell/mwifiex/util.h | 4 +-
> drivers/s390/net/qeth_core_mpc.h | 10 +-
> drivers/scsi/aacraid/aachba.c | 4 +-
> drivers/scsi/be2iscsi/be_cmds.h | 2 +-
> drivers/scsi/cxgbi/libcxgbi.c | 2 +-
> drivers/scsi/smartpqi/smartpqi_init.c | 6 +-
> .../staging/media/davinci_vpfe/dm365_ipipe.c | 36 ++---
> drivers/target/iscsi/cxgbit/cxgbit_main.c | 2 +-
> drivers/usb/atm/usbatm.c | 2 +-
> drivers/usb/gadget/function/f_fs.c | 2 +-
> fs/befs/linuxvfs.c | 2 +-
> fs/ext2/super.c | 2 +-
> fs/ext4/super.c | 2 +-
> fs/freevxfs/vxfs_super.c | 2 +-
> fs/orangefs/super.c | 2 +-
> fs/ufs/super.c | 2 +-
> include/linux/filter.h | 12 +-
> include/linux/kernel.h | 12 +-
> include/linux/kvm_host.h | 2 +-
> include/linux/phy_led_triggers.h | 2 +-
> include/linux/slab.h | 2 +-
> include/linux/stddef.h | 10 +-
> include/net/garp.h | 2 +-
> include/net/ip_tunnels.h | 6 +-
> include/net/mrp.h | 2 +-
> include/net/netfilter/nf_conntrack_helper.h | 2 +-
> include/net/netfilter/nf_tables_core.h | 2 +-
> include/net/sock.h | 2 +-
> ipc/util.c | 2 +-
> kernel/bpf/cgroup.c | 2 +-
> kernel/bpf/local_storage.c | 4 +-
> kernel/fork.c | 2 +-
> kernel/signal.c | 12 +-
> kernel/utsname.c | 2 +-
> net/802/mrp.c | 6 +-
> net/batman-adv/main.c | 2 +-
> net/bpf/test_run.c | 4 +-
> net/bridge/br.c | 2 +-
> net/caif/caif_socket.c | 2 +-
> net/core/dev.c | 2 +-
> net/core/filter.c | 140 +++++++++---------
> net/core/flow_dissector.c | 10 +-
> net/core/skbuff.c | 2 +-
> net/core/xdp.c | 4 +-
> net/dccp/proto.c | 2 +-
> net/ipv4/ip_gre.c | 4 +-
> net/ipv4/ip_vti.c | 4 +-
> net/ipv4/raw.c | 2 +-
> net/ipv4/tcp.c | 2 +-
> net/ipv6/ip6_gre.c | 4 +-
> net/ipv6/raw.c | 2 +-
> net/iucv/af_iucv.c | 2 +-
> net/netfilter/nf_tables_api.c | 4 +-
> net/netfilter/nfnetlink_cthelper.c | 2 +-
> net/netfilter/nft_ct.c | 12 +-
> net/netfilter/nft_masq.c | 2 +-
> net/netfilter/nft_nat.c | 6 +-
> net/netfilter/nft_redir.c | 2 +-
> net/netfilter/nft_tproxy.c | 4 +-
> net/netfilter/xt_RATEEST.c | 2 +-
> net/netlink/af_netlink.c | 2 +-
> net/openvswitch/datapath.c | 2 +-
> net/openvswitch/flow.h | 4 +-
> net/rxrpc/af_rxrpc.c | 2 +-
> net/sched/act_ct.c | 4 +-
> net/sched/cls_flower.c | 2 +-
> net/sctp/socket.c | 4 +-
> net/unix/af_unix.c | 2 +-
> security/integrity/ima/ima_policy.c | 4 +-
> sound/soc/codecs/hdmi-codec.c | 2 +-
> virt/kvm/kvm_main.c | 2 +-
> 131 files changed, 345 insertions(+), 360 deletions(-)
>
> --
> 2.17.1
>
next prev parent reply other threads:[~2019-09-24 13:07 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-24 8:14 [PATCH 0/5] Add and use sizeof_member macro to bring uniformity Pankaj Bharadiya
2019-09-24 8:14 ` [PATCH 1/5] linux/kernel.h: Add sizeof_member macro Pankaj Bharadiya
2019-09-24 8:14 ` [PATCH 2/5] treewide: Use " Pankaj Bharadiya
2019-09-24 8:14 ` [PATCH 3/5] MIPS: OCTEON: use sizeof_member macro instead of SIZEOF_FIELD Pankaj Bharadiya
2019-09-24 8:14 ` [PATCH 4/5] linux/kernel.h: Remove FIELD_SIZEOF macro Pankaj Bharadiya
2019-09-24 8:14 ` [PATCH 5/5] stddef.h: Remove sizeof_field macro Pankaj Bharadiya
2019-09-24 13:00 ` Bharadiya,Pankaj [this message]
-- strict thread matches above, loose matches on Subject: below --
2019-09-24 10:51 [PATCH 0/5] Add and use sizeof_member macro to bring uniformity Pankaj Bharadiya
2019-09-24 13:02 ` Bharadiya,Pankaj
2019-09-24 10:58 Pankaj Bharadiya
2019-09-24 16:28 ` Kees Cook
2019-09-24 17:07 ` Kees Cook
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190924130046.GA112306@pktinlab \
--to=pankaj.laxminarayan.bharadiya@intel.com \
--cc=akpm@linux-foundation.org \
--cc=keescook@chromium.org \
--cc=kernel-hardening@lists.openwall.com \
--cc=mayhs11saini@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.