A nit on the description: "paremeter" should be "parameter"

LGTM

LGTM

Correct author in commit log.

In D38551#877968, @vangyzen wrote:

Common practice for attributing the submitter with git is:

git commit --amend --author='Robert Herndon <Robert.Herndon@Dell.com>'

Re-submit

I'll commit the change.

LGTM

I'll go ahead and commit.

• Fix typo

• Shorten comment

Looks good; just a few style nits.

Looks pretty good. I would agree that it should be tested on a LEGACY (non-UEFI) BIOS.

LGTM

LGTM

Seems reasonable, although I think the explanation from your description of this change should really go in the code comment rather than the rather terse and enigmatic "Prevent pollution from bogus symbols".

Allocate the buffer based on max_xfer_size as suggested by a review comment.

Combine statements that calculate the transfer size as suggested by review comments.

In D25363#559516, @cem wrote:

Do delete events actually notify on the deleted file? I would expect to receive a notification on the pid's parent directory, but am not sure about the pid file itself.

In D25312#558449, @jhb wrote:

BTW, one thing this leaves ambiguous is [whether] you can declare variables in the middle of blocks vs at the beginning. That is:

Switch from a (uint64_t) cast to specifying ULL on a numeric constant
to widen the data type of an expression (suggested by imp@).

Changes based on some review comments.

Changes based on some review comments.