From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5C2E9C433FE for ; Fri, 19 Nov 2021 12:47:29 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 2104361AFB for ; Fri, 19 Nov 2021 12:47:29 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 2104361AFB Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 6D06E6F4E4; Fri, 19 Nov 2021 12:47:28 +0000 (UTC) Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by gabe.freedesktop.org (Postfix) with ESMTPS id EF7D56F4E4 for ; Fri, 19 Nov 2021 12:47:27 +0000 (UTC) X-IronPort-AV: E=McAfee;i="6200,9189,10172"; a="215121319" X-IronPort-AV: E=Sophos;i="5.87,247,1631602800"; d="scan'208";a="215121319" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Nov 2021 04:47:27 -0800 X-IronPort-AV: E=Sophos;i="5.87,247,1631602800"; d="scan'208";a="507883791" Received: from sgconnee-mobl1.ger.corp.intel.com (HELO localhost) ([10.252.21.83]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Nov 2021 04:47:15 -0800 From: Jani Nikula To: Alejandro Colomar , LKML In-Reply-To: <20211119113644.1600-1-alx.manpages@gmail.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20211119113644.1600-1-alx.manpages@gmail.com> Date: Fri, 19 Nov 2021 14:47:08 +0200 Message-ID: <87mtm0jos3.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Intel-gfx] [PATCH 00/17] Add memberof(), split some headers, and slightly simplify code X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Alejandro Colomar , Corey Minyard , Ajit Khaparde , "Michael S. Tsirkin" , Jason Wang , Rasmus Villemoes , Bjorn Andersson , Chris Mason , Christian Brauner , Ketan Mukadam , Somnath Kotur , linux-scsi@vger.kernel.org, Subbu Seetharaman , "Rafael J. Wysocki" , Russell King , Mike Rapoport , linux-acpi@vger.kernel.org, Miguel Ojeda , Borislav Petkov , virtualization@lists.linux-foundation.org, Len Brown , Kees Cook , Arnd Bergmann , John Hubbard , intel-gfx@lists.freedesktop.org, Josef Bacik , Jitendra Bhivare , Jonathan Cameron , David Sterba , Andy Shevchenko , linux-arm-kernel@lists.infradead.org, Sriharsha Basavapatna , netdev@vger.kernel.org, Nick Desaulniers , "John S . Gruber" , Andrew Morton , linux-btrfs@vger.kernel.org Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Fri, 19 Nov 2021, Alejandro Colomar wrote: > Hi all, > > I simplified some xxxof() macros, > by adding a new macro memberof(), > which implements a common operation in many of them. > > I also splitted many of those macros into tiny headers, > since I noticed that touching those headers implied > recompiling almost the whole kernel. > > Hopefully after this patch there will be less > things to recompile after touching one of those. > > Having simpler headers means that now one can > include one of those without pulling too much stuff > that might break other stuff. > > I removed some unnecessary casts too. > > Every few commits in this series > and of course after the last commit > I rebuilt the kernel and run for a while with it without any problems. > > Please note that I have written very few kernel code > and for example some files wouldn't let me include some of these files, > so I didn't change those. > > What I mean is that, > even though this is super obvious and shouldn't break stuff, > and I'm not new to C, > I'm quite new to the kernel, > and ask that reviewers take deep look, please. > > > In the first and second commits > I changed a lot of stuff in many parts, > and that's why I CCd so many people (also in this cover letter). > However, to avoid spamming, > and since it would be a nightmare to > find all the relevant people affected in so many different areas, > I only CCd in 01, 02 and in the cover letter. > If anyone is interested in reading the full patch set, > I sent it to the LKML. I think with the patch split you have this would be a nightmare to get merged. Please consider refactoring the headers first, and once those are reviewed and merged, you can proceed with using them elsewhere. For example, we'd want the drm/i915 changes in patches separate from changes to other drivers or the core headers. BR, Jani. > > > Thanks, > Alex > > > Alejandro Colomar (17): > linux/container_of.h: Add memberof(T, m) > Use memberof(T, m) instead of explicit NULL dereference > Replace some uses of memberof() by its wrappers > linux/memberof.h: Move memberof() to separate header > linux/typeof_member.h: Move typeof_member() to a separate header > Simplify sizeof(typeof_member()) to sizeof_field() > linux/NULL.h: Move NULL to a separate header > linux/offsetof.h: Move offsetof(T, m) to a separate header > linux/offsetof.h: Implement offsetof() in terms of memberof() > linux/container_of.h: Implement container_of_safe() in terms of > container_of() > linux/container_of.h: Cosmetic > linux/container_of.h: Remove unnecessary cast to (void *) > linux/sizeof_field.h: Move sizeof_field(T, m) to a separate header > include/linux/: Include a smaller header if just for NULL > linux/offsetofend.h: Move offsetofend(T, m) to a separate header > linux/array_size.h: Move ARRAY_SIZE(arr) to a separate header > include/: Include for ARRAY_SIZE() > > arch/x86/include/asm/bootparam_utils.h | 3 +- > arch/x86/kernel/signal_compat.c | 5 ++-- > drivers/gpu/drm/i915/i915_sw_fence.c | 1 + > drivers/gpu/drm/i915/i915_utils.h | 5 ++-- > drivers/gpu/drm/i915/intel_runtime_pm.h | 3 +- > drivers/net/ethernet/emulex/benet/be.h | 10 +++---- > drivers/net/ethernet/i825xx/ether1.c | 7 +++-- > drivers/platform/x86/wmi.c | 3 +- > drivers/scsi/be2iscsi/be.h | 12 ++++---- > drivers/scsi/be2iscsi/be_cmds.h | 5 +++- > fs/btrfs/ctree.h | 5 ++-- > fs/proc/inode.c | 1 + > include/acpi/actypes.h | 4 ++- > include/crypto/internal/blake2b.h | 1 + > include/crypto/internal/blake2s.h | 1 + > include/crypto/internal/chacha.h | 1 + > include/drm/drm_mipi_dbi.h | 1 + > include/drm/drm_mode_object.h | 1 + > include/kunit/test.h | 1 + > include/linux/NULL.h | 10 +++++++ > include/linux/arm_ffa.h | 1 + > include/linux/array_size.h | 15 ++++++++++ > include/linux/blk_types.h | 1 + > include/linux/can/core.h | 1 + > include/linux/clk-provider.h | 1 + > include/linux/container_of.h | 28 ++++++++++------- > include/linux/counter.h | 1 + > include/linux/crash_core.h | 1 + > include/linux/efi.h | 1 + > include/linux/extable.h | 2 +- > include/linux/f2fs_fs.h | 1 + > include/linux/filter.h | 3 ++ > include/linux/fs.h | 1 + > include/linux/genl_magic_func.h | 1 + > include/linux/hashtable.h | 1 + > include/linux/ieee80211.h | 1 + > include/linux/kbuild.h | 3 ++ > include/linux/kernel.h | 7 +---- > include/linux/kfifo.h | 1 + > include/linux/kvm_host.h | 3 ++ > include/linux/libata.h | 1 + > include/linux/llist.h | 1 + > include/linux/memberof.h | 11 +++++++ > include/linux/mlx5/device.h | 1 + > include/linux/mlx5/driver.h | 1 + > include/linux/mm_types.h | 1 + > include/linux/moduleparam.h | 3 ++ > include/linux/mtd/rawnand.h | 1 + > include/linux/netdevice.h | 1 + > include/linux/netfilter.h | 1 + > include/linux/nvme-fc.h | 2 ++ > include/linux/offsetof.h | 17 +++++++++++ > include/linux/offsetofend.h | 19 ++++++++++++ > include/linux/pagemap.h | 1 + > include/linux/phy.h | 1 + > include/linux/phy_led_triggers.h | 1 + > include/linux/pinctrl/machine.h | 1 + > include/linux/property.h | 1 + > include/linux/rcupdate.h | 1 + > include/linux/rcupdate_wait.h | 1 + > include/linux/regmap.h | 1 + > include/linux/sched/task.h | 1 + > include/linux/sizeof_field.h | 14 +++++++++ > include/linux/skb_array.h | 1 + > include/linux/skbuff.h | 1 + > include/linux/skmsg.h | 3 ++ > include/linux/slab.h | 2 ++ > include/linux/spinlock_types.h | 1 + > include/linux/stddef.h | 30 +++---------------- > include/linux/string.h | 5 +++- > include/linux/surface_aggregator/controller.h | 1 + > include/linux/surface_aggregator/serial_hub.h | 1 + > include/linux/swap.h | 1 + > include/linux/ti-emif-sram.h | 1 + > include/linux/typeof_member.h | 11 +++++++ > include/linux/ucs2_string.h | 2 +- > include/linux/vdpa.h | 1 + > include/linux/virtio_config.h | 17 ++++++----- > include/linux/wireless.h | 2 ++ > include/net/bond_3ad.h | 1 + > include/net/dsa.h | 1 + > include/net/ip_vs.h | 1 + > include/net/netfilter/nf_conntrack_tuple.h | 1 + > include/net/netfilter/nf_tables.h | 1 + > include/net/netlink.h | 1 + > include/rdma/uverbs_ioctl.h | 1 + > include/rdma/uverbs_named_ioctl.h | 1 + > include/scsi/scsi_host.h | 1 + > include/sound/soc-dapm.h | 1 + > include/sound/soc.h | 1 + > include/trace/events/wbt.h | 1 + > include/uapi/linux/netfilter/xt_sctp.h | 1 + > include/xen/hvm.h | 1 + > kernel/kallsyms.c | 3 +- > 94 files changed, 255 insertions(+), 79 deletions(-) > create mode 100644 include/linux/NULL.h > create mode 100644 include/linux/array_size.h > create mode 100644 include/linux/memberof.h > create mode 100644 include/linux/offsetof.h > create mode 100644 include/linux/offsetofend.h > create mode 100644 include/linux/sizeof_field.h > create mode 100644 include/linux/typeof_member.h -- Jani Nikula, Intel Open Source Graphics Center From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 131C4C433EF for ; Fri, 19 Nov 2021 12:47:29 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id ECE3361ACE for ; Fri, 19 Nov 2021 12:47:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231466AbhKSMu3 (ORCPT ); Fri, 19 Nov 2021 07:50:29 -0500 Received: from mga09.intel.com ([134.134.136.24]:14412 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229457AbhKSMu3 (ORCPT ); Fri, 19 Nov 2021 07:50:29 -0500 X-IronPort-AV: E=McAfee;i="6200,9189,10172"; a="234238009" X-IronPort-AV: E=Sophos;i="5.87,247,1631602800"; d="scan'208";a="234238009" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Nov 2021 04:47:27 -0800 X-IronPort-AV: E=Sophos;i="5.87,247,1631602800"; d="scan'208";a="507883791" Received: from sgconnee-mobl1.ger.corp.intel.com (HELO localhost) ([10.252.21.83]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Nov 2021 04:47:15 -0800 From: Jani Nikula To: Alejandro Colomar , LKML Cc: Alejandro Colomar , Ajit Khaparde , Andrew Morton , Andy Shevchenko , Arnd Bergmann , Bjorn Andersson , Borislav Petkov , Corey Minyard , Chris Mason , Christian Brauner , David Sterba , Jason Wang , Jitendra Bhivare , John Hubbard , "John S . Gruber" , Jonathan Cameron , Joonas Lahtinen , Josef Bacik , Kees Cook , Ketan Mukadam , Len Brown , "Michael S. Tsirkin" , Miguel Ojeda , Mike Rapoport , Nick Desaulniers , "Rafael J. Wysocki" , Rasmus Villemoes , Rodrigo Vivi , Russell King , Somnath Kotur , Sriharsha Basavapatna , Subbu Seetharaman , intel-gfx@lists.freedesktop.org, linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-btrfs@vger.kernel.org, linux-scsi@vger.kernel.org, netdev@vger.kernel.org, virtualization@lists.linux-foundation.org Subject: Re: [PATCH 00/17] Add memberof(), split some headers, and slightly simplify code In-Reply-To: <20211119113644.1600-1-alx.manpages@gmail.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20211119113644.1600-1-alx.manpages@gmail.com> Date: Fri, 19 Nov 2021 14:47:08 +0200 Message-ID: <87mtm0jos3.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org On Fri, 19 Nov 2021, Alejandro Colomar wrote: > Hi all, > > I simplified some xxxof() macros, > by adding a new macro memberof(), > which implements a common operation in many of them. > > I also splitted many of those macros into tiny headers, > since I noticed that touching those headers implied > recompiling almost the whole kernel. > > Hopefully after this patch there will be less > things to recompile after touching one of those. > > Having simpler headers means that now one can > include one of those without pulling too much stuff > that might break other stuff. > > I removed some unnecessary casts too. > > Every few commits in this series > and of course after the last commit > I rebuilt the kernel and run for a while with it without any problems. > > Please note that I have written very few kernel code > and for example some files wouldn't let me include some of these files, > so I didn't change those. > > What I mean is that, > even though this is super obvious and shouldn't break stuff, > and I'm not new to C, > I'm quite new to the kernel, > and ask that reviewers take deep look, please. > > > In the first and second commits > I changed a lot of stuff in many parts, > and that's why I CCd so many people (also in this cover letter). > However, to avoid spamming, > and since it would be a nightmare to > find all the relevant people affected in so many different areas, > I only CCd in 01, 02 and in the cover letter. > If anyone is interested in reading the full patch set, > I sent it to the LKML. I think with the patch split you have this would be a nightmare to get merged. Please consider refactoring the headers first, and once those are reviewed and merged, you can proceed with using them elsewhere. For example, we'd want the drm/i915 changes in patches separate from changes to other drivers or the core headers. BR, Jani. > > > Thanks, > Alex > > > Alejandro Colomar (17): > linux/container_of.h: Add memberof(T, m) > Use memberof(T, m) instead of explicit NULL dereference > Replace some uses of memberof() by its wrappers > linux/memberof.h: Move memberof() to separate header > linux/typeof_member.h: Move typeof_member() to a separate header > Simplify sizeof(typeof_member()) to sizeof_field() > linux/NULL.h: Move NULL to a separate header > linux/offsetof.h: Move offsetof(T, m) to a separate header > linux/offsetof.h: Implement offsetof() in terms of memberof() > linux/container_of.h: Implement container_of_safe() in terms of > container_of() > linux/container_of.h: Cosmetic > linux/container_of.h: Remove unnecessary cast to (void *) > linux/sizeof_field.h: Move sizeof_field(T, m) to a separate header > include/linux/: Include a smaller header if just for NULL > linux/offsetofend.h: Move offsetofend(T, m) to a separate header > linux/array_size.h: Move ARRAY_SIZE(arr) to a separate header > include/: Include for ARRAY_SIZE() > > arch/x86/include/asm/bootparam_utils.h | 3 +- > arch/x86/kernel/signal_compat.c | 5 ++-- > drivers/gpu/drm/i915/i915_sw_fence.c | 1 + > drivers/gpu/drm/i915/i915_utils.h | 5 ++-- > drivers/gpu/drm/i915/intel_runtime_pm.h | 3 +- > drivers/net/ethernet/emulex/benet/be.h | 10 +++---- > drivers/net/ethernet/i825xx/ether1.c | 7 +++-- > drivers/platform/x86/wmi.c | 3 +- > drivers/scsi/be2iscsi/be.h | 12 ++++---- > drivers/scsi/be2iscsi/be_cmds.h | 5 +++- > fs/btrfs/ctree.h | 5 ++-- > fs/proc/inode.c | 1 + > include/acpi/actypes.h | 4 ++- > include/crypto/internal/blake2b.h | 1 + > include/crypto/internal/blake2s.h | 1 + > include/crypto/internal/chacha.h | 1 + > include/drm/drm_mipi_dbi.h | 1 + > include/drm/drm_mode_object.h | 1 + > include/kunit/test.h | 1 + > include/linux/NULL.h | 10 +++++++ > include/linux/arm_ffa.h | 1 + > include/linux/array_size.h | 15 ++++++++++ > include/linux/blk_types.h | 1 + > include/linux/can/core.h | 1 + > include/linux/clk-provider.h | 1 + > include/linux/container_of.h | 28 ++++++++++------- > include/linux/counter.h | 1 + > include/linux/crash_core.h | 1 + > include/linux/efi.h | 1 + > include/linux/extable.h | 2 +- > include/linux/f2fs_fs.h | 1 + > include/linux/filter.h | 3 ++ > include/linux/fs.h | 1 + > include/linux/genl_magic_func.h | 1 + > include/linux/hashtable.h | 1 + > include/linux/ieee80211.h | 1 + > include/linux/kbuild.h | 3 ++ > include/linux/kernel.h | 7 +---- > include/linux/kfifo.h | 1 + > include/linux/kvm_host.h | 3 ++ > include/linux/libata.h | 1 + > include/linux/llist.h | 1 + > include/linux/memberof.h | 11 +++++++ > include/linux/mlx5/device.h | 1 + > include/linux/mlx5/driver.h | 1 + > include/linux/mm_types.h | 1 + > include/linux/moduleparam.h | 3 ++ > include/linux/mtd/rawnand.h | 1 + > include/linux/netdevice.h | 1 + > include/linux/netfilter.h | 1 + > include/linux/nvme-fc.h | 2 ++ > include/linux/offsetof.h | 17 +++++++++++ > include/linux/offsetofend.h | 19 ++++++++++++ > include/linux/pagemap.h | 1 + > include/linux/phy.h | 1 + > include/linux/phy_led_triggers.h | 1 + > include/linux/pinctrl/machine.h | 1 + > include/linux/property.h | 1 + > include/linux/rcupdate.h | 1 + > include/linux/rcupdate_wait.h | 1 + > include/linux/regmap.h | 1 + > include/linux/sched/task.h | 1 + > include/linux/sizeof_field.h | 14 +++++++++ > include/linux/skb_array.h | 1 + > include/linux/skbuff.h | 1 + > include/linux/skmsg.h | 3 ++ > include/linux/slab.h | 2 ++ > include/linux/spinlock_types.h | 1 + > include/linux/stddef.h | 30 +++---------------- > include/linux/string.h | 5 +++- > include/linux/surface_aggregator/controller.h | 1 + > include/linux/surface_aggregator/serial_hub.h | 1 + > include/linux/swap.h | 1 + > include/linux/ti-emif-sram.h | 1 + > include/linux/typeof_member.h | 11 +++++++ > include/linux/ucs2_string.h | 2 +- > include/linux/vdpa.h | 1 + > include/linux/virtio_config.h | 17 ++++++----- > include/linux/wireless.h | 2 ++ > include/net/bond_3ad.h | 1 + > include/net/dsa.h | 1 + > include/net/ip_vs.h | 1 + > include/net/netfilter/nf_conntrack_tuple.h | 1 + > include/net/netfilter/nf_tables.h | 1 + > include/net/netlink.h | 1 + > include/rdma/uverbs_ioctl.h | 1 + > include/rdma/uverbs_named_ioctl.h | 1 + > include/scsi/scsi_host.h | 1 + > include/sound/soc-dapm.h | 1 + > include/sound/soc.h | 1 + > include/trace/events/wbt.h | 1 + > include/uapi/linux/netfilter/xt_sctp.h | 1 + > include/xen/hvm.h | 1 + > kernel/kallsyms.c | 3 +- > 94 files changed, 255 insertions(+), 79 deletions(-) > create mode 100644 include/linux/NULL.h > create mode 100644 include/linux/array_size.h > create mode 100644 include/linux/memberof.h > create mode 100644 include/linux/offsetof.h > create mode 100644 include/linux/offsetofend.h > create mode 100644 include/linux/sizeof_field.h > create mode 100644 include/linux/typeof_member.h -- Jani Nikula, Intel Open Source Graphics Center From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4E15FC433EF for ; Fri, 19 Nov 2021 12:47:33 +0000 (UTC) Received: from smtp3.osuosl.org (smtp3.osuosl.org [140.211.166.136]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id EC98761ACE for ; Fri, 19 Nov 2021 12:47:32 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org EC98761ACE Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.linux-foundation.org Received: from localhost (localhost [127.0.0.1]) by smtp3.osuosl.org (Postfix) with ESMTP id 9725360656; Fri, 19 Nov 2021 12:47:32 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp3.osuosl.org ([127.0.0.1]) by localhost (smtp3.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id CazdXb8b-ldY; Fri, 19 Nov 2021 12:47:31 +0000 (UTC) Received: from lists.linuxfoundation.org (lf-lists.osuosl.org [140.211.9.56]) by smtp3.osuosl.org (Postfix) with ESMTPS id E465361BF4; Fri, 19 Nov 2021 12:47:30 +0000 (UTC) Received: from lf-lists.osuosl.org (localhost [127.0.0.1]) by lists.linuxfoundation.org (Postfix) with ESMTP id C631BC001E; Fri, 19 Nov 2021 12:47:30 +0000 (UTC) Received: from smtp3.osuosl.org (smtp3.osuosl.org [IPv6:2605:bc80:3010::136]) by lists.linuxfoundation.org (Postfix) with ESMTP id E961AC0012 for ; Fri, 19 Nov 2021 12:47:28 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp3.osuosl.org (Postfix) with ESMTP id E490761BF4 for ; Fri, 19 Nov 2021 12:47:28 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp3.osuosl.org ([127.0.0.1]) by localhost (smtp3.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id FHR7hNkQRmTA for ; Fri, 19 Nov 2021 12:47:28 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.8.0 Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by smtp3.osuosl.org (Postfix) with ESMTPS id 0A60460656 for ; Fri, 19 Nov 2021 12:47:27 +0000 (UTC) X-IronPort-AV: E=McAfee;i="6200,9189,10172"; a="258199391" X-IronPort-AV: E=Sophos;i="5.87,247,1631602800"; d="scan'208";a="258199391" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Nov 2021 04:47:27 -0800 X-IronPort-AV: E=Sophos;i="5.87,247,1631602800"; d="scan'208";a="507883791" Received: from sgconnee-mobl1.ger.corp.intel.com (HELO localhost) ([10.252.21.83]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Nov 2021 04:47:15 -0800 From: Jani Nikula To: Alejandro Colomar , LKML Subject: Re: [PATCH 00/17] Add memberof(), split some headers, and slightly simplify code In-Reply-To: <20211119113644.1600-1-alx.manpages@gmail.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20211119113644.1600-1-alx.manpages@gmail.com> Date: Fri, 19 Nov 2021 14:47:08 +0200 Message-ID: <87mtm0jos3.fsf@intel.com> MIME-Version: 1.0 Cc: Alejandro Colomar , Corey Minyard , Ajit Khaparde , "Michael S. Tsirkin" , Joonas Lahtinen , Rasmus Villemoes , Bjorn Andersson , Chris Mason , Christian Brauner , Ketan Mukadam , Somnath Kotur , linux-scsi@vger.kernel.org, Subbu Seetharaman , "Rafael J. Wysocki" , Russell King , Mike Rapoport , linux-acpi@vger.kernel.org, Miguel Ojeda , Borislav Petkov , virtualization@lists.linux-foundation.org, Len Brown , Kees Cook , Arnd Bergmann , John Hubbard , intel-gfx@lists.freedesktop.org, Josef Bacik , Jitendra Bhivare , Rodrigo Vivi , Jonathan Cameron , David Sterba , Andy Shevchenko , linux-arm-kernel@lists.infradead.org, Sriharsha Basavapatna , netdev@vger.kernel.org, Nick Desaulniers , "John S . Gruber" , Andrew Morton , linux-btrfs@vger.kernel.org X-BeenThere: virtualization@lists.linux-foundation.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Linux virtualization List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: virtualization-bounces@lists.linux-foundation.org Sender: "Virtualization" On Fri, 19 Nov 2021, Alejandro Colomar wrote: > Hi all, > > I simplified some xxxof() macros, > by adding a new macro memberof(), > which implements a common operation in many of them. > > I also splitted many of those macros into tiny headers, > since I noticed that touching those headers implied > recompiling almost the whole kernel. > > Hopefully after this patch there will be less > things to recompile after touching one of those. > > Having simpler headers means that now one can > include one of those without pulling too much stuff > that might break other stuff. > > I removed some unnecessary casts too. > > Every few commits in this series > and of course after the last commit > I rebuilt the kernel and run for a while with it without any problems. > > Please note that I have written very few kernel code > and for example some files wouldn't let me include some of these files, > so I didn't change those. > > What I mean is that, > even though this is super obvious and shouldn't break stuff, > and I'm not new to C, > I'm quite new to the kernel, > and ask that reviewers take deep look, please. > > > In the first and second commits > I changed a lot of stuff in many parts, > and that's why I CCd so many people (also in this cover letter). > However, to avoid spamming, > and since it would be a nightmare to > find all the relevant people affected in so many different areas, > I only CCd in 01, 02 and in the cover letter. > If anyone is interested in reading the full patch set, > I sent it to the LKML. I think with the patch split you have this would be a nightmare to get merged. Please consider refactoring the headers first, and once those are reviewed and merged, you can proceed with using them elsewhere. For example, we'd want the drm/i915 changes in patches separate from changes to other drivers or the core headers. BR, Jani. > > > Thanks, > Alex > > > Alejandro Colomar (17): > linux/container_of.h: Add memberof(T, m) > Use memberof(T, m) instead of explicit NULL dereference > Replace some uses of memberof() by its wrappers > linux/memberof.h: Move memberof() to separate header > linux/typeof_member.h: Move typeof_member() to a separate header > Simplify sizeof(typeof_member()) to sizeof_field() > linux/NULL.h: Move NULL to a separate header > linux/offsetof.h: Move offsetof(T, m) to a separate header > linux/offsetof.h: Implement offsetof() in terms of memberof() > linux/container_of.h: Implement container_of_safe() in terms of > container_of() > linux/container_of.h: Cosmetic > linux/container_of.h: Remove unnecessary cast to (void *) > linux/sizeof_field.h: Move sizeof_field(T, m) to a separate header > include/linux/: Include a smaller header if just for NULL > linux/offsetofend.h: Move offsetofend(T, m) to a separate header > linux/array_size.h: Move ARRAY_SIZE(arr) to a separate header > include/: Include for ARRAY_SIZE() > > arch/x86/include/asm/bootparam_utils.h | 3 +- > arch/x86/kernel/signal_compat.c | 5 ++-- > drivers/gpu/drm/i915/i915_sw_fence.c | 1 + > drivers/gpu/drm/i915/i915_utils.h | 5 ++-- > drivers/gpu/drm/i915/intel_runtime_pm.h | 3 +- > drivers/net/ethernet/emulex/benet/be.h | 10 +++---- > drivers/net/ethernet/i825xx/ether1.c | 7 +++-- > drivers/platform/x86/wmi.c | 3 +- > drivers/scsi/be2iscsi/be.h | 12 ++++---- > drivers/scsi/be2iscsi/be_cmds.h | 5 +++- > fs/btrfs/ctree.h | 5 ++-- > fs/proc/inode.c | 1 + > include/acpi/actypes.h | 4 ++- > include/crypto/internal/blake2b.h | 1 + > include/crypto/internal/blake2s.h | 1 + > include/crypto/internal/chacha.h | 1 + > include/drm/drm_mipi_dbi.h | 1 + > include/drm/drm_mode_object.h | 1 + > include/kunit/test.h | 1 + > include/linux/NULL.h | 10 +++++++ > include/linux/arm_ffa.h | 1 + > include/linux/array_size.h | 15 ++++++++++ > include/linux/blk_types.h | 1 + > include/linux/can/core.h | 1 + > include/linux/clk-provider.h | 1 + > include/linux/container_of.h | 28 ++++++++++------- > include/linux/counter.h | 1 + > include/linux/crash_core.h | 1 + > include/linux/efi.h | 1 + > include/linux/extable.h | 2 +- > include/linux/f2fs_fs.h | 1 + > include/linux/filter.h | 3 ++ > include/linux/fs.h | 1 + > include/linux/genl_magic_func.h | 1 + > include/linux/hashtable.h | 1 + > include/linux/ieee80211.h | 1 + > include/linux/kbuild.h | 3 ++ > include/linux/kernel.h | 7 +---- > include/linux/kfifo.h | 1 + > include/linux/kvm_host.h | 3 ++ > include/linux/libata.h | 1 + > include/linux/llist.h | 1 + > include/linux/memberof.h | 11 +++++++ > include/linux/mlx5/device.h | 1 + > include/linux/mlx5/driver.h | 1 + > include/linux/mm_types.h | 1 + > include/linux/moduleparam.h | 3 ++ > include/linux/mtd/rawnand.h | 1 + > include/linux/netdevice.h | 1 + > include/linux/netfilter.h | 1 + > include/linux/nvme-fc.h | 2 ++ > include/linux/offsetof.h | 17 +++++++++++ > include/linux/offsetofend.h | 19 ++++++++++++ > include/linux/pagemap.h | 1 + > include/linux/phy.h | 1 + > include/linux/phy_led_triggers.h | 1 + > include/linux/pinctrl/machine.h | 1 + > include/linux/property.h | 1 + > include/linux/rcupdate.h | 1 + > include/linux/rcupdate_wait.h | 1 + > include/linux/regmap.h | 1 + > include/linux/sched/task.h | 1 + > include/linux/sizeof_field.h | 14 +++++++++ > include/linux/skb_array.h | 1 + > include/linux/skbuff.h | 1 + > include/linux/skmsg.h | 3 ++ > include/linux/slab.h | 2 ++ > include/linux/spinlock_types.h | 1 + > include/linux/stddef.h | 30 +++---------------- > include/linux/string.h | 5 +++- > include/linux/surface_aggregator/controller.h | 1 + > include/linux/surface_aggregator/serial_hub.h | 1 + > include/linux/swap.h | 1 + > include/linux/ti-emif-sram.h | 1 + > include/linux/typeof_member.h | 11 +++++++ > include/linux/ucs2_string.h | 2 +- > include/linux/vdpa.h | 1 + > include/linux/virtio_config.h | 17 ++++++----- > include/linux/wireless.h | 2 ++ > include/net/bond_3ad.h | 1 + > include/net/dsa.h | 1 + > include/net/ip_vs.h | 1 + > include/net/netfilter/nf_conntrack_tuple.h | 1 + > include/net/netfilter/nf_tables.h | 1 + > include/net/netlink.h | 1 + > include/rdma/uverbs_ioctl.h | 1 + > include/rdma/uverbs_named_ioctl.h | 1 + > include/scsi/scsi_host.h | 1 + > include/sound/soc-dapm.h | 1 + > include/sound/soc.h | 1 + > include/trace/events/wbt.h | 1 + > include/uapi/linux/netfilter/xt_sctp.h | 1 + > include/xen/hvm.h | 1 + > kernel/kallsyms.c | 3 +- > 94 files changed, 255 insertions(+), 79 deletions(-) > create mode 100644 include/linux/NULL.h > create mode 100644 include/linux/array_size.h > create mode 100644 include/linux/memberof.h > create mode 100644 include/linux/offsetof.h > create mode 100644 include/linux/offsetofend.h > create mode 100644 include/linux/sizeof_field.h > create mode 100644 include/linux/typeof_member.h -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1CC42C433FE for ; Fri, 19 Nov 2021 12:49:14 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id CF3F96124B for ; Fri, 19 Nov 2021 12:49:13 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org CF3F96124B Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=NJL4WZh8PnagiGbukwODVfWs0Xdo96VSwtoGEHkQlT8=; b=KadxQ5cwq5578B meX7mjgT1Lzn6EDOYRYE5qeKnsKWG/O92tpO5btGrzycTRBvKk+ljg7ikK0pr0kkgO3S2Wvb7wdY6 nXNYq7w+AALvumfkuQDsFsTz4SOPTfh5oHIzisV8jkklNXHErGHfNMiNhuly25SnkpsBeXolYzonw MnqV+pEfBEHG7wMSxBfNth/F1UOfkubqU0oQFzMO3a23LYNp+S734Dx6lXAYUXdgMHzJlmRw2OYl1 IUaga+Un2oCRGO9jjprPYPZVMEaGKhO9mUC9diurX9a1MJIt+3DjtmW/whLBefKy/qxvsVlh3Ed/T LZgAoADb4VhpV7WeSkSQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mo3Ih-00APKt-Q9; Fri, 19 Nov 2021 12:47:31 +0000 Received: from mga06.intel.com ([134.134.136.31]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mo3Ie-00APJy-3d for linux-arm-kernel@lists.infradead.org; Fri, 19 Nov 2021 12:47:29 +0000 X-IronPort-AV: E=McAfee;i="6200,9189,10172"; a="295221546" X-IronPort-AV: E=Sophos;i="5.87,247,1631602800"; d="scan'208";a="295221546" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Nov 2021 04:47:27 -0800 X-IronPort-AV: E=Sophos;i="5.87,247,1631602800"; d="scan'208";a="507883791" Received: from sgconnee-mobl1.ger.corp.intel.com (HELO localhost) ([10.252.21.83]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Nov 2021 04:47:15 -0800 From: Jani Nikula To: Alejandro Colomar , LKML Cc: Alejandro Colomar , Ajit Khaparde , Andrew Morton , Andy Shevchenko , Arnd Bergmann , Bjorn Andersson , Borislav Petkov , Corey Minyard , Chris Mason , Christian Brauner , David Sterba , Jason Wang , Jitendra Bhivare , John Hubbard , "John S . Gruber" , Jonathan Cameron , Joonas Lahtinen , Josef Bacik , Kees Cook , Ketan Mukadam , Len Brown , "Michael S. Tsirkin" , Miguel Ojeda , Mike Rapoport , Nick Desaulniers , "Rafael J. Wysocki" , Rasmus Villemoes , Rodrigo Vivi , Russell King , Somnath Kotur , Sriharsha Basavapatna , Subbu Seetharaman , intel-gfx@lists.freedesktop.org, linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-btrfs@vger.kernel.org, linux-scsi@vger.kernel.org, netdev@vger.kernel.org, virtualization@lists.linux-foundation.org Subject: Re: [PATCH 00/17] Add memberof(), split some headers, and slightly simplify code In-Reply-To: <20211119113644.1600-1-alx.manpages@gmail.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20211119113644.1600-1-alx.manpages@gmail.com> Date: Fri, 19 Nov 2021 14:47:08 +0200 Message-ID: <87mtm0jos3.fsf@intel.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211119_044728_240005_1ECB72D6 X-CRM114-Status: GOOD ( 30.80 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, 19 Nov 2021, Alejandro Colomar wrote: > Hi all, > > I simplified some xxxof() macros, > by adding a new macro memberof(), > which implements a common operation in many of them. > > I also splitted many of those macros into tiny headers, > since I noticed that touching those headers implied > recompiling almost the whole kernel. > > Hopefully after this patch there will be less > things to recompile after touching one of those. > > Having simpler headers means that now one can > include one of those without pulling too much stuff > that might break other stuff. > > I removed some unnecessary casts too. > > Every few commits in this series > and of course after the last commit > I rebuilt the kernel and run for a while with it without any problems. > > Please note that I have written very few kernel code > and for example some files wouldn't let me include some of these files, > so I didn't change those. > > What I mean is that, > even though this is super obvious and shouldn't break stuff, > and I'm not new to C, > I'm quite new to the kernel, > and ask that reviewers take deep look, please. > > > In the first and second commits > I changed a lot of stuff in many parts, > and that's why I CCd so many people (also in this cover letter). > However, to avoid spamming, > and since it would be a nightmare to > find all the relevant people affected in so many different areas, > I only CCd in 01, 02 and in the cover letter. > If anyone is interested in reading the full patch set, > I sent it to the LKML. I think with the patch split you have this would be a nightmare to get merged. Please consider refactoring the headers first, and once those are reviewed and merged, you can proceed with using them elsewhere. For example, we'd want the drm/i915 changes in patches separate from changes to other drivers or the core headers. BR, Jani. > > > Thanks, > Alex > > > Alejandro Colomar (17): > linux/container_of.h: Add memberof(T, m) > Use memberof(T, m) instead of explicit NULL dereference > Replace some uses of memberof() by its wrappers > linux/memberof.h: Move memberof() to separate header > linux/typeof_member.h: Move typeof_member() to a separate header > Simplify sizeof(typeof_member()) to sizeof_field() > linux/NULL.h: Move NULL to a separate header > linux/offsetof.h: Move offsetof(T, m) to a separate header > linux/offsetof.h: Implement offsetof() in terms of memberof() > linux/container_of.h: Implement container_of_safe() in terms of > container_of() > linux/container_of.h: Cosmetic > linux/container_of.h: Remove unnecessary cast to (void *) > linux/sizeof_field.h: Move sizeof_field(T, m) to a separate header > include/linux/: Include a smaller header if just for NULL > linux/offsetofend.h: Move offsetofend(T, m) to a separate header > linux/array_size.h: Move ARRAY_SIZE(arr) to a separate header > include/: Include for ARRAY_SIZE() > > arch/x86/include/asm/bootparam_utils.h | 3 +- > arch/x86/kernel/signal_compat.c | 5 ++-- > drivers/gpu/drm/i915/i915_sw_fence.c | 1 + > drivers/gpu/drm/i915/i915_utils.h | 5 ++-- > drivers/gpu/drm/i915/intel_runtime_pm.h | 3 +- > drivers/net/ethernet/emulex/benet/be.h | 10 +++---- > drivers/net/ethernet/i825xx/ether1.c | 7 +++-- > drivers/platform/x86/wmi.c | 3 +- > drivers/scsi/be2iscsi/be.h | 12 ++++---- > drivers/scsi/be2iscsi/be_cmds.h | 5 +++- > fs/btrfs/ctree.h | 5 ++-- > fs/proc/inode.c | 1 + > include/acpi/actypes.h | 4 ++- > include/crypto/internal/blake2b.h | 1 + > include/crypto/internal/blake2s.h | 1 + > include/crypto/internal/chacha.h | 1 + > include/drm/drm_mipi_dbi.h | 1 + > include/drm/drm_mode_object.h | 1 + > include/kunit/test.h | 1 + > include/linux/NULL.h | 10 +++++++ > include/linux/arm_ffa.h | 1 + > include/linux/array_size.h | 15 ++++++++++ > include/linux/blk_types.h | 1 + > include/linux/can/core.h | 1 + > include/linux/clk-provider.h | 1 + > include/linux/container_of.h | 28 ++++++++++------- > include/linux/counter.h | 1 + > include/linux/crash_core.h | 1 + > include/linux/efi.h | 1 + > include/linux/extable.h | 2 +- > include/linux/f2fs_fs.h | 1 + > include/linux/filter.h | 3 ++ > include/linux/fs.h | 1 + > include/linux/genl_magic_func.h | 1 + > include/linux/hashtable.h | 1 + > include/linux/ieee80211.h | 1 + > include/linux/kbuild.h | 3 ++ > include/linux/kernel.h | 7 +---- > include/linux/kfifo.h | 1 + > include/linux/kvm_host.h | 3 ++ > include/linux/libata.h | 1 + > include/linux/llist.h | 1 + > include/linux/memberof.h | 11 +++++++ > include/linux/mlx5/device.h | 1 + > include/linux/mlx5/driver.h | 1 + > include/linux/mm_types.h | 1 + > include/linux/moduleparam.h | 3 ++ > include/linux/mtd/rawnand.h | 1 + > include/linux/netdevice.h | 1 + > include/linux/netfilter.h | 1 + > include/linux/nvme-fc.h | 2 ++ > include/linux/offsetof.h | 17 +++++++++++ > include/linux/offsetofend.h | 19 ++++++++++++ > include/linux/pagemap.h | 1 + > include/linux/phy.h | 1 + > include/linux/phy_led_triggers.h | 1 + > include/linux/pinctrl/machine.h | 1 + > include/linux/property.h | 1 + > include/linux/rcupdate.h | 1 + > include/linux/rcupdate_wait.h | 1 + > include/linux/regmap.h | 1 + > include/linux/sched/task.h | 1 + > include/linux/sizeof_field.h | 14 +++++++++ > include/linux/skb_array.h | 1 + > include/linux/skbuff.h | 1 + > include/linux/skmsg.h | 3 ++ > include/linux/slab.h | 2 ++ > include/linux/spinlock_types.h | 1 + > include/linux/stddef.h | 30 +++---------------- > include/linux/string.h | 5 +++- > include/linux/surface_aggregator/controller.h | 1 + > include/linux/surface_aggregator/serial_hub.h | 1 + > include/linux/swap.h | 1 + > include/linux/ti-emif-sram.h | 1 + > include/linux/typeof_member.h | 11 +++++++ > include/linux/ucs2_string.h | 2 +- > include/linux/vdpa.h | 1 + > include/linux/virtio_config.h | 17 ++++++----- > include/linux/wireless.h | 2 ++ > include/net/bond_3ad.h | 1 + > include/net/dsa.h | 1 + > include/net/ip_vs.h | 1 + > include/net/netfilter/nf_conntrack_tuple.h | 1 + > include/net/netfilter/nf_tables.h | 1 + > include/net/netlink.h | 1 + > include/rdma/uverbs_ioctl.h | 1 + > include/rdma/uverbs_named_ioctl.h | 1 + > include/scsi/scsi_host.h | 1 + > include/sound/soc-dapm.h | 1 + > include/sound/soc.h | 1 + > include/trace/events/wbt.h | 1 + > include/uapi/linux/netfilter/xt_sctp.h | 1 + > include/xen/hvm.h | 1 + > kernel/kallsyms.c | 3 +- > 94 files changed, 255 insertions(+), 79 deletions(-) > create mode 100644 include/linux/NULL.h > create mode 100644 include/linux/array_size.h > create mode 100644 include/linux/memberof.h > create mode 100644 include/linux/offsetof.h > create mode 100644 include/linux/offsetofend.h > create mode 100644 include/linux/sizeof_field.h > create mode 100644 include/linux/typeof_member.h -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel