Re: [PATCH 15/27] block: use bdev_alignment_offset in part_alignment_offset_show

2022-04-06 Thread Alan Robinson
Hi Christoph, On Wed, Apr 06, 2022 at 06:05:04AM +, Christoph Hellwig wrote: > From: Christoph Hellwig > Subject: [PATCH 15/27] block: use bdev_alignment_offset in > part_alignment_offset_show > > Replace the open coded offset calculation with the proper helper. > This is an ABI change in

Re: [PATCH 12/15] swiotlb: provide swiotlb_init variants that remap the buffer

2022-04-04 Thread Alan Robinson
Hi Christoph, On Mon, Apr 04, 2022 at 05:05:56AM +, Christoph Hellwig wrote: > From: Christoph Hellwig > Subject: [PATCH 12/15] swiotlb: provide swiotlb_init variants that remap > the buffer > > To shared more code between swiotlb and xen-swiotlb, offer a > swiotlb_init_remap interface and

[PATCH v2] tools/libxl: add missing blank in message

2021-07-27 Thread Alan Robinson
Add missing blank giving "an emulation" instead of "anemulation" while making the text a single source line. Signed-off-by: Alan Robinson --- Changed since v1: * text as as single line, requested by Juergen tools/libs/light/libxl_dm.c | 6 ++ 1 file changed, 2 insert

[PATCH] tools/libxl: add missing blank in message

2021-07-23 Thread Alan Robinson
From: Alan Robinson Add trailing blank to first part of concatenated string giving "an emulated" instead of "anemulated". Signed-off-by: Alan Robinson --- tools/libs/light/libxl_dm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/libs/ligh

Re: [Xen-devel] [PATCH v3 5/6] mm: add 'is_special_page' macro...

2020-03-05 Thread Alan Robinson
A typo... On Thu, Mar 05, 2020 at 01:45:03PM +0100, pdurr...@amzn.com wrote: > > PGC_extra pages are intended to hold data structures that are associated > with a domain and my be mapped by that domain. They should not be treated s/my/may/ > as 'normal' guest pages (i.e. RAM or page tables).

Re: [Xen-devel] [PATCH] x86: don't offer Hyper-V option when "PV Shim Exclusive"

2019-12-05 Thread Alan Robinson
On Thu, Dec 05, 2019 at 04:42:08PM +0100, Jan Beulich wrote: > > This only added dead code. Use "if" instead of "depends on" to make > (halfway) clear that other guest options should also go in thae same s/thae/the/ > block. Move the option down such that the shim related options get >

Re: [Xen-devel] [PATCH 3/5] xen/domain: Audit config->max_vcpus during {, arch_}check_domain_config()

2018-10-08 Thread Alan Robinson
On Fri, Oct 05, 2018 at 04:54:47PM +0200, Andrew Cooper wrote: > The purpose of this is to move the auduting to be earlier than > arch_domain_create(). > s/auduting/auditing/ Alan smime.p7s Description: S/MIME cryptographic signature ___ Xen-devel

Re: [Xen-devel] [PATCH] xen/blkfront: correct purging of persistent grants

2018-09-28 Thread Alan Robinson
On Fri, Sep 28, 2018 at 09:28:27AM +0200, Juergen Gross wrote: > Commit a46b53672b2c2e3770b38a4abf90d16364d2584b ("xen/blkfront: cleanup > stale persistent grants") introduced a regression as purged persistent > grants were not pu into the list of free grants again. Correct that. s/pu /put / Alan

Re: [Xen-devel] [PATCH v5 04/16] xen/arm: mm: Remove unused M2P code

2018-03-15 Thread Alan Robinson
Hi Julien, > At the same time move the remaining M2P define just above just above > set_gpfn_from_mfn to keep all the dummy helpers for M2P together. At the same time move the remaining M2P define just above set_gpfn_from_mfn to keep all the dummy helpers for M2P together. can't really

[Xen-devel] [PATCH v2 ] get_maintainers.pl: Avoid THE_REST when files are added or removed

2018-02-22 Thread Alan Robinson
When files are added or removed /dev/null is used as a place holder name in the patch for the absent file. Don't try and find a MAINTAINER for this place holder, it only ever flags and then spams THE REST, behaviour for a real filename is unchanged. Signed-off-by: Alan Robinson <alan.ro

Re: [Xen-devel] [PATCH] get_maintainers.pl: Avoid THE_REST when files are added or removed

2018-02-22 Thread Alan Robinson
Hi Jan, On Thu, Feb 22, 2018 at 02:45:48AM -0700, Jan Beulich wrote: > Oh, I see - it's the file name collection code you change, not the > consuming side, I'm sorry for the noise. However, isn't > omitting the leading / a potential problem then? The leading / is stripped away several lines

Re: [Xen-devel] [PATCH] get_maintainers.pl: Avoid THE_REST when files are added or removed

2018-02-22 Thread Alan Robinson
> > Also, please send patches _to_ the list, _cc_-ing individuals as > needed. Ahh..sorry about that Alan -- Alan Robinson Enterprise Platform Services, Germany Fujitsu Mies-van-der-Rohe-Str. 8, 80807 Muenchen, Deutschland Tel.: +49 (89) 62060 3927 Mob.

[Xen-devel] [PATCH] get_maintainers.pl: Avoid THE_REST when files are added or removed

2018-02-22 Thread Alan Robinson
When files are added or removed /dev/null is used as a place holder name in the patch for the absent file. Don't try and find a MAINTAINER for this place holder, it only ever flags and then spams THE REST. Signed-off-by: Alan Robinson <alan.robin...@ts.fujitsu.com> --- Tested by pret

Re: [Xen-devel] [PATCH 30/30] xen: use the BYTE-based definitions

2018-02-15 Thread Alan Robinson
Hi Philippe, On Thu, Feb 15, 2018 at 09:23:52AM -0300, Philippe Mathieu-Daudé wrote: > > Can I add your R-b tag once fixed? Respin will be: > > +xenstore_write_int(dom, "memory/target", ram_size / K_BYTE); > +xenstore_write_int(vm, "memory", ram_size / M_BYTE); > +