Re: [PATCH v9 1/1] efi: a misc char interface for user to update efi firmware

From	Borislav Petkov <bp@alien8.de>
Newsgroups	linux.kernel
Subject	Re: [PATCH v9 1/1] efi: a misc char interface for user to update efi firmware
Date	2015-11-01 12:00 +0100
Message-ID	<qpYwG-7a6-11@gated-at.bofh.it> (permalink)
References	<qovGq-VU-3@gated-at.bofh.it> <qovGr-VU-25@gated-at.bofh.it> <qpYdk-73i-11@gated-at.bofh.it> <qpYwG-7a6-13@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Sun, Nov 01, 2015 at 10:52:52AM +0000, Kweh, Hock Leong wrote:
> Could you share me your dumb file? I did perform negative test, but I did
> not get these dump stack in dmesg. Thanks.

I think almost any file works:

cat /bin/ls > /dev/efi_capsule_loader

> > > +#define UPLOAD_DONE -1
> > 
> > Isn't the fact that upload was finished a success message? If so, why is it a
> > negative value?
> 
> This is to indicate an upload is done and pending for close(2). If a subsequence
> write(2) perform, return error. Comments inputted by Matt and Andy.

But in that case you can return ERR_OCCURRED. UPLOAD_DONE still doesn't
look like a negative value to me as it signals that the upload was done
and thus successful as no errors happened during the upload.

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[PATCH v9 0/1] Enable capsule loader interface for efi firmware updating "Kweh, Hock Leong" <hock.leong.kweh@intel.com> - 2015-10-28 11:00 +0100
  [PATCH v9 1/1] efi: a misc char interface for user to update efi firmware "Kweh, Hock Leong" <hock.leong.kweh@intel.com> - 2015-10-28 11:00 +0100
    Re: [PATCH v9 1/1] efi: a misc char interface for user to update efi  firmware Borislav Petkov <bp@alien8.de> - 2015-11-01 11:40 +0100
      Re: [PATCH v9 1/1] efi: a misc char interface for user to update efi  firmware Borislav Petkov <bp@alien8.de> - 2015-11-01 12:00 +0100
        RE: [PATCH v9 1/1] efi: a misc char interface for user to update  efi firmware "Kweh, Hock Leong" <hock.leong.kweh@intel.com> - 2015-11-01 12:20 +0100
          Re: [PATCH v9 1/1] efi: a misc char interface for user to update efi  firmware Borislav Petkov <bp@alien8.de> - 2015-11-01 14:00 +0100
            RE: [PATCH v9 1/1] efi: a misc char interface for user to update  efi firmware "Kweh, Hock Leong" <hock.leong.kweh@intel.com> - 2015-11-02 08:20 +0100
              Re: [PATCH v9 1/1] efi: a misc char interface for user to update efi  firmware Borislav Petkov <bp@alien8.de> - 2015-11-03 21:20 +0100
                RE: [PATCH v9 1/1] efi: a misc char interface for user to update  efi firmware "Kweh, Hock Leong" <hock.leong.kweh@intel.com> - 2015-11-05 04:50 +0100
      RE: [PATCH v9 1/1] efi: a misc char interface for user to update  efi firmware "Kweh, Hock Leong" <hock.leong.kweh@intel.com> - 2015-11-01 12:00 +0100

csiph-web