| Message ID | 20230523123801.2007784-3-jiri@resnulli.us (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 1bb1b57898504da4e10d48b901556278e161c7fd |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | devlink: small port_new/del() cleanup | expand |
| Context | Check | Description |
|---|---|---|
| netdev/series_format | success | Posting correctly formatted |
| netdev/tree_selection | success | Clearly marked for net-next |
| netdev/fixes_present | success | Fixes tag not required for -next series |
| netdev/header_inline | success | No static functions without inline keyword in header files |
| netdev/build_32bit | success | Errors and warnings before: 388 this patch: 388 |
| netdev/cc_maintainers | success | CCed 6 of 6 maintainers |
| netdev/build_clang | success | Errors and warnings before: 23 this patch: 23 |
| netdev/verify_signedoff | success | Signed-off-by tag matches author and committer |
| netdev/deprecated_api | success | None detected |
| netdev/check_selftest | success | No net selftest shell script |
| netdev/verify_fixes | success | No Fixes tag |
| netdev/build_allmodconfig_warn | success | Errors and warnings before: 538 this patch: 538 |
| netdev/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 16 lines checked |
| netdev/kdoc | success | Errors and warnings before: 14 this patch: 14 |
| netdev/source_inline | success | Was 0 now: 0 |
On Tue, May 23, 2023 at 02:38:00PM +0200, Jiri Pirko wrote: > From: Jiri Pirko <jiri@nvidia.com> > > All commands are called holding instance lock. Remove the outdated > comment that says otherwise. > > Signed-off-by: Jiri Pirko <jiri@nvidia.com> Reviewed-by: Simon Horman <simon.horman@corigine.com>
diff --git a/include/net/devlink.h b/include/net/devlink.h index ccea6e079777..24a48f3d4c35 100644 --- a/include/net/devlink.h +++ b/include/net/devlink.h @@ -1506,8 +1506,6 @@ struct devlink_ops { * attributes * * Notes: - * - Called without devlink instance lock being held. Drivers must - * implement own means of synchronization * - On success, drivers must register a port with devlink core * * Return: 0 on success, negative value otherwise. @@ -1525,8 +1523,6 @@ struct devlink_ops { * to delete a previously created port function * * Notes: - * - Called without devlink instance lock being held. Drivers must - * implement own means of synchronization * - On success, drivers must unregister the corresponding devlink * port *