Received-SPF: pass (google.com: domain of linux-wireless+bounces-1542-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <728b6538-19b3-4fb4-b91e-f72080c4c9f0@westermo.com>
Date: Fri, 5 Jan 2024 21:35:40 +0100
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] iw: strip NLA_FLAGS from printed bands
Content-Language: en-US
To: Johannes Berg <johannes@sipsolutions.net>,
        "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
CC: Zefir Kurtisi <Zefir.Kurtisi@westermo.com>
References: <20231221222842.1310957-1-matthias.may@westermo.com>
 <6a54a2afd9e2b1b67822e3ca9b86654e3886feef.camel@sipsolutions.net>
From: Matthias May <matthias.may@westermo.com>
In-Reply-To: <6a54a2afd9e2b1b67822e3ca9b86654e3886feef.camel@sipsolutions.net>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?OWpSUjEzSUpYQ0pNZDBOdUZ5akR0ZXVvbVNFOHZ0U1c5dFJCV0R2UHpJNmFm?=
 =?utf-8?B?RXE5bldqRmg2d3VwM3dDSFBzWkNYMExtM3VxVjQ1dlVxTi9oaC9TTjByV0tG?=
 =?utf-8?B?U1RPMTBlM28rajg5bEoxKzdJRlNSZW1iUnZSVTh5UEprNDUrdFBkdFFDa2JZ?=
 =?utf-8?B?T3F6TmRab1N1VS9OSGU3M2FwaVRHbXBMZFFSL0Y4RW5PZFBGMFRVWENIN2Nu?=
 =?utf-8?B?UjVGTkdKbkJiOFFUSkc1UDRIakxYNFpqSjhESVNEaHRiWUxjc1pYSTJZQXZI?=
 =?utf-8?B?VDBJK1k2Nk90MC80UGpSZ0ZwWndGWlFWWU9hQUhUUTRKQWFCOEVmYlVEN0Vz?=
 =?utf-8?B?RU5wMVVOeUREQTZadnBFeVlnTE5TdkxxcGZaOVFXeHZkK2hVVi9FSjhsZW9h?=
 =?utf-8?B?SnN0czNOUUJ2SElEQTdYaklXcERhRnFPWC9ROWIxZWUyVkZEdUJEQStGRzdF?=
 =?utf-8?B?NGcxMFlmR3c4eTQ5TUNoUXpLQ0gyTURNcWN6eGRwRkdHRlF2SFZHZFhBVG52?=
 =?utf-8?B?QWtHY1ZTTS84REpsRTIyWjJkdTRrRjlXejVJMnhIeWdvTnRaNE1FVGZRL2gr?=
 =?utf-8?B?TFJSNitsc3A3RStoOTdFSU9iRGJkZUtiSDBoY3ZNT3hjK1dJbjdrMUdmTUJ3?=
 =?utf-8?B?bGtjNEszSnJja3NsS3dHazE2KytKRDdYUnFZYVpXR3lvUk4vVHJsWWVZT2VZ?=
 =?utf-8?B?d1ZqNkU4OXYwUDdjOCtYL3g0UHNaSjVqSlpNUmU2TXQvZmhPcVI3dzRVMm4y?=
 =?utf-8?B?Slc0NWRFUjJVMkdPcCtlQkUwNmh0NmVQNXJqSnFZVG5tek5kY0tya0ZkTFNp?=
 =?utf-8?B?WUtEb011YnY3bE15QTAyWmtmUHBLQWdZN0p4c0g5TDlhQzJxZFNJNjdaTG5Q?=
 =?utf-8?B?WXUrWmZjWUo3RzFVSitSWXdpeHdjWnVCTE56QlJVSXcyNTZjeG9xVjNVNEho?=
 =?utf-8?B?ZlhMN0lqSzc0ZDBCdUxVd2NlSURKa0VSSXg4MkhGekZFd1hLUWh2cEh5TUx1?=
 =?utf-8?B?b2tnZTVQaFpzOHFLUllrUUI0ZHNzb1ZtMEhBUnVnT1ZGdUJ6QnZDd0FpQkw4?=
 =?utf-8?B?RDFRUC9SV3ptS09aQTRiSUlRcGVSVDViMXZ6QTQzMzR6SHQ0WnVBN01LRGpL?=
 =?utf-8?B?eUtPcUdUcHRrN0hDUFFPd205RXlzRmZ6Yk00bjZQY0RxVnBseEkwRHFTVHFj?=
 =?utf-8?B?QUp1RyswekU1QVJMWGl2Y0JnZXhySVJ0eHBLYVREajduck16bkVLb3pQL1Zy?=
 =?utf-8?B?UWJQZmVxRnhHWUI0eGlvUytiQnk2QUxmUG9vMHdreUZCQTBPa0RVUmRwS0Qz?=
 =?utf-8?B?U004ZDFLM0JQem1oaGRWQVFGS25UM2wyUWJTMWZXbW5JREptZ2Q4UngwU05G?=
 =?utf-8?B?eUR1bFpkR2dYU2VxaTI3TnFoWHE4ck5lUGFaWjVFQzJPZ2xsSmtvM0tCdVlp?=
 =?utf-8?B?MWlpaCtvTGtnd1hpcWgwWHdvbnF1S09XOEhtYzY2MktNUGJZamE2anNTWVFC?=
 =?utf-8?B?QnpVR2hKN0Q4Mm4vbkVlNDM1NTZENTBKdVlTNTVBUGVQSGtTN2ZzSndkY1Bp?=
 =?utf-8?B?LzdYMnJtVmJjSGFpT1owSjlQUzEzQzdDT2xVUWNYMlMwZ2pjTFlBNURQWWJ1?=
 =?utf-8?B?UnErME95STdwZnR0czZCcEVnSVNyTWVJUUNGTk5VdHRJdUJxc1o1ZmkwYnNa?=
 =?utf-8?B?Mk1EU0E0ZTZNNG5RQ1pLM20wNlF6SVg5bFlUcnY5WDk4VU0rM1NSd0FIV1ZN?=
 =?utf-8?B?MTB1cnNmZDQ3czUzUlBKUDFDZFh6MnZCNWZ0c0pzay83TlJrLzMzVTVVRGFj?=
 =?utf-8?B?b1o3aFNKcldNQ1NqM0szaHNlc3doV01zREwzZytDUFBnYVF1TzVIZ0c2OU9M?=
 =?utf-8?B?NWs4aU5qRjEzRUx6NVVuQytuRUl0WklneUJGb3lJeVlERjFmUGZJeUxEdTIw?=
 =?utf-8?B?akFOZ2tUaE4xbmlsUEQ4VWNzcmFtOEE2bC9VLysxeFd4WGNqS1hUVk1XaTZK?=
 =?utf-8?B?U0ZQMXdaSjF2V29rQUYyOWNNbDkvSU1kV0QxQ250cjlyOTJpUzUzUjRDYzc4?=
 =?utf-8?B?WmVUSU1tU3E4SUZZQzdHQ3ZuM1lxb3lDNENIblRwT3p4aG5kQytpdDgxNzNX?=
 =?utf-8?Q?T6lWXOJLWuQwE3KT+nP1PRpkz?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 72608f40-2da1-4b06-9101-08dc0e2de2cf
X-MS-Exchange-CrossTenant-AuthSource: DB9P192MB1388.EURP192.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Jan 2024 20:35:42.3235
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4b2e9b91-de77-4ca7-8130-c80faee67059
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: VOtfOqxKcH2V0ZtSGfqkoa3WXzLcGpBTW1cXz94HR4iqsj7rw3djMq0i4txacPTbg52BLpjpL98wY0K5kiRwIA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9P192MB0902
X-OrganizationHeadersPreserved: AM9P192MB0902.EURP192.PROD.OUTLOOK.COM
X-CrossPremisesHeadersPromoted: EX01GLOBAL.beijerelectronics.com
X-CrossPremisesHeadersFiltered: EX01GLOBAL.beijerelectronics.com
X-OriginatorOrg: westermo.com
X-Proofpoint-GUID: tBZUidp8DsS0TpxJWAtYGHecojnTvp-0
X-Proofpoint-ORIG-GUID: tBZUidp8DsS0TpxJWAtYGHecojnTvp-0

On 31/12/2023 22:59, Johannes Berg wrote:
> On Thu, 2023-12-21 at 22:29 +0000, Matthias May wrote:
>> nl_band->nla_type might have NLA_F_NESTED (0x8000) set,
> I'm curious - how does that happen?
>
> Not that the change looks wrong per se, but ... I don't think this can
> happen?
>
> johannes
>
Hi Johannes

We saw that happen on one of our platforms where we are forced to use an 
out-of-tree driver by QCA (SPF12.2).
If you think this is something that can not happen with upstream 
drivers, feel free to ignore this patch.

BR
Matthias