From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:53252) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gxf7e-0003Rd-BN for qemu-devel@nongnu.org; Sat, 23 Feb 2019 16:46:15 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gxf7d-00080s-7W for qemu-devel@nongnu.org; Sat, 23 Feb 2019 16:46:14 -0500 References: <20190222162555.13764-1-amagdy.afifi@gmail.com> <20190222162555.13764-2-amagdy.afifi@gmail.com> From: Eric Blake Message-ID: <8f36a26e-7e20-1b9c-18bd-a5ce4b3d9c14@redhat.com> Date: Sat, 23 Feb 2019 15:45:59 -0600 MIME-Version: 1.0 In-Reply-To: <20190222162555.13764-2-amagdy.afifi@gmail.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] riscv: Add proper alignment check and pending 'C' extension upon misa writes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: amagdy.afifi@gmail.com, qemu-devel@nongnu.org Cc: qemu-riscv@nongnu.org, sagark@eecs.berkeley.edu, kbastian@mail.uni-paderborn.de, palmer@sifive.com, mjc@sifive.com, Alistair.Francis@wdc.com On 2/22/19 10:25 AM, amagdy.afifi@gmail.com wrote: > From: ahmed_magdy > > Signed-off-by: ahmed_magdy This appears to be your first contribution to qemu. Welcome to the community! Typically, a Signed-off-by designation should be a proper name (what you would sign a legal document with, as it has a legal significance on your right to contribute the code). Using all lowercase and _ instead of space looks like a username, and while I am not one to tell you it can't be a legal name, it is unusual enough to at least raise my suspicions. Furthermore, your commit message doesn't give any details beyond the "what" in the subject line. The body of the commit message should explain the "why" (what bug are you fixing, how to reproduce it), so that a reviewer stands a chance of determining if the code matches the description you gave, and if the issue you describe really does warrant the inclusion of your patch. You gave a brief "why" in your cover letter: "I'm submiting this patch to properly check the next instruction alignment and scheduale compression extenstion enable upon 'MISA' register writes to later aligned instruction through exporting next instruction 'pc' to riscv cpu state" where it would be wise to include an improved version of that text with this commit proper (since the cover letter does not get applied to git). For that matter, when sending a single patch, a cover letter is optional (it is only mandatory when sending a multi-patch series). For more patch submission hints, see: https://wiki.qemu.org/Contribute/SubmitAPatch -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org