On 11 February 2016 at 14:14, Matt Fleming <matt@xxxxxxxxxxxxxxxxxxx> wrote: > On Wed, 13 Jan, at 05:32:40PM, Sylvain Chouleur wrote: >> From: Sylvain Chouleur <sylvain.chouleur@xxxxxxxxx> >> >> This patch remove the efivars lock to use a single lock for the whole >> vars.c file. The goal of this lock is to protect concurrent calls to >> efi variable services, registering and unregistering. This allows to >> register new efivars operations without having in-progress call. >> >> Signed-off-by: Sylvain Chouleur <sylvain.chouleur@xxxxxxxxx> >> --- >> drivers/firmware/efi/vars.c | 85 +++++++++++++++++++++++++-------------------- >> include/linux/efi.h | 6 ---- >> 2 files changed, 48 insertions(+), 43 deletions(-) > > Sorry for the delay in reviewing this. I wasn't on Cc so I only found > it when searching through the linux-efi archive. > > This change looks fine to me and a nice clean up. Ard any comments? No looks good to me -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
