Received-SPF: pass (google.com: domain of linux-kernel+bounces-52689-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
From: "Sa, Nuno" <Nuno.Sa@analog.com>
To: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        "Rafael J. Wysocki"
	<rafael@kernel.org>,
        Frank Rowand <frowand.list@gmail.com>,
        Rob Herring
	<robh+dt@kernel.org>, Daniel Scally <djrscally@gmail.com>,
        Heikki Krogerus
	<heikki.krogerus@linux.intel.com>,
        Sakari Ailus
	<sakari.ailus@linux.intel.com>,
        Len Brown <lenb@kernel.org>,
        "linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH v2 2/2] of: dynamic: flush devlinks workqueue before
 destroying the changeset
Thread-Topic: [PATCH v2 2/2] of: dynamic: flush devlinks workqueue before
 destroying the changeset
Thread-Index: AQHaWC/za6qNUilHpEGYd1mFVIhg5LD7t9JA
Date: Mon, 5 Feb 2024 13:10:33 +0000
Message-ID: 
 <SJ0PR03MB677807F5ABCD412E94EA1BBF99472@SJ0PR03MB6778.namprd03.prod.outlook.com>
References: <20240205-fix-device-links-overlays-v2-0-5344f8c79d57@analog.com>
 <20240205-fix-device-links-overlays-v2-2-5344f8c79d57@analog.com>
 <ZcDWQ52BGix3ocUC@smile.fi.intel.com>
In-Reply-To: <ZcDWQ52BGix3ocUC@smile.fi.intel.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: 
 =?iso-8859-1?Q?shJeO29wWuVDcHCXBFhvlFcV1MuBjZH8QzIOLqQeHOtLl+3bOq59v4ZYTP?=
 =?iso-8859-1?Q?U/ZYDaK5BPkao5HizXeROL4bmcUbYyrxveFP/yRqn+2kFyZoWkS6A+T+Md?=
 =?iso-8859-1?Q?n0Nddm+bFK2PSy44WN9ZsN2AKNwMh+4YOGELILnZuRL8mwRuKmC//4Pfjg?=
 =?iso-8859-1?Q?ld095w6TICIgM+f0+e/3tP5i0UQRqiQZw0YST/LIp+tc9iyRmEhs1sFEOU?=
 =?iso-8859-1?Q?/5IHs9bVCoNLe6Abc4uEtx2kSV33n+i+/09mCUQCjXBR3K15B8WQI1NVca?=
 =?iso-8859-1?Q?RhysF1p7DDMh3wuoc32dll88XHfbXxMl8hH/P4FJZ6dJ0GJO65pe2yOOZG?=
 =?iso-8859-1?Q?7wGVDXFCNvJ7YANgHNNs9dECGiUgekufK/UEt4Pbf5l1+uMrzzrQBua7Ot?=
 =?iso-8859-1?Q?maVg40QRO6nc2pZvwn8AVbPW6cOg3mTQdnysohWbXlYcT1qvLYqB9nyXIx?=
 =?iso-8859-1?Q?h+pzpEH6lg95UZze1aineArKcmIdeQXeLv+HvLES+IVeXVurbhQXm4SOqS?=
 =?iso-8859-1?Q?BMswtt9DB0sokZ6S84ZQVjUgcvmbEroJguJifS53+5W+tXeE07OPN4+Nnl?=
 =?iso-8859-1?Q?AZ+e3Qz9zwtCzdqXDl9NsuUfH6G5zs5L9wxyV0CRVRJyH1F7+SvH8/t+8B?=
 =?iso-8859-1?Q?2van1AhkfnP/+bK9USfPuxw6+sknH8F2jnZWVCMXzDEa6iQuvfLj6sM6BW?=
 =?iso-8859-1?Q?oNy0zO3aoSaawBttrQT+61ToDTLZvpc0K7hxXUFdV5R0XwISl1OQALwlpp?=
 =?iso-8859-1?Q?KzWLKCwFsVhhRbBHcVF2IxGyI2wjro8/t3m/QFdjmsotNrWTaOUC+9fhzU?=
 =?iso-8859-1?Q?rVuq4DuXqUJ3z6Z4rLyKwK6UbVrLTJrQQKA6w6EAYF7qDZvaDs7AXfAI89?=
 =?iso-8859-1?Q?agV7NQA7RP3/q5mBs5dDp8n4734ap9Ihsy5DuPl6EAtj5aCpBDQuDJtyXx?=
 =?iso-8859-1?Q?ZmovupHtfo1qz2Pft+Bm7pGCVR6jqUX75qfdCki8vjtM4xNGx44pSnm+OZ?=
 =?iso-8859-1?Q?K3MZ6ptBH9tjEisws4gh5BmMK4Q/E38Ej7yWwV7xF1DRA00A1TyMXCP/JF?=
 =?iso-8859-1?Q?FMWjyRkH8UmiFwHMkKPkRtUSPHAc/+bMcyNr+q0WO4b4JaXRqY30neN6U/?=
 =?iso-8859-1?Q?scHjv2mf0aiRaUnxiDYuCzg7ut0SMhz7tzb8BFiEBzy3R+WFarSh3ESoj9?=
 =?iso-8859-1?Q?L2dEwPQeqQBfRgY4IL/iIOT23O1xUlK3cXdijDDo1WjsfTNh97/cnUdA/5?=
 =?iso-8859-1?Q?52HUWgqx6vdVbMgZmY8jaIQIe4THGcejSteaUReKC8nPGihRllahXZ213U?=
 =?iso-8859-1?Q?P3oZjBZaSkyLVbNb7NxG93dttem7joMcaMNLakJSQ5jKQ57aRL7Dy0IHnB?=
 =?iso-8859-1?Q?EQcmUHK1aL8H4iiHmFGtf9+WnSv+U0jPTBCffsAxv7wHleKznP57/U3O1o?=
 =?iso-8859-1?Q?zG4o3sgf6VQLN1tvWLNmP6BXxpjme5V0GfnAscJF0a1TaNIxvWTrWNXNcc?=
 =?iso-8859-1?Q?hUOd5IUY8oKk/zVLjZf5KU7QFszNKdQngFEhJYAHL8Qs2uDwi2b6JA8rwF?=
 =?iso-8859-1?Q?Ip6sGKvk3OP+EOniirGlbw+8TtUhEQ/LJagQRvTReIF2ty5/WwlsescUGB?=
 =?iso-8859-1?Q?+cqLivsB9PI9k=3D?=
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: analog.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR03MB6778.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: dc54c6ab-80e3-439e-cbf1-08dc264bd63f
X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Feb 2024 13:10:33.8318
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: eaa689b4-8f87-40e0-9c6f-7228de4d754a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: sNhJyulKyDmdpGTeXBbCmOGNA/Xu3n8QPEyylIPUbBIpj0NfuSxYRhstd8BToCSlNGHUh9B0ifN5OnF7ffl2wQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR03MB5630
X-Proofpoint-GUID: 5bedthlHUmf7refX-5s4NIZhwVA7RVIf
X-Proofpoint-ORIG-GUID: 5bedthlHUmf7refX-5s4NIZhwVA7RVIf
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2024-02-05_08,2024-01-31_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 lowpriorityscore=0
 priorityscore=1501 spamscore=0 malwarescore=0 mlxscore=0 mlxlogscore=593
 bulkscore=0 impostorscore=0 phishscore=0 adultscore=0 suspectscore=0
 clxscore=1011 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.19.0-2401310000 definitions=main-2402050099


> -----Original Message-----
> From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Sent: Monday, February 5, 2024 1:36 PM
> To: Sa, Nuno <Nuno.Sa@analog.com>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>; Rafael J. Wysocki
> <rafael@kernel.org>; Frank Rowand <frowand.list@gmail.com>; Rob Herring
> <robh+dt@kernel.org>; Daniel Scally <djrscally@gmail.com>; Heikki Krogeru=
s
> <heikki.krogerus@linux.intel.com>; Sakari Ailus <sakari.ailus@linux.intel=
com>;
> Len Brown <lenb@kernel.org>; linux-acpi@vger.kernel.org;
> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH v2 2/2] of: dynamic: flush devlinks workqueue before
> destroying the changeset
>=20
> [External]
>=20
> On Mon, Feb 05, 2024 at 01:09:33PM +0100, Nuno Sa via B4 Relay wrote:
> > From: Nuno Sa <nuno.sa@analog.com>
> >
> > Device links will drop their supplier + consumer refcounts
> > asynchronously. That means that the refcount of the of_node attached to
> > these devices will also be dropped asynchronously and so we cannot
> > guarantee the DT overlay assumption that the of_node refcount must be 1=
 in
> > __of_changeset_entry_destroy().
> >
> > Given the above, call the new fwnode_links_flush_queue() helper to flus=
h
> > the devlink workqueue so we can be sure that all links are dropped befo=
re
> > doing the proper checks.
>=20
> Have you seen my comments against v1?

Hmm, not really. lei is not fetching any email (even this one - that's why =
I'm using my work email + awesome
email client to reply).
And looking directly in lore... I'm also not seeing any reply from you. May=
be there's an issue.

- Nuno S=E1