From: Jan Beulich <jbeulich@suse.com>
To: Andrew Cooper <andrew.cooper3@citrix.com>
Cc: "xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>,
"Wei Liu" <wl@xen.org>, "Roger Pau Monné" <roger.pau@citrix.com>
Subject: Re: [PATCH 4/5] x86/ELF: also record FS/GS bases in elf_core_save_regs()
Date: Mon, 28 Sep 2020 15:38:41 +0200 [thread overview]
Message-ID: <26b0de09-98f7-aa27-5348-03dde141f365@suse.com> (raw)
In-Reply-To: <5e95ba48-68dc-81f4-1d62-f3310b0ba091@citrix.com>
On 28.09.2020 15:04, Andrew Cooper wrote:
> On 28/09/2020 13:06, Jan Beulich wrote:
>> Signed-off-by: Jan Beulich <jbeulich@suse.com>
>
> Any idea why this wasn't done before?
No. My only guess is laziness, in the sense of "I'll do it later" and
then forgetting.
> At a minimum, I'd be tempted to
> put a sentence in the commit message saying "no idea why this wasn't
> done before".
Sure, done.
> Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
Thanks.
Jan
next prev parent reply other threads:[~2020-09-28 13:39 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-28 12:04 [PATCH 0/5] x86: introduce read_sregs() and elf_core_save_regs() adjustments Jan Beulich
2020-09-28 12:05 ` [PATCH 1/5] x86: introduce read_sregs() to allow storing to memory directly Jan Beulich
2020-09-28 12:47 ` Andrew Cooper
2020-09-28 13:29 ` Jan Beulich
2020-09-28 14:49 ` Jan Beulich
2020-09-28 15:11 ` Andrew Cooper
2020-09-28 12:05 ` [PATCH 2/5] x86/ELF: don't open-code read_sreg() Jan Beulich
2020-09-28 12:57 ` Andrew Cooper
2020-09-28 12:06 ` [PATCH 3/5] x86/ELF: don't store function pointer in elf_core_save_regs() Jan Beulich
2020-09-28 12:58 ` Andrew Cooper
2020-09-28 12:06 ` [PATCH 4/5] x86/ELF: also record FS/GS bases " Jan Beulich
2020-09-28 13:04 ` Andrew Cooper
2020-09-28 13:38 ` Jan Beulich [this message]
2020-09-28 12:07 ` [PATCH 5/5] x86/ELF: eliminate pointless local variable from elf_core_save_regs() Jan Beulich
2020-09-28 13:06 ` Andrew Cooper
2020-09-28 15:04 ` [PATCH 6/5] x86/ELF: drop unnecessary volatile from asm()-s in elf_core_save_regs() Jan Beulich
2020-09-28 15:15 ` Andrew Cooper
2020-09-28 15:40 ` Jan Beulich
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-all 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=26b0de09-98f7-aa27-5348-03dde141f365@suse.com \
--to=jbeulich@suse.com \
--cc=andrew.cooper3@citrix.com \
--cc=roger.pau@citrix.com \
--cc=wl@xen.org \
--cc=xen-devel@lists.xenproject.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.