Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Kubalewski, Arkadiusz" <arkadiusz.kubalewski@intel.com>
To:     Jakub Kicinski <kuba@kernel.org>
CC:     "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "davem@davemloft.net" <davem@davemloft.net>,
        "pabeni@redhat.com" <pabeni@redhat.com>,
        "edumazet@google.com" <edumazet@google.com>,
        "chuck.lever@oracle.com" <chuck.lever@oracle.com>
Subject: RE: [PATCH net-next] tools: ynl-gen: fix nested policy attribute type
Thread-Topic: [PATCH net-next] tools: ynl-gen: fix nested policy attribute
 type
Thread-Index: AQHZnk1/UXRcaWtblku7CYhq2FDxoK+Jd+2AgADChtA=
Date:   Wed, 14 Jun 2023 12:28:24 +0000
Message-ID: <DM6PR11MB4657C9D29C67874FC90365EC9B5AA@DM6PR11MB4657.namprd11.prod.outlook.com>
References: <20230613231709.150622-1-arkadiusz.kubalewski@intel.com>
 <20230613175051.32196c59@kernel.org>
In-Reply-To: <20230613175051.32196c59@kernel.org>
Accept-Language: pl-PL, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?hdpv9bcK3NubXkhfksZLcaORN8M17p7mGFmXehWzYy0gXRyrje8hJC26i3Zm?=
 =?us-ascii?Q?nhTU8962O5I4j6BR9xgvImp7Aq1u02llzlTenYeP3xisJndgf2yzVEaGcHge?=
 =?us-ascii?Q?NidEvDJV2fDJGc7tvg3vrbh9BeTOjgdo21NtUaqwPDeKkXomWbIYyOogNV69?=
 =?us-ascii?Q?vdcPf9zQGjkPV3v1u6Y59+sJZ9/uP4pucxbwN3tFDnhsU2L9LaTp3fQ+eSFU?=
 =?us-ascii?Q?nZhX342DADs7kM/IsHylvYw/32tGq/B6TdbOHZMHOzb4IPEQCNs4ZoOyCZF0?=
 =?us-ascii?Q?+dqRJqZ2sGXL2bu5HsQ81hwAd520ErKdewYmc3OVNQSvg2XAOGjC03ufMSSN?=
 =?us-ascii?Q?SWcFg3lb8mr197DhPs/S28gNhJnMsvBYX0e4VM9polBGpsL+A8TSmu8BC8Q+?=
 =?us-ascii?Q?zAA5sdrkQ5xAt6R3dN6PSmxxlDQ84CI+CS4Zl5NVOI28vE6SLINwa3eT9PSB?=
 =?us-ascii?Q?FYm1T5a+fT4SlFocjxVyLCenHbHherCIlAhB6uXU7LgpszE0jOYlr3FjHj2M?=
 =?us-ascii?Q?MC4uJMy9GCywtbX3vZWSF1/BcRZWkuoS+bjUmJVp0+RTcwWifv37OcEFTQtX?=
 =?us-ascii?Q?KcfiYHlXoOppSBtQqfOiK4bXyXFfRAG47fzavdMlm7KDP33UUq/uZFKXgYBO?=
 =?us-ascii?Q?BiNJ0p7gy65/74LXxBVosroFFkaUWU4+15E7IWbmqNDUctYwb1IRffwP26/5?=
 =?us-ascii?Q?1fuZssJMVGSVrg8NwJqPe1NHpFGqs7XQifrBHPgV6WiMLYLHEZvJAtq7LMYG?=
 =?us-ascii?Q?HnImBZ9lvx7k9DTduuHa1zUaluyhMwxC1ie+AzKcSPtPWzfOcrSG1ZNdIY1Z?=
 =?us-ascii?Q?ZHwHfbMVSjdRmDCnZ1ajGLuC4P9H1oHD5PKCxCG4R8bqs5Z6Dm+vtemYFVH+?=
 =?us-ascii?Q?ipd5XxsfW94MnR582gLj4RHTSqcQDu0LzIDRbSvA5rxTxJN1DfgnYUxEPxAM?=
 =?us-ascii?Q?+Gz8lU+VTKPbV1A+oK/Xj272tWSgicNCcSIlgN3Dr1JX+I6IpPnRHoEHVCBs?=
 =?us-ascii?Q?YXxWJQ0Me4aGipbE8UXRn7dN/6krGRLFzlAWaEr0UiCMHS1DeeUIpsyEf+vV?=
 =?us-ascii?Q?48AxZxT1+uyW5+ZOAZitTY9CaGyZRvPd6FQvQeXuySsa2s9uZp/R2qYiXwSl?=
 =?us-ascii?Q?imu9BZXh5MXjMXGSfmqtTDXKl3qGxkQXpTMkTfxuZAfu3GExGt7I/AELEIIF?=
 =?us-ascii?Q?+Q4DO5MqGPpo77jU2YfTyfsJIXeF8JdvMiB1qqWYo4QnLGGFY9d1hFGng+UG?=
 =?us-ascii?Q?m9i2owxiM6m91r75DwiruoYKvlyQjZdyIz0iZjhq5dMu62gzaWp2fC/jCBAZ?=
 =?us-ascii?Q?f/GnEy89M23yAKRi4I6xocPkfUayogmU6xAf0ekDfX9REkw3oTr4lLG6UVn/?=
 =?us-ascii?Q?5023KB2IqlVPSDNgt71gcHmGBJf83qZgY3B8GPj3CvhiFzbi6baARlc43Soj?=
 =?us-ascii?Q?bJtSSPW+Ma5/ipQHx1XDyviwsB4Yt6rctf/70SJ3uPy1jdNaBFAY39uugA6W?=
 =?us-ascii?Q?yEWqt9q+bxib92wSqL1E5fyOojIc9jAdnMB7x73OFAtEzy8KqaMCEGlQWQ0/?=
 =?us-ascii?Q?8Pre0KFKP+2S1mR5Fgx9c/scse8lZDWVkyf8JffOAjaOkGyfyIiO+q0k7dp/?=
 =?us-ascii?Q?XQ=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB4657.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 454af03b-5d6f-468a-f5ac-08db6cd2d971
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Jun 2023 12:28:24.9711
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: lrvnG8boQ7JZD/nCv0Kx4N+6714iEprXivTEbv5vEVoR4IiFDuhSYaY9qI431Hpdm1lpOaZc+sPgH3EBrvnSw3eXK+kB7jrTIlHp78Ki5f8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR11MB8140
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

>From: Jakub Kicinski <kuba@kernel.org>
>Sent: Wednesday, June 14, 2023 2:51 AM
>To: Kubalewski, Arkadiusz <arkadiusz.kubalewski@intel.com>
>Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org;
>davem@davemloft.net; pabeni@redhat.com; edumazet@google.com;
>chuck.lever@oracle.com
>Subject: Re: [PATCH net-next] tools: ynl-gen: fix nested policy attribute
>type
>
>On Wed, 14 Jun 2023 01:17:07 +0200 Arkadiusz Kubalewski wrote:
>> When nested multi-attribute is used in yaml spec, generated type in
>> the netlink policy is NLA_NEST, which is wrong as there is no such type.
>> Fix be adding `ed` sufix for policy generated for 'nest' type attribute
>> when the attribute is parsed as TypeMultiAttr class.
>
>I CCed you on my changes which address the same issue, they have
>already been merged:
>
>https://lore.kernel.org/all/20230612155920.1787579-1-kuba@kernel.org/
>
>I think that covers the first two patches. What am I missing? :S

Yep, sorry I missed them.
Did rebase and it seems your changes are great, everything works,
please drop this patch.

Thank you!
Arkadiusz