Re: [Xen-devel] [PATCH v2 1/2] x86/boot: fix early error display

[Daniel Kiper <daniel.kiper@xxxxxxxxxx>]

On Tue, Oct 17, 2017 at 04:41:37PM -0500, Doug Goldstein wrote:
> From: David Esler <drumandstrum@xxxxxxxxx>
>
> In 9180f5365524 a change was made to the send_chr function to take in
> C-strings and print out a character at a time until a NULL was
> encountered. However there is no code to increment the current character
> position resulting in an endless loop of the first character. This adds
> a simple increment.
>
> Reviewed-by: Doug Goldstein <cardoe@xxxxxxxxxx>

I was told that "Reviewed-by: ..." should be after SOB.

> Signed-off-by: David Esler <drumandstrum@xxxxxxxxx>

In general Reviewed-by: Daniel Kiper <daniel.kiper@xxxxxxxxxx>
Though please take into account Jan's request WRT to commit
message. Or I am OK with Jan's changes before committing.

Daniel

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel