linux-kernel - Re: [RFC][PATCH 1/2] Duplicate a shared code between write and erase callbacks

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20120705131802.GE5637@redhat.com>
Date:	Thu, 5 Jul 2012 09:18:02 -0400
From:	Don Zickus <dzickus@...hat.com>
To:	Seiji Aguchi <seiji.aguchi@....com>
Cc:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"Luck, Tony (tony.luck@...el.com)" <tony.luck@...el.com>,
	"mikew@...gle.com" <mikew@...gle.com>,
	"Matthew Garrett (mjg@...hat.com)" <mjg@...hat.com>,
	"dle-develop@...ts.sourceforge.net" 
	<dle-develop@...ts.sourceforge.net>,
	Satoru Moriya <satoru.moriya@....com>
Subject: Re: [RFC][PATCH 1/2] Duplicate a shared code between write and erase
 callbacks

On Tue, Jul 03, 2012 at 11:34:58PM +0000, Seiji Aguchi wrote:
> This patch duplicates a code of erase callback shared with write callback so that they work independently.

Can you expand this changelog to explain why this is needed and what
problem you are trying to fix?  I know you talked about it in the patch
series header, but the header doesn't get committed.

Cheers,
Don

> 
> Signed-off-by: Seiji Aguchi <seiji.aguchi@....com>
> ---
>  drivers/firmware/efivars.c |   46 +++++++++++++++++++++++++++++++++++++++++++-
>  1 files changed, 45 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/firmware/efivars.c b/drivers/firmware/efivars.c index 47408e8..4929254 100644
> --- a/drivers/firmware/efivars.c
> +++ b/drivers/firmware/efivars.c
> @@ -756,7 +756,51 @@ static int efi_pstore_write(enum pstore_type_id type,  static int efi_pstore_erase(enum pstore_type_id type, u64 id,
>  			    struct pstore_info *psi)
>  {
> -	efi_pstore_write(type, 0, &id, (unsigned int)id, 0, psi);
> +	char stub_name[DUMP_NAME_LEN];
> +	efi_char16_t efi_name[DUMP_NAME_LEN];
> +	efi_guid_t vendor = LINUX_EFI_CRASH_GUID;
> +	struct efivars *efivars = psi->data;
> +	struct efivar_entry *entry, *found = NULL;
> +	int i;
> +
> +	sprintf(stub_name, "dump-type%u-%llu-", type, id);
> +
> +	spin_lock(&efivars->lock);
> +
> +	for (i = 0; i < DUMP_NAME_LEN; i++)
> +		efi_name[i] = stub_name[i];
> +
> +	/*
> +	 * Clean up any entries with the same name
> +	 */
> +
> +	list_for_each_entry(entry, &efivars->list, list) {
> +		get_var_data_locked(efivars, &entry->var);
> +
> +		if (efi_guidcmp(entry->var.VendorGuid, vendor))
> +			continue;
> +		if (utf16_strncmp(entry->var.VariableName, efi_name,
> +				  utf16_strlen(efi_name)))
> +			continue;
> +		/* Needs to be a prefix */
> +		if (entry->var.VariableName[utf16_strlen(efi_name)] == 0)
> +			continue;
> +
> +		/* found */
> +		found = entry;
> +		efivars->ops->set_variable(entry->var.VariableName,
> +					   &entry->var.VendorGuid,
> +					   PSTORE_EFI_ATTRIBUTES,
> +					   0, NULL);
> +	}
> +
> +	if (found)
> +		list_del(&found->list);
> +
> +	spin_unlock(&efivars->lock);
> +
> +	if (found)
> +		efivar_unregister(found);
>  
>  	return 0;
>  }
> --
> 1.7.1
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/