platform/x86: hp-bioscfg: move mutex_lock() down in hp_add_other_attributes()

Browse source

attr_name_kobj's memory allocation is done with mutex_lock() held, this
is not needed.

Move allocation outside of mutex_lock() so unlock is not needed when
allocation fails.

Suggested-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Harshit Mogalapalli <harshit.m.mogalapalli@oracle.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Link: https://lore.kernel.org/r/20231113200742.3593548-2-harshit.m.mogalapalli@oracle.com
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>

...

This commit is contained in:

Harshit Mogalapalli 2023-11-13 12:07:38 -08:00 • committed by Ilpo Järvinen

parent c5dbf04160

commit 5736aa9537

No known key found for this signature in database

GPG key ID: 59AC4F6153E5CE31

1 changed files with 4 additions and 6 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

10

drivers/platform/x86/hp/hp-bioscfg/bioscfg.c

View file