All of lore.kernel.org
 help / color / mirror / Atom feed
From: Doug Goldstein <cardoe@cardoe.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: Tamas K Lengyel <tamas@tklengyel.com>, Keir Fraser <keir@xen.org>,
	Razvan Cojocaru <rcojocaru@bitdefender.com>,
	Andrew Cooper <andrew.cooper3@citrix.com>,
	xen-devel@lists.xen.org, Daniel De Graaf <dgdegra@tycho.nsa.gov>
Subject: Re: [PATCH v7 25/28] build: convert HAS_MEM_PAGING use to Kconfig
Date: Fri, 11 Dec 2015 06:57:36 -0600	[thread overview]
Message-ID: <566AC840.4020909@cardoe.com> (raw)
In-Reply-To: <566AB90502000078000BE974@prv-mh.provo.novell.com>


[-- Attachment #1.1: Type: text/plain, Size: 1135 bytes --]

On 12/11/15 4:52 AM, Jan Beulich wrote:
>>>> On 10.12.15 at 17:48, <cardoe@cardoe.com> wrote:
>> --- a/config/x86_32.mk
>> +++ b/config/x86_32.mk
>> @@ -6,7 +6,6 @@ CONFIG_MIGRATE := y
>>  CONFIG_XCUTILS := y
>>  
>>  HAS_MEM_ACCESS := y
>> -HAS_MEM_PAGING := y
>>  HAS_MEM_SHARING := y
>>  
>>  CFLAGS += -m32 -march=i686
>> --- a/config/x86_64.mk
>> +++ b/config/x86_64.mk
>> @@ -7,7 +7,6 @@ CONFIG_MIGRATE := y
>>  CONFIG_XCUTILS := y
>>  
>>  HAS_MEM_ACCESS := y
>> -HAS_MEM_PAGING := y
>>  HAS_MEM_SHARING := y
>>  
>>  CONFIG_XEN_INSTALL_SUFFIX := .gz
> 
> I'm sure I gave this comment before: Considering that the previous
> patch converted HAS_MEM_ACCESS, the patch context here shows
> that this was an incomplete conversion.
> 
> Jan
> 

That's not the case. HAS_MEM_ACCESS is used in the tools/ directory
currently as well so it has to remain here for now. I was going to look
at what changes could be made there after this series. Likely it'll be
turned on always in the tools stack and just gracefully handle the fact
that the hypervisor might have it disabled.

-- 
Doug Goldstein


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 959 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

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

  reply	other threads:[~2015-12-11 12:57 UTC|newest]

Thread overview: 57+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-10 16:48 [PATCH v7 00/28] Kconfig conversion Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 01/28] build: import Kbuild/Kconfig from Linux 4.3 Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 02/28] MAINTAINERS: add myself for kconfig Doug Goldstein
2015-12-10 17:21   ` Jan Beulich
2015-12-10 17:26     ` Doug Goldstein
2015-12-11  7:50       ` Jan Beulich
2015-12-10 16:48 ` [PATCH v7 03/28] build: build Kconfig and config rules Doug Goldstein
2015-12-14 16:35   ` Jan Beulich
2015-12-14 17:52     ` Doug Goldstein
2015-12-14 17:57       ` [PATCH] squash into 'build: build Kconfig and config rules' Doug Goldstein
2015-12-14 19:25         ` Doug Goldstein
2015-12-14 19:24       ` Doug Goldstein
2015-12-15  7:42       ` [PATCH v7 03/28] build: build Kconfig and config rules Jan Beulich
2015-12-10 16:48 ` [PATCH v7 04/28] build: use generated Kconfig options for Xen Doug Goldstein
2015-12-14 16:37   ` Jan Beulich
2015-12-14 18:00     ` [PATCH] squash into 'build: use generated Kconfig option for Xen' Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 05/28] build: convert HAS_PASSTHROUGH use to Kconfig Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 06/28] build: convert HAS_DEVICE_TREE " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 07/28] build: convert HAS_PCI " Doug Goldstein
2015-12-14 15:52   ` [PATCH] squash into 'build: convert HAS_PCI use to Kconfig' Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 08/28] build: convert HAS_NS16550 use to Kconfig Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 09/28] build: convert HAS_IOPORTS " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 10/28] build: convert HAS_ACPI " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 11/28] build: convert HAS_VIDEO " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 12/28] build: convert HAS_VGA " Doug Goldstein
2015-12-14 13:23   ` Jan Beulich
2015-12-14 15:57     ` [PATCH] squash into 'build: convert HAS_VGA use to Kconfig' Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 13/28] build: convert HAS_CPUFREQ use to Kconfig Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 14/28] build: convert HAS_GDBSX " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 15/28] build: convert HAS_PDX " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 16/28] build: convert HAS_KEXEC / KEXEC " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 17/28] build: convert HAS_ARM_HDLCD " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 18/28] build: convert HAS_CADENCE_UART " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 19/28] build: convert HAS_PL011 " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 20/28] build: convert HAS_EXYNOS4210 " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 21/28] build: convert HAS_OMAP " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 22/28] build: convert HAS_SCIF " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 23/28] build: convert HAS_EHCI " Doug Goldstein
2015-12-14 13:33   ` Jan Beulich
2015-12-14 16:00     ` [PATCH] squash into 'build: convert HAS_EHCI use to Kconfig' Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 24/28] build: convert HAS_MEM_ACCESS use to Kconfig Doug Goldstein
2015-12-11  8:37   ` Razvan Cojocaru
2015-12-10 16:48 ` [PATCH v7 25/28] build: convert HAS_MEM_PAGING " Doug Goldstein
2015-12-11  8:43   ` Razvan Cojocaru
2015-12-11 10:52   ` Jan Beulich
2015-12-11 12:57     ` Doug Goldstein [this message]
2015-12-11 13:23       ` Jan Beulich
2015-12-11 16:15         ` [PATCH] squash into 'build: convert HAS_MEM_ACCESS to use Kconfig' Doug Goldstein
2015-12-14  9:01           ` Jan Beulich
2015-12-14 14:21             ` Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 26/28] build: convert HAS_MEM_SHARING use to Kconfig Doug Goldstein
2015-12-11  8:46   ` Razvan Cojocaru
2015-12-10 16:48 ` [PATCH v7 27/28] build: convert HAS_GICV3 " Doug Goldstein
2015-12-10 16:48 ` [PATCH v7 28/28] build: convert CONFIG_COMPAT " Doug Goldstein
2015-12-10 16:59 ` [PATCH v7 00/28] Kconfig conversion Jan Beulich
2015-12-10 17:14   ` Doug Goldstein
2015-12-14 13:29 ` 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=566AC840.4020909@cardoe.com \
    --to=cardoe@cardoe.com \
    --cc=JBeulich@suse.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=dgdegra@tycho.nsa.gov \
    --cc=keir@xen.org \
    --cc=rcojocaru@bitdefender.com \
    --cc=tamas@tklengyel.com \
    --cc=xen-devel@lists.xen.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.