All of lore.kernel.org
 help / color / mirror / Atom feed
From: Doug Ledford <dledford@redhat.com>
To: Saeed Mahameed <saeedm@mellanox.com>,
	"David S. Miller" <davem@davemloft.net>
Cc: netdev@vger.kernel.org, linux-rdma@vger.kernel.org,
	Leon Romanovsky <leonro@mellanox.com>,
	Jason Gunthorpe <jgg@mellanox.com>,
	Aviad Yehezkel <aviadye@mellanox.com>,
	Matan Barak <matanb@mellanox.com>,
	Boris Pismenny <borisp@mellanox.com>
Subject: Re: [for-next V2 12/13] {net,IB}/mlx5: Add flow steering helpers
Date: Wed, 07 Mar 2018 14:58:00 -0500	[thread overview]
Message-ID: <1520452680.3126.35.camel@redhat.com> (raw)
In-Reply-To: <20180307063516.22773-13-saeedm@mellanox.com>

[-- Attachment #1: Type: text/plain, Size: 2103 bytes --]

On Tue, 2018-03-06 at 22:35 -0800, Saeed Mahameed wrote:
> diff --git a/include/linux/mlx5/fs_helpers.h b/include/linux/mlx5/fs_helpers.h
> new file mode 100644
> index 000000000000..7b476bbae731
> --- /dev/null
> +++ b/include/linux/mlx5/fs_helpers.h
> @@ -0,0 +1,134 @@
> +/*
> + * Copyright (c) 2018, Mellanox Technologies. All rights reserved.
> + *
> + * This software is available to you under a choice of one of two
> + * licenses.  You may choose to be licensed under the terms of the GNU
> + * General Public License (GPL) Version 2, available from the file
> + * COPYING in the main directory of this source tree, or the
> + * OpenIB.org BSD license below:
> + *
> + *     Redistribution and use in source and binary forms, with or
> + *     without modification, are permitted provided that the following
> + *     conditions are met:
> + *
> + *      - Redistributions of source code must retain the above
> + *        copyright notice, this list of conditions and the following
> + *        disclaimer.
> + *
> + *      - Redistributions in binary form must reproduce the above
> + *        copyright notice, this list of conditions and the following
> + *        disclaimer in the documentation and/or other materials
> + *        provided with the distribution.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> + * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
> + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
> + * BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
> + * ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
> + * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
> + * SOFTWARE.
> + */
> +

Nit: I would prefer a license identifier line followed by just the
copyright tag, not the full boiler plate.

-- 
Doug Ledford <dledford@redhat.com>
    GPG KeyID: B826A3330E572FDD
    Key fingerprint = AE6B 1BDA 122B 23B4 265B  1274 B826 A333 0E57 2FDD

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2018-03-07 19:58 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-07  6:35 [pull request][for-next V2 00/13] Mellanox, mlx5 IPSec updates 2018-02-28-1 Saeed Mahameed
2018-03-07  6:35 ` [for-next V2 01/13] IB/mlx5: Removed not used parameters Saeed Mahameed
2018-03-07  6:35 ` [for-next V2 02/13] net/mlx5: Fixed compilation issue when CONFIG_MLX5_ACCEL is disabled Saeed Mahameed
2018-03-07  6:35 ` [for-next V2 03/13] net/mlx5e: Wait for FPGA command responses with a timeout Saeed Mahameed
2018-03-07 10:25   ` Kirill Tkhai
2018-03-07 20:01     ` Saeed Mahameed
2018-03-07  6:35 ` [for-next V2 04/13] net/mlx5e: Fixed sleeping inside atomic context Saeed Mahameed
2018-03-07  6:35 ` [for-next V2 05/13] net/mlx5e: Removed not need synchronize_rcu Saeed Mahameed
2018-03-07  6:35 ` [for-next V2 06/13] net/mlx5: FPGA and IPSec initialization to be before flow steering Saeed Mahameed
2018-03-07  6:35 ` [for-next V2 07/13] IB/mlx5: Pass mlx5_flow_act struct instead of multiple arguments Saeed Mahameed
2018-03-07  6:35 ` [for-next V2 08/13] {net,IB}/mlx5: Add has_tag to mlx5_flow_act Saeed Mahameed
2018-03-07  6:35 ` [for-next V2 09/13] net/mlx5: Add shim layer between fs and cmd Saeed Mahameed
2018-03-07  6:35 ` [for-next V2 10/13] net/mlx5: Add empty egress namespace to flow steering core Saeed Mahameed
2018-03-07  6:35 ` [for-next V2 11/13] net/mlx5: Embed mlx5_flow_act into fs_fte Saeed Mahameed
2018-03-07  6:35 ` [for-next V2 12/13] {net,IB}/mlx5: Add flow steering helpers Saeed Mahameed
2018-03-07 19:58   ` Doug Ledford [this message]
2018-03-07  6:35 ` [for-next V2 13/13] net/mlx5: Flow steering cmd interface should get the fte when deleting Saeed Mahameed
2018-03-07 19:57 ` [pull request][for-next V2 00/13] Mellanox, mlx5 IPSec updates 2018-02-28-1 Doug Ledford
2018-03-07 20:31 ` David Miller
2018-03-07 20:41   ` Doug Ledford
2018-03-07 20:57     ` Doug Ledford
2018-03-07 23:13       ` Saeed Mahameed

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=1520452680.3126.35.camel@redhat.com \
    --to=dledford@redhat.com \
    --cc=aviadye@mellanox.com \
    --cc=borisp@mellanox.com \
    --cc=davem@davemloft.net \
    --cc=jgg@mellanox.com \
    --cc=leonro@mellanox.com \
    --cc=linux-rdma@vger.kernel.org \
    --cc=matanb@mellanox.com \
    --cc=netdev@vger.kernel.org \
    --cc=saeedm@mellanox.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.