mbox series

[v2,00/11] add SPDX tags to files with Intel copyright

Message ID 20190726164343.77981-1-xiaolong.ye@intel.com (mailing list archive)
Headers
Series add SPDX tags to files with Intel copyright |

Message

Xiaolong Ye July 26, 2019, 4:43 p.m. UTC
  This patchset replaces full BSD-3 license header with spdx tag for the
remaining files that have Intel copyright.


V2 changes:

1. remove "All rights reserved"
2. change "Intel shannon Ltd" to "Intel Corporation"
3. add ack tag from Bruce


Xiaolong Ye (11):
  net/ixgbe: replace license text with SPDX tag
  net/i40e: replace license text with SPDX tag
  net/e1000: replace license text with SPDX tag
  net/vmxnet3: replace license text with SPDX tag
  examples/qos_sched: replace license text with SPDX tag
  app/test-eventdev: replace license text with SPDX tag
  ethdev: replace license text with SPDX tag
  lpm: replace license text with SPDX tag
  port: replace license text with SPDX tag
  hash: replace license text with SPDX tag
  eal: replace license text with SPDX tag

 app/test-eventdev/parser.c                    | 35 ++---------------
 drivers/net/e1000/base/README                 | 34 ++--------------
 drivers/net/i40e/base/README                  | 34 ++--------------
 drivers/net/ixgbe/base/README                 | 34 ++--------------
 drivers/net/vmxnet3/base/README               | 34 ++--------------
 examples/qos_sched/profile.cfg                | 32 +--------------
 examples/qos_sched/profile_ov.cfg             | 32 +--------------
 .../common/include/arch/x86/rte_atomic_32.h   | 33 +---------------
 .../common/include/arch/x86/rte_atomic_64.h   | 33 +---------------
 lib/librte_eal/common/include/rte_keepalive.h | 32 +--------------
 lib/librte_eal/freebsd/BSDmakefile.meson      | 33 +---------------
 lib/librte_ethdev/rte_mtr.h                   | 36 ++---------------
 lib/librte_ethdev/rte_tm.h                    | 37 ++----------------
 lib/librte_hash/rte_jhash.h                   | 33 +---------------
 lib/librte_lpm/rte_lpm_neon.h                 | 39 ++-----------------
 lib/librte_port/rte_port_kni.c                | 35 ++---------------
 lib/librte_port/rte_port_kni.h                | 35 ++---------------
 17 files changed, 46 insertions(+), 535 deletions(-)
  

Comments

Hemant Agrawal July 29, 2019, 7:12 a.m. UTC | #1
> This patchset replaces full BSD-3 license header with spdx tag for the
> remaining files that have Intel copyright.

[Hemant]  Is it possible for you to also cover some of other remaining Intel copyright files. E.g.

drivers/net/i40e/i40e_rxtx_vec_avx2.c
doc/guides/eventdevs/opdl.rst

drivers/net/i40e/i40e_rxtx_vec_altivec.c

> 
> 
> V2 changes:
> 
> 1. remove "All rights reserved"
> 2. change "Intel shannon Ltd" to "Intel Corporation"
> 3. add ack tag from Bruce
> 
> 
> Xiaolong Ye (11):
>   net/ixgbe: replace license text with SPDX tag
>   net/i40e: replace license text with SPDX tag
>   net/e1000: replace license text with SPDX tag
>   net/vmxnet3: replace license text with SPDX tag
>   examples/qos_sched: replace license text with SPDX tag
>   app/test-eventdev: replace license text with SPDX tag
>   ethdev: replace license text with SPDX tag
>   lpm: replace license text with SPDX tag
>   port: replace license text with SPDX tag
>   hash: replace license text with SPDX tag
>   eal: replace license text with SPDX tag
> 
>  app/test-eventdev/parser.c                    | 35 ++---------------
>  drivers/net/e1000/base/README                 | 34 ++--------------
>  drivers/net/i40e/base/README                  | 34 ++--------------
>  drivers/net/ixgbe/base/README                 | 34 ++--------------
>  drivers/net/vmxnet3/base/README               | 34 ++--------------
>  examples/qos_sched/profile.cfg                | 32 +--------------
>  examples/qos_sched/profile_ov.cfg             | 32 +--------------
>  .../common/include/arch/x86/rte_atomic_32.h   | 33 +---------------
>  .../common/include/arch/x86/rte_atomic_64.h   | 33 +---------------
>  lib/librte_eal/common/include/rte_keepalive.h | 32 +--------------
>  lib/librte_eal/freebsd/BSDmakefile.meson      | 33 +---------------
>  lib/librte_ethdev/rte_mtr.h                   | 36 ++---------------
>  lib/librte_ethdev/rte_tm.h                    | 37 ++----------------
>  lib/librte_hash/rte_jhash.h                   | 33 +---------------
>  lib/librte_lpm/rte_lpm_neon.h                 | 39 ++-----------------
>  lib/librte_port/rte_port_kni.c                | 35 ++---------------
>  lib/librte_port/rte_port_kni.h                | 35 ++---------------
>  17 files changed, 46 insertions(+), 535 deletions(-)
> 
> --
> 2.17.1
  
Xiaolong Ye July 29, 2019, 7:27 a.m. UTC | #2
Hi, Hemant

On 07/29, Hemant Agrawal wrote:
>> This patchset replaces full BSD-3 license header with spdx tag for the
>> remaining files that have Intel copyright.
>
>[Hemant]  Is it possible for you to also cover some of other remaining Intel copyright files. E.g.
>
>drivers/net/i40e/i40e_rxtx_vec_avx2.c
>doc/guides/eventdevs/opdl.rst
>
>drivers/net/i40e/i40e_rxtx_vec_altivec.c

drivers/net/i40e/i40e_rxtx_vec_avx2.c and drivers/net/i40e/i40e_rxtx_vec_altivec.c has been handled
by https://patches.dpdk.org/patch/56985/ which has been merged in next-net tree.

I'll submit a new patch to address doc/guides/eventdevs/opdl.rst.

Thanks,
Xiaolong

>
>> 
>> 
>> V2 changes:
>> 
>> 1. remove "All rights reserved"
>> 2. change "Intel shannon Ltd" to "Intel Corporation"
>> 3. add ack tag from Bruce
>> 
>> 
>> Xiaolong Ye (11):
>>   net/ixgbe: replace license text with SPDX tag
>>   net/i40e: replace license text with SPDX tag
>>   net/e1000: replace license text with SPDX tag
>>   net/vmxnet3: replace license text with SPDX tag
>>   examples/qos_sched: replace license text with SPDX tag
>>   app/test-eventdev: replace license text with SPDX tag
>>   ethdev: replace license text with SPDX tag
>>   lpm: replace license text with SPDX tag
>>   port: replace license text with SPDX tag
>>   hash: replace license text with SPDX tag
>>   eal: replace license text with SPDX tag
>> 
>>  app/test-eventdev/parser.c                    | 35 ++---------------
>>  drivers/net/e1000/base/README                 | 34 ++--------------
>>  drivers/net/i40e/base/README                  | 34 ++--------------
>>  drivers/net/ixgbe/base/README                 | 34 ++--------------
>>  drivers/net/vmxnet3/base/README               | 34 ++--------------
>>  examples/qos_sched/profile.cfg                | 32 +--------------
>>  examples/qos_sched/profile_ov.cfg             | 32 +--------------
>>  .../common/include/arch/x86/rte_atomic_32.h   | 33 +---------------
>>  .../common/include/arch/x86/rte_atomic_64.h   | 33 +---------------
>>  lib/librte_eal/common/include/rte_keepalive.h | 32 +--------------
>>  lib/librte_eal/freebsd/BSDmakefile.meson      | 33 +---------------
>>  lib/librte_ethdev/rte_mtr.h                   | 36 ++---------------
>>  lib/librte_ethdev/rte_tm.h                    | 37 ++----------------
>>  lib/librte_hash/rte_jhash.h                   | 33 +---------------
>>  lib/librte_lpm/rte_lpm_neon.h                 | 39 ++-----------------
>>  lib/librte_port/rte_port_kni.c                | 35 ++---------------
>>  lib/librte_port/rte_port_kni.h                | 35 ++---------------
>>  17 files changed, 46 insertions(+), 535 deletions(-)
>> 
>> --
>> 2.17.1
>
  
Hemant Agrawal July 29, 2019, 7:28 a.m. UTC | #3
> > This patchset replaces full BSD-3 license header with spdx tag for the
> > remaining files that have Intel copyright.
> 
> [Hemant]  Is it possible for you to also cover some of other remaining Intel
> copyright files. E.g.
> 
> drivers/net/i40e/i40e_rxtx_vec_avx2.c
> doc/guides/eventdevs/opdl.rst
> 
> drivers/net/i40e/i40e_rxtx_vec_altivec.c

[Hemant] Actually there are more files from Intel without SPDX tags (using:  grep -Lr "SPDX" *)

drivers/net/fm10k/base/fm10k_api.c
drivers/net/fm10k/base/fm10k_mbx.h
drivers/net/fm10k/base/fm10k_osdep.h
drivers/net/fm10k/base/fm10k_vf.c
drivers/net/fm10k/base/fm10k_mbx.c
drivers/net/fm10k/base/fm10k_api.h
drivers/net/fm10k/base/fm10k_type.h
drivers/net/fm10k/base/fm10k_common.h
drivers/net/fm10k/base/fm10k_common.c
drivers/net/fm10k/base/fm10k_pf.h
drivers/net/fm10k/base/fm10k_pf.c
drivers/net/fm10k/base/fm10k_vf.h
drivers/net/fm10k/base/fm10k_tlv.h
drivers/net/fm10k/base/fm10k_tlv.c

drivers/net/iavf/base/iavf_adminq.h
drivers/net/iavf/base/iavf_alloc.h
drivers/net/iavf/base/iavf_type.h
drivers/net/iavf/base/iavf_hmc.h
drivers/net/iavf/base/iavf_adminq_cmd.h
drivers/net/iavf/base/iavf_register.h
drivers/net/iavf/base/iavf_devids.h
drivers/net/iavf/base/iavf_adminq.c
drivers/net/iavf/base/iavf_common.c
drivers/net/iavf/base/iavf_lan_hmc.h
drivers/net/iavf/base/virtchnl.h
drivers/net/iavf/base/iavf_status.h
drivers/net/iavf/base/iavf_prototype.h

drivers/net/e1000/base/e1000_hw.h
drivers/net/e1000/base/e1000_osdep.h
drivers/net/e1000/base/e1000_82571.c
drivers/net/e1000/base/e1000_mbx.c
drivers/net/e1000/base/e1000_mbx.h
drivers/net/e1000/base/e1000_api.h
drivers/net/e1000/base/e1000_82543.h
drivers/net/e1000/base/e1000_defines.h
drivers/net/e1000/base/e1000_regs.h
drivers/net/e1000/base/e1000_82541.c
drivers/net/e1000/base/e1000_phy.h
drivers/net/e1000/base/e1000_mac.c
drivers/net/e1000/base/e1000_vf.h
drivers/net/e1000/base/e1000_api.c
drivers/net/e1000/base/e1000_80003es2lan.h
drivers/net/e1000/base/e1000_80003es2lan.c
drivers/net/e1000/base/e1000_ich8lan.c
drivers/net/e1000/base/e1000_osdep.c
drivers/net/e1000/base/e1000_manage.h
drivers/net/e1000/base/e1000_ich8lan.h
drivers/net/e1000/base/e1000_vf.c
drivers/net/e1000/base/e1000_nvm.h
drivers/net/e1000/base/e1000_82571.h
drivers/net/e1000/base/e1000_i210.h
drivers/net/e1000/base/e1000_phy.c
drivers/net/e1000/base/e1000_manage.c
drivers/net/e1000/base/e1000_82541.h
drivers/net/e1000/base/e1000_82540.c
drivers/net/e1000/base/e1000_82543.c
drivers/net/e1000/base/e1000_82575.h
drivers/net/e1000/base/e1000_82575.c
drivers/net/e1000/base/e1000_nvm.c
drivers/net/e1000/base/e1000_mac.h
drivers/net/e1000/base/e1000_82542.c
drivers/net/e1000/base/e1000_i210.c

 
> >
> >
> > V2 changes:
> >
> > 1. remove "All rights reserved"
> > 2. change "Intel shannon Ltd" to "Intel Corporation"
> > 3. add ack tag from Bruce
> >
> >
> > Xiaolong Ye (11):
> >   net/ixgbe: replace license text with SPDX tag
> >   net/i40e: replace license text with SPDX tag
> >   net/e1000: replace license text with SPDX tag
> >   net/vmxnet3: replace license text with SPDX tag
> >   examples/qos_sched: replace license text with SPDX tag
> >   app/test-eventdev: replace license text with SPDX tag
> >   ethdev: replace license text with SPDX tag
> >   lpm: replace license text with SPDX tag
> >   port: replace license text with SPDX tag
> >   hash: replace license text with SPDX tag
> >   eal: replace license text with SPDX tag
> >
> >  app/test-eventdev/parser.c                    | 35 ++---------------
> >  drivers/net/e1000/base/README                 | 34 ++--------------
> >  drivers/net/i40e/base/README                  | 34 ++--------------
> >  drivers/net/ixgbe/base/README                 | 34 ++--------------
> >  drivers/net/vmxnet3/base/README               | 34 ++--------------
> >  examples/qos_sched/profile.cfg                | 32 +--------------
> >  examples/qos_sched/profile_ov.cfg             | 32 +--------------
> >  .../common/include/arch/x86/rte_atomic_32.h   | 33 +---------------
> >  .../common/include/arch/x86/rte_atomic_64.h   | 33 +---------------
> >  lib/librte_eal/common/include/rte_keepalive.h | 32 +--------------
> >  lib/librte_eal/freebsd/BSDmakefile.meson      | 33 +---------------
> >  lib/librte_ethdev/rte_mtr.h                   | 36 ++---------------
> >  lib/librte_ethdev/rte_tm.h                    | 37 ++----------------
> >  lib/librte_hash/rte_jhash.h                   | 33 +---------------
> >  lib/librte_lpm/rte_lpm_neon.h                 | 39 ++-----------------
> >  lib/librte_port/rte_port_kni.c                | 35 ++---------------
> >  lib/librte_port/rte_port_kni.h                | 35 ++---------------
> >  17 files changed, 46 insertions(+), 535 deletions(-)
> >
> > --
> > 2.17.1
  
Xiaolong Ye July 29, 2019, 7:37 a.m. UTC | #4
Hi, Hemant

On 07/29, Hemant Agrawal wrote:
>> > This patchset replaces full BSD-3 license header with spdx tag for the
>> > remaining files that have Intel copyright.
>> 
>> [Hemant]  Is it possible for you to also cover some of other remaining Intel
>> copyright files. E.g.
>> 
>> drivers/net/i40e/i40e_rxtx_vec_avx2.c
>> doc/guides/eventdevs/opdl.rst
>> 
>> drivers/net/i40e/i40e_rxtx_vec_altivec.c
>
>[Hemant] Actually there are more files from Intel without SPDX tags (using:  grep -Lr "SPDX" *)
>

They have been addressed by patchset http://mails.dpdk.org/archives/dev/2019-July/139370.html which
has been merged into next-net tree.

Thanks,
Xiaolong


>drivers/net/fm10k/base/fm10k_api.c
>drivers/net/fm10k/base/fm10k_mbx.h
>drivers/net/fm10k/base/fm10k_osdep.h
>drivers/net/fm10k/base/fm10k_vf.c
>drivers/net/fm10k/base/fm10k_mbx.c
>drivers/net/fm10k/base/fm10k_api.h
>drivers/net/fm10k/base/fm10k_type.h
>drivers/net/fm10k/base/fm10k_common.h
>drivers/net/fm10k/base/fm10k_common.c
>drivers/net/fm10k/base/fm10k_pf.h
>drivers/net/fm10k/base/fm10k_pf.c
>drivers/net/fm10k/base/fm10k_vf.h
>drivers/net/fm10k/base/fm10k_tlv.h
>drivers/net/fm10k/base/fm10k_tlv.c
>
>drivers/net/iavf/base/iavf_adminq.h
>drivers/net/iavf/base/iavf_alloc.h
>drivers/net/iavf/base/iavf_type.h
>drivers/net/iavf/base/iavf_hmc.h
>drivers/net/iavf/base/iavf_adminq_cmd.h
>drivers/net/iavf/base/iavf_register.h
>drivers/net/iavf/base/iavf_devids.h
>drivers/net/iavf/base/iavf_adminq.c
>drivers/net/iavf/base/iavf_common.c
>drivers/net/iavf/base/iavf_lan_hmc.h
>drivers/net/iavf/base/virtchnl.h
>drivers/net/iavf/base/iavf_status.h
>drivers/net/iavf/base/iavf_prototype.h
>
>drivers/net/e1000/base/e1000_hw.h
>drivers/net/e1000/base/e1000_osdep.h
>drivers/net/e1000/base/e1000_82571.c
>drivers/net/e1000/base/e1000_mbx.c
>drivers/net/e1000/base/e1000_mbx.h
>drivers/net/e1000/base/e1000_api.h
>drivers/net/e1000/base/e1000_82543.h
>drivers/net/e1000/base/e1000_defines.h
>drivers/net/e1000/base/e1000_regs.h
>drivers/net/e1000/base/e1000_82541.c
>drivers/net/e1000/base/e1000_phy.h
>drivers/net/e1000/base/e1000_mac.c
>drivers/net/e1000/base/e1000_vf.h
>drivers/net/e1000/base/e1000_api.c
>drivers/net/e1000/base/e1000_80003es2lan.h
>drivers/net/e1000/base/e1000_80003es2lan.c
>drivers/net/e1000/base/e1000_ich8lan.c
>drivers/net/e1000/base/e1000_osdep.c
>drivers/net/e1000/base/e1000_manage.h
>drivers/net/e1000/base/e1000_ich8lan.h
>drivers/net/e1000/base/e1000_vf.c
>drivers/net/e1000/base/e1000_nvm.h
>drivers/net/e1000/base/e1000_82571.h
>drivers/net/e1000/base/e1000_i210.h
>drivers/net/e1000/base/e1000_phy.c
>drivers/net/e1000/base/e1000_manage.c
>drivers/net/e1000/base/e1000_82541.h
>drivers/net/e1000/base/e1000_82540.c
>drivers/net/e1000/base/e1000_82543.c
>drivers/net/e1000/base/e1000_82575.h
>drivers/net/e1000/base/e1000_82575.c
>drivers/net/e1000/base/e1000_nvm.c
>drivers/net/e1000/base/e1000_mac.h
>drivers/net/e1000/base/e1000_82542.c
>drivers/net/e1000/base/e1000_i210.c
>
> 
>> >
>> >
>> > V2 changes:
>> >
>> > 1. remove "All rights reserved"
>> > 2. change "Intel shannon Ltd" to "Intel Corporation"
>> > 3. add ack tag from Bruce
>> >
>> >
>> > Xiaolong Ye (11):
>> >   net/ixgbe: replace license text with SPDX tag
>> >   net/i40e: replace license text with SPDX tag
>> >   net/e1000: replace license text with SPDX tag
>> >   net/vmxnet3: replace license text with SPDX tag
>> >   examples/qos_sched: replace license text with SPDX tag
>> >   app/test-eventdev: replace license text with SPDX tag
>> >   ethdev: replace license text with SPDX tag
>> >   lpm: replace license text with SPDX tag
>> >   port: replace license text with SPDX tag
>> >   hash: replace license text with SPDX tag
>> >   eal: replace license text with SPDX tag
>> >
>> >  app/test-eventdev/parser.c                    | 35 ++---------------
>> >  drivers/net/e1000/base/README                 | 34 ++--------------
>> >  drivers/net/i40e/base/README                  | 34 ++--------------
>> >  drivers/net/ixgbe/base/README                 | 34 ++--------------
>> >  drivers/net/vmxnet3/base/README               | 34 ++--------------
>> >  examples/qos_sched/profile.cfg                | 32 +--------------
>> >  examples/qos_sched/profile_ov.cfg             | 32 +--------------
>> >  .../common/include/arch/x86/rte_atomic_32.h   | 33 +---------------
>> >  .../common/include/arch/x86/rte_atomic_64.h   | 33 +---------------
>> >  lib/librte_eal/common/include/rte_keepalive.h | 32 +--------------
>> >  lib/librte_eal/freebsd/BSDmakefile.meson      | 33 +---------------
>> >  lib/librte_ethdev/rte_mtr.h                   | 36 ++---------------
>> >  lib/librte_ethdev/rte_tm.h                    | 37 ++----------------
>> >  lib/librte_hash/rte_jhash.h                   | 33 +---------------
>> >  lib/librte_lpm/rte_lpm_neon.h                 | 39 ++-----------------
>> >  lib/librte_port/rte_port_kni.c                | 35 ++---------------
>> >  lib/librte_port/rte_port_kni.h                | 35 ++---------------
>> >  17 files changed, 46 insertions(+), 535 deletions(-)
>> >
>> > --
>> > 2.17.1
>
  
Ferruh Yigit July 29, 2019, 12:17 p.m. UTC | #5
On 7/26/2019 5:43 PM, Xiaolong Ye wrote:
> This patchset replaces full BSD-3 license header with spdx tag for the
> remaining files that have Intel copyright.
> 
> 
> V2 changes:
> 
> 1. remove "All rights reserved"
> 2. change "Intel shannon Ltd" to "Intel Corporation"
> 3. add ack tag from Bruce
> 
> 
> Xiaolong Ye (11):
>   net/ixgbe: replace license text with SPDX tag
>   net/i40e: replace license text with SPDX tag
>   net/e1000: replace license text with SPDX tag
>   net/vmxnet3: replace license text with SPDX tag
>   examples/qos_sched: replace license text with SPDX tag
>   app/test-eventdev: replace license text with SPDX tag
>   ethdev: replace license text with SPDX tag
>   lpm: replace license text with SPDX tag
>   port: replace license text with SPDX tag
>   hash: replace license text with SPDX tag
>   eal: replace license text with SPDX tag

Series applied to dpdk-next-net/master, thanks.

Except from [07/11] & [09/11] which are missing ack from copyright owners.