<div dir="ltr"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Fri, 2021-08-20 at 01:26 +0000, HAGIO KAZUHITO(萩尾　一仁) wrote:<br>> -----Original Message-----<br>> > Dear Kazu,<br>> > <br>> > Sorry for late reply<br>> > Thanks for your suggestion and I have prepared a V2 patch, please<br>> > help<br>> > to review.<br>> <br>> ok, I've modified your v2 patch to fix the following compilation<br>> warning<br>> and rewrite the commit log, and attached it.<br>> <br>> arm64.c: In function ‘arm64_init’:<br>> arm64.c:3728:35: warning: ‘note’ may be used uninitialized in this<br>> function [-Wmaybe-uninitialized]<br></blockquote><div><br></div><div>This warning can be eliminated as follows:</div><div>--- a/arm64.c<br>+++ b/arm64.c<br>@@ -3698,14 +3698,48 @@ arm64_get_crash_notes(void)<br> {<br>        struct machine_specific *ms = machdep->machspec;<br>        ulong crash_notes;<br>-       Elf64_Nhdr *note;<br>+       Elf64_Nhdr *note = NULL;<br>        ulong offset;<br></div><div><br></div><div>But I didn't see this correction in V2. Can you help to add the initialization of the note variable Kazu. Otherwise:</div><div>Acked-by:  Lianbo Jiang <<a href="mailto:lijiang@redhat.com">lijiang@redhat.com</a>></div><div> </div><div>BTW: the patch format is still unfriendly to me.</div><div><br></div><div>Thanks</div><div>Lianbo</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">>      offset = roundup(offset + note->n_namesz, 4);<br>>                                    ^~<br>> Kazu, thanks for your kind help.<br><br>> Lianbo, could you review the attached patch?<br>> <br>> Thanks,<br>> Kazu<br></blockquote><div> </div></div>