public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: <Ankit_Singh3@Dell.com>
To: <edk2-devel@lists.01.org>
Subject: DxeCapsuleLib returns Status Issue
Date: Mon, 29 Aug 2016 13:21:16 +0530	[thread overview]
Message-ID: <F99FFC4B77F8B14CBD8ED39478A5FA211D4BA332B4@BLRX7MCDC203.AMER.DELL.COM> (raw)

Dell - Internal Use - Confidential
Hi EDK2 Developers,

We are incorporating DxeCapsuleLib for FMP Capsule Update into our UEFI product, but we are hitting few issues as described below.

In case of Capsule Update "SetImage" is randomly done for all the FMP Handles in case Image Type Id GUID and Image Index  matches (this is expected as there might be multiple similar hardware), but looks like the returns of each FMP update is not handled.
For example if there are 5 FMP handles (can be for different-different devices) and assume that any particular device handle is at 3rd index, therefore the update goes through successfully on the 3rd attempt but since HandleCount value is 5 it tries further with 4th and 5th Handle. This 4th & 5th Handle attempt can fail for any of the calls within the FMP Handle "for loop" (HandleProtocol/GetImageInfo) and hence the final status is returned as FAILURE to application layer.

Below is the code-snippet from DxeCapsuleLib.c , in case of HandleProtocol & GetImageInfo failure, "for" loop for Handle count is continued and the previous Status value is over-written with this new return (return from HandleProtocol & GetImageInfo) and finally returned to application.
[cid:image001.jpg@01D201F8.38564220]


Proposed Solution:-
Can ProcessFmpCapsuleImage() have an extra OUT parameter which gives the list of all successful FMP Handles along with the updated GUID values, so that any application can make the judgment of Update SUCCESS/FAILURE based on FMP Handle and proceed accordingly at application layer.

Regards,
Ankit Singh



             reply	other threads:[~2016-08-29  7:51 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-29  7:51 Ankit_Singh3 [this message]
2016-08-29  7:58 ` DxeCapsuleLib returns Status Issue Yao, Jiewen
     [not found]   ` <F99FFC4B77F8B14CBD8ED39478A5FA211D4BA333AE@BLRX7MCDC203.AMER.DELL.COM>
2016-08-31  0:54     ` Yao, Jiewen
2016-09-02  1:51       ` Yao, Jiewen
2016-09-02  3:24         ` Ankit_Singh3
2016-09-02  3:30           ` Yao, Jiewen
     [not found] ` <4A89E2EF3DFEDB4C8BFDE51014F606A14B3D62F7@SHSMSX104.ccr.corp.intel.com>
2016-08-29 15:24   ` Zhang, Chao B
  -- strict thread matches above, loose matches on Subject: below --
2016-08-24 10:22 Ankit_Singh3

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=F99FFC4B77F8B14CBD8ED39478A5FA211D4BA332B4@BLRX7MCDC203.AMER.DELL.COM \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox