Re: [PATCH] Remove warning in efi_enter_virtual_mode V2

2013-04-19 Thread Greg KH
On Thu, Apr 18, 2013 at 11:00:27PM -0700, H. Peter Anvin wrote:
 The only real use for command line options is to be able to get users
 to test something during troubleshooting.

Ok, but the option shouldn't be used to solve the problem.

greg k-h
--
To unsubscribe from this list: send the line unsubscribe linux-efi in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] Remove warning in efi_enter_virtual_mode V2

2013-04-19 Thread H. Peter Anvin
Nope, just a vehicle for experiments.

Greg KH gre...@linuxfoundation.org wrote:

On Thu, Apr 18, 2013 at 11:00:27PM -0700, H. Peter Anvin wrote:
 The only real use for command line options is to be able to get users
 to test something during troubleshooting.

Ok, but the option shouldn't be used to solve the problem.

greg k-h

-- 
Sent from my mobile phone. Please excuse brevity and lack of formatting.
--
To unsubscribe from this list: send the line unsubscribe linux-efi in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] Remove warning in efi_enter_virtual_mode

2013-04-19 Thread Josh Boyer
On Fri, Apr 19, 2013 at 08:50:14AM +0100, Matt Fleming wrote:
 On 04/19/2013 01:18 AM, Darren Hart wrote:
  On 04/18/2013 09:19 AM, Matt Fleming wrote:
 
  Could you give it a spin on your MinnowBoard?
  
  I've removed the patch I reference above and applied your patch to my
  3.8.4 MinnowBoard dev tree. It panics with:
 
 D'oh. OK, at this point I'm inclined to apply Josh Boyer's patch on top
 of my urgent branch which will address the WARNING people are hitting on
 i386. I updated the commit message a little.
 
 Josh (Boyer), are you guys still carrying this patch and have you seen
 any fallout? I notice your SoB isn't on the patch that Darren posted, am
 I OK to add it?

Yeah, we're still carrying it.  We've had it around since August of 2011
to cover the bug mentioned.  I'm not aware of any further fallout it
might have caused, but again we don't support 32-bit EFI in Fedora.
That's not to say Fedora won't work, we just don't focus on it.

I don't actually remember authoring the patch, but it's been a while so:

Signed-off-by: Josh Boyer jwbo...@redhat.com

josh
--
To unsubscribe from this list: send the line unsubscribe linux-efi in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[GIT PULL] EFI fixes for v3.10

2013-04-19 Thread Matt Fleming
Hi guys,

There's a few bug fixes sitting in the EFI urgent branch. Please
consider pulling.

The following changes since commit ca0ba26fbbd2d81c43085df49ce0abfe34535a90:

  efivars: Fix check for CONFIG_EFI_VARS_PSTORE_DEFAULT_DISABLE (2013-03-22 
20:08:01 +)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/mfleming/efi.git urgent

for you to fetch changes up to 8c58bf3eec3b8fc8162fe557e9361891c20758f2:

  x86,efi: Implement efi_no_storage_paranoia parameter (2013-04-17 15:13:38 
+0100)


Matt Fleming (1):
  x86, efivars: firmware bug workarounds should be in platform code

Matthew Garrett (3):
  Move utf16 functions to kernel core and rename
  efi: Pass boot services variable info to runtime code
  efi: Distinguish between remaining space and actually used space

Richard Weinberger (2):
  x86,efi: Check max_size only if it is non-zero.
  x86,efi: Implement efi_no_storage_paranoia parameter

Sergey Vlasov (2):
  x86/Kconfig: Make EFI select UCS2_STRING
  efi: Export efi_query_variable_store() for efivars.ko

 Documentation/kernel-parameters.txt   |   6 ++
 arch/x86/Kconfig  |   1 +
 arch/x86/boot/compressed/eboot.c  |  47 ++
 arch/x86/include/asm/efi.h|   7 ++
 arch/x86/include/uapi/asm/bootparam.h |   1 +
 arch/x86/platform/efi/efi.c   | 168 +-
 drivers/firmware/Kconfig  |   1 +
 drivers/firmware/efivars.c|  98 +---
 include/linux/efi.h   |   9 +-
 include/linux/ucs2_string.h   |  14 +++
 lib/Kconfig   |   3 +
 lib/Makefile  |   2 +
 lib/ucs2_string.c |  51 +++
 13 files changed, 325 insertions(+), 83 deletions(-)
 create mode 100644 include/linux/ucs2_string.h
 create mode 100644 lib/ucs2_string.c

-- 
Matt Fleming, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line unsubscribe linux-efi in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] EFI fixes for v3.10

2013-04-19 Thread H. Peter Anvin
On 04/19/2013 10:38 AM, Matt Fleming wrote:
 Hi guys,
 
 There's a few bug fixes sitting in the EFI urgent branch. Please
 consider pulling.
 

We are *post -rc7* and days away from release.

I can't push this to Linus without a detailed description of what this
is and why we need it now.

Yes, I can wing it, but I shouldn't *have to* especially not when half
out of my gourd...

-hpa


--
To unsubscribe from this list: send the line unsubscribe linux-efi in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] EFI fixes for v3.10

2013-04-19 Thread H. Peter Anvin
On 04/19/2013 10:38 AM, Matt Fleming wrote:
 
 Richard Weinberger (2):
   x86,efi: Check max_size only if it is non-zero.
   x86,efi: Implement efi_no_storage_paranoia parameter
 

OK, I'll pull this for now, but this *really* need to be an automatic
quirk of some sort, especially when there is a named board in the mix...

-hpa


--
To unsubscribe from this list: send the line unsubscribe linux-efi in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html