Jakub Kicinski 556c755a4d net: make dev_alloc_name() call dev_prep_valid_name()
__dev_alloc_name() handles both the sprintf and non-sprintf
target names. This complicates the code.

dev_prep_valid_name() already handles the non-sprintf case,
before calling __dev_alloc_name(), make the only other caller
also go thru dev_prep_valid_name(). This way we can drop
the non-sprintf handling in __dev_alloc_name() in one of
the next changes.

commit 55a5ec9b7710 ("Revert "net: core: dev_get_valid_name is now the same as dev_alloc_name_ns"") and
commit 029b6d140550 ("Revert "net: core: maybe return -EEXIST in __dev_alloc_name"")
tell us that we can't start returning -EEXIST from dev_alloc_name()
on name duplicates. Bite the bullet and pass the expected errno to
dev_prep_valid_name().

dev_prep_valid_name() must now propagate out the allocated id
for printf names.

Reviewed-by: Jiri Pirko <jiri@nvidia.com>
Link: https://lore.kernel.org/r/20231023152346.3639749-3-kuba@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
2023-10-24 13:02:58 -07:00
..
2023-10-23 08:45:25 +01:00
2023-10-01 19:09:54 +01:00
2023-05-31 13:06:57 +02:00
2023-10-23 16:16:51 -07:00
2023-08-29 17:39:15 -07:00
2023-10-23 09:35:01 +01:00
2023-03-16 17:34:40 -07:00
2023-06-24 15:41:46 -07:00
2023-10-23 09:35:01 +01:00
2023-10-01 19:09:54 +01:00
2023-06-28 16:43:10 -07:00
2023-10-01 19:09:54 +01:00
2023-10-11 10:39:01 +01:00