Re: [PATCH]: PVH: remove FEATURES_PVH macro

2012-11-26 Thread Konrad Rzeszutek Wilk
On Wed, Nov 14, 2012 at 06:19:33PM -0800, Mukesh Rathor wrote: > PVH: remove macro FEATURES_PVH and put PVH strings in the ELFNOTE line, > because there's a null char before FEATURES_PVH and in the FEATURES_PVH > strings since this is not C file ping? Jan had a comment about it to keep the #ifde

Re: [PATCH 2/5] Expand the steal time msr to also contain the consigned time.

2012-11-27 Thread Konrad Rzeszutek Wilk
On Mon, Nov 26, 2012 at 02:36:45PM -0600, Michael Wolf wrote: > Add a consigned field. This field will hold the time lost due to capping or > overcommit. > The rest of the time will still show up in the steal-time field. > > Signed-off-by: Michael Wolf > --- > arch/x86/include/asm/paravirt.h

Re: [PATCH 1/1] mm: Export a function to get vm committed memory

2012-11-27 Thread Konrad Rzeszutek Wilk
n to retrieve guest memory > > commitment. > > This function is also used in Xen self ballooning code. > > > > Signed-off-by: K. Y. Srinivasan > > Acked-by: David Rientjes I am bit late to this party - and back from vacation - so not sure if this is merged in or not.

Re: [PATCH v8 00/46] x86, mm: map ram from top-down with BRK and memblock.

2012-11-27 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:38:37PM -0800, Yinghai Lu wrote: > rebase patchset together tip/x86/mm2 on top of linus v3.7-rc4 > Pardon me for taking so long to respond. Right after you posted it I had a mini-conference and then vacation and I am just now unburrying myself from email avalanche. I w

Re: [PATCH 2/8] mm: frontswap: lazy initialization to allow tmem backends to build/run as modules

2012-11-27 Thread Konrad Rzeszutek Wilk
On Mon, Nov 19, 2012 at 02:25:16PM -0800, Andrew Morton wrote: > On Mon, 19 Nov 2012 08:53:46 +0800 > Bob Liu wrote: > > > On Sat, Nov 17, 2012 at 7:16 AM, Andrew Morton > > wrote: > > > On Wed, 14 Nov 2012 13:57:06 -0500 > > > Konrad Rzeszutek Wilk wr

Re: [PATCH v8 04/46] x86, mm: Move init_memory_mapping calling out of setup.c

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:38:41PM -0800, Yinghai Lu wrote: > Now init_memory_mapping is called two times, later will be called for every > ram ranges. What is 'later'? Can you say in which of the patches it will be called for every RAM range? > > Could put all related init_mem calling together

Re: [PATCH v8 07/46] x86, mm: Find early page table buffer together

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:38:44PM -0800, Yinghai Lu wrote: > We should not do that in every calling of init_memory_mapping. Right. > > At the same time need to move down early_memtest, and could remove > after_bootmem ->'remov

Re: [PATCH v8 03/46] x86, mm: Move down find_early_table_space()

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:38:40PM -0800, Yinghai Lu wrote: > It will need to call split_mem_range(). .. which implies that it needs to call it now, but I could not find the call in your patch. Then realized you meant to do that in the further patches. To avoid this confusion I would recommend you

Re: [PATCH v8 06/46] x86, mm: Change find_early_table_space() paramters

2012-11-28 Thread Konrad Rzeszutek Wilk
s/paramters/parameters/ On Fri, Nov 16, 2012 at 07:38:43PM -0800, Yinghai Lu wrote: > call split_mem_range inside the function. So this looks to introduce an extra split_mem_range which does the same operations on 'mr' that is done in find_early_table_space. Could the find_early_table_space use

Re: [PATCH v8 08/46] x86, mm: Separate out calculate_table_space_size()

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:38:45PM -0800, Yinghai Lu wrote: > It should take physical address range that will need to be mapped. > find_early_table_space should take range that pgt buff should be in. > > Separating page table size calculating and finding early page table to > reduce confusing. Yo

Re: [PATCH v8 12/46] x86, mm: use pfn_range_is_mapped() with CPA

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:38:49PM -0800, Yinghai Lu wrote: > We are going to map ram only, so under max_low_pfn_mapped, > between 4g and max_pfn_mapped does not mean mapped at all. I think I know what you are saying but I am having a hard time parsing it. Is this what you mean? "We check to see

Re: [PATCH v8 13/46] x86, mm: use pfn_range_is_mapped() with gart

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:38:50PM -0800, Yinghai Lu wrote: > We are going to map ram only, so under max_low_pfn_mapped, > between 4g and max_pfn_mapped does not mean mapped at all. Perhaps the re-write I mentioned in the earlier email could help here. > > Use pfn_range_is_mapped() directly. >

Re: [PATCH v8 14/46] x86, mm: use pfn_range_is_mapped() with reserve_initrd

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:38:51PM -0800, Yinghai Lu wrote: > We are going to map ram only, so under max_low_pfn_mapped, > between 4g and max_pfn_mapped does not mean mapped at all. > > Use pfn_range_is_mapped() to find out if range is mapped for initrd. > > That could happen bootloader put initr

Re: [PATCH v8 15/46] x86, mm: Only direct map addresses that are marked as E820_RAM

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:38:52PM -0800, Yinghai Lu wrote: > From: Jacob Shin > > Currently direct mappings are created for [ 0 to max_low_pfn< and [ 4GB to max_pfn< backed by actual DRAM. This is fine for holes under 4GB which are covered > by fixed and variable range MTRRs to be UC. However, w

Re: [PATCH v8 21/46] x86, mm: setup page table in top-down

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:38:58PM -0800, Yinghai Lu wrote: > Get pgt_buf early from BRK, and use it to map PMD_SIZE from top at first. > Then use mapped pages to map more ranges below, and keep looping until > all pages get mapped. > > alloc_low_page will use page from BRK at first, after that bu

Re: [PATCH v8 22/46] x86, mm: Remove early_memremap workaround for page table accessing on 64bit

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:38:59PM -0800, Yinghai Lu wrote: > We try to put page table high to make room for kdump, and at that time > those ranges are not mapped yet, and have to use ioremap to access it. ^^^ -> were -> had > > Now after patch that pre-map page tab

Re: [PATCH v8 23/46] x86, mm: Remove parameter in alloc_low_page for 64bit

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:39:00PM -0800, Yinghai Lu wrote: > Now all page table buf are pre-mapped, and could use virtual address directly. ^-> we can > So don't need to remember physical address anymore. > > Remove that phys pointer in alloc_low_

Re: [PATCH v8 29/46] x86, mm: only call early_ioremap_page_table_range_init() once

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:39:06PM -0800, Yinghai Lu wrote: > On 32bit, before patcheset that only set page table for ram, we only > call that one time. Um.. Which patchset? x86, mm: Only direct map addresses that are marked as E820_RAM? > > Now, we are calling that during every init_memory_map

Re: [PATCH v8 34/46] x86, mm: Add check before clear pte above max_low_pfn on 32bit

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:39:11PM -0800, Yinghai Lu wrote: > During test patch that adjust page_size_mask to map small range ram with > big page size, found page table is setup wrongly for 32bit. And Which patch is that? x86, mm: Add global page_size_mask and probe one time only Can you include

Re: [PATCH v8 46/46] x86, mm: Let "memmap=" take more entries one time

2012-11-28 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 07:39:23PM -0800, Yinghai Lu wrote: > Current "memmap=" only can take one entry every time. > when we have more entries, we have to use memmap= for each of them. > > For pxe booting, we have command line length limitation, those extra > "memmap=" would waste too much space.

Re: [PATCH V1 1/2] Xen acpi memory hotplug driver

2012-11-28 Thread Konrad Rzeszutek Wilk
On Wed, Nov 21, 2012 at 11:45:04AM +, Liu, Jinsong wrote: > >From 630c65690c878255ce71e7c1172338ed08709273 Mon Sep 17 00:00:00 2001 > From: Liu Jinsong > Date: Tue, 20 Nov 2012 21:14:37 +0800 > Subject: [PATCH 1/2] Xen acpi memory hotplug driver > > Xen acpi memory hotplug consists of 2 logic

Re: [PATCH v8 00/46] x86, mm: map ram from top-down with BRK and memblock.

2012-11-28 Thread Konrad Rzeszutek Wilk
On Tue, Nov 27, 2012 at 04:17:55PM -0500, Konrad Rzeszutek Wilk wrote: > On Fri, Nov 16, 2012 at 07:38:37PM -0800, Yinghai Lu wrote: > > rebase patchset together tip/x86/mm2 on top of linus v3.7-rc4 > > > > Pardon me for taking so long to respond. Right after you pos

Re: [PATCH v8 00/46] x86, mm: map ram from top-down with BRK and memblock.

2012-11-28 Thread Konrad Rzeszutek Wilk
On Wed, Nov 28, 2012 at 11:47:51AM -0800, Yinghai Lu wrote: > On Wed, Nov 28, 2012 at 11:35 AM, Konrad Rzeszutek Wilk > wrote: > > > > Have done so. I really like how the top-down mechanism works. It is pretty > > neat! > > > > Yinghai, I had mostly just c

Re: [PATCH 00/19 v4] Improve IRQ remapping abstraction in x86 core code

2012-11-28 Thread Konrad Rzeszutek Wilk
On Tue, Nov 20, 2012 at 02:12:37PM +0100, Joerg Roedel wrote: > Hi, > > here is the fourth version of the patch-set to improve the abstraction > of interrupt remapping in the x86 core code. A more detailed description > can be found in the original post at: Are these patches available on some git

Re: [PATCH 00/19 v4] Improve IRQ remapping abstraction in x86 core code

2012-11-28 Thread Konrad Rzeszutek Wilk
://lkml.org/lkml/2012/8/7/317 > I looked at the patches and you can also add Reviewed-by: Konrad Rzeszutek Wilk If you have a git tree I would like to test them as well. Thx. > Changes from v3->v4: > > * Rebased to v3.7-rc6 > * Added Acked-bys > * Fixed

Re: linux-next: build warning after merge of the xen-two tree

2012-11-29 Thread Konrad Rzeszutek Wilk
On Thu, Nov 29, 2012 at 02:25:30PM +1100, Stephen Rothwell wrote: > Hi Konrad, > > After merging the xen-two tree, today's linux-next build (x86_64 > allmodconfig) produced this warning: > > arch/x86/xen/enlighten.c: In function 'init_hvm_pv_info': > arch/x86/xen/enlighten.c:1617:16: warning: unu

Re: [PATCH 3/5] xen-blkfront: switch from llist to list

2013-03-19 Thread Konrad Rzeszutek Wilk
nk #3 FAILED at 105. Hunk #4 FAILED at 371. patch: **** malformed patch at line 172: ork) and idea why? Could you resent the patch as an attachment please? > > Signed-off-by: Roger Pau Monné > [Part of the description] > Signed-off-by: Konrad Rzeszutek Wilk > Cc: xen-de...@lis

Re: [PATCH 3/5] xen-blkfront: switch from llist to list

2013-03-19 Thread Konrad Rzeszutek Wilk
nné CC: sta...@vger.kernel.org [v1: Redid the git commit description] Signed-off-by: Konrad Rzeszutek Wilk diff --git a/drivers/block/xen-blkfront.c b/drivers/block/xen-blkfront.c index 9620644..97324cd1 100644 --- a/drivers/block/xen-blkfront.c +++ b/drivers/block/xen-blkfront.c @@

Re: [PATCH 1/5] xen-blkback: don't store dev_bus_addr

2013-03-19 Thread Konrad Rzeszutek Wilk
the 'seg[i].buf' to be 'seg[i].offset' as it makes much more sense - as we use that value in bio_add_page which as the fourth argument expects the offset. We hadn't used the physical address as part of this at all. Signed-off-by: Roger P

Re: [PATCH RFC 12/12] xen-block: implement indirect descriptors

2013-03-19 Thread Konrad Rzeszutek Wilk
On Mon, Mar 18, 2013 at 06:06:38PM +0100, Roger Pau Monné wrote: > On 28/02/13 11:28, Roger Pau Monne wrote: > > Indirect descriptors introduce a new block operation > > (BLKIF_OP_INDIRECT) that passes grant references instead of segments > > in the request. This grant references are filled with ar

Re: [Xen-devel] [PATCH 1/5] xen-blkback: don't store dev_bus_addr

2013-03-19 Thread Konrad Rzeszutek Wilk
On Tue, Mar 19, 2013 at 02:55:56PM +, Jan Beulich wrote: > >>> On 19.03.13 at 15:32, Konrad Rzeszutek Wilk > >>> wrote: > > On Mon, Mar 18, 2013 at 05:49:32PM +0100, Roger Pau Monne wrote: > >> dev_bus_addr returned in the grant ref map operation is t

Re: [PATCH v2 3/4] introduce zero-filled page stat count

2013-03-19 Thread Konrad Rzeszutek Wilk
On Sun, Mar 17, 2013 at 8:58 AM, Ric Mason wrote: > Hi Konrad, > > On 03/16/2013 09:06 PM, Konrad Rzeszutek Wilk wrote: >> >> On Thu, Mar 14, 2013 at 06:08:16PM +0800, Wanpeng Li wrote: >>> >>> Introduce zero-filled page statistics to monitor the number

Re: [PATCH v2 1/4] introduce zero filled pages handler

2013-03-19 Thread Konrad Rzeszutek Wilk
On Sat, Mar 16, 2013 at 2:24 PM, Dan Magenheimer wrote: >> From: Konrad Rzeszutek Wilk [mailto:kon...@darnok.org] >> Subject: Re: [PATCH v2 1/4] introduce zero filled pages handler >> >> > + >> > + for (pos = 0; pos < PAGE_SIZE / sizeof(*page);

Re: [Xen-devel] [PATCH 1/5] xen-blkback: don't store dev_bus_addr

2013-03-19 Thread Konrad Rzeszutek Wilk
d the physical address as part of this at all. Signed-off-by: Roger Pau Monné Cc: Konrad Rzeszutek Wilk Cc: xen-de...@lists.xen.org [v1: s/buf/offset/] Signed-off-by: Konrad Rzeszutek Wilk diff --git a/drivers/block/xen-blkback/blkback.c b/drivers/block/xen-blkback/b

[PATCH] xen/acpi-processor: Don't dereference struct acpi_processor on all CPUs.

2013-03-19 Thread Konrad Rzeszutek Wilk
Call Trace: [] ? read_acpi_id+0x12b/0x12b [xen_acpi_processor] [] do_one_initcall+0x12a/0x180 [] load_module+0x1cd3/0x2870 [] ? ddebug_proc_open+0xc0/0xc0 [] sys_init_module+0xd7/0x120 [] system_call_fastpath+0x16/0x1b on some machines. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/xen/xen-acpi-p

[PATCH] intel-pstate: Use #defines instead of hard-coded values.

2013-03-20 Thread Konrad Rzeszutek Wilk
They are defined in coreboot (MSR_PLATFORM) and the other one is already defined in msr-index.h. Lets use those. CC: rafael.j.wyso...@intel.com CC: dirk.j.brande...@intel.com Signed-off-by: Konrad Rzeszutek Wilk --- arch/x86/include/uapi/asm/msr-index.h | 1 + drivers/cpufreq/intel_pstate.c

[GIT PULL] (xen) stable/for-jens-3.9

2013-03-21 Thread Konrad Rzeszutek Wilk
Hey Jens, Please git pull the following branch: git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git stable/for-jens-3.9 which has a bunch of fixes. They vary from being able to deal with unknown requests, overflow in statistics, compile warnings, bug in the error path, removal of unne

Re: [PATCH] mm/x86: Patch out arch_flush_lazy_mmu_mode() when running on bare metal

2013-03-21 Thread Konrad Rzeszutek Wilk
On Wed, Mar 13, 2013 at 09:44:43AM -0400, Josh Boyer wrote: > On Wed, Mar 13, 2013 at 09:25:44AM -0400, Boris Ostrovsky wrote: > > On 03/01/2013 07:14 AM, Josh Boyer wrote: > > >On Thu, Feb 28, 2013 at 04:52:20PM -0800, H. Peter Anvin wrote: > > >>On 02/28/2013 04:42 PM, Josh Boyer wrote: > > >>>On

Re: [PATCH v2] Make frontswap+cleancache and its friend be modularized.

2013-02-04 Thread Konrad Rzeszutek Wilk
On Mon, Feb 04, 2013 at 06:21:12PM -0600, Ric Mason wrote: > On Mon, 2013-02-04 at 09:14 -0600, Seth Jennings wrote: > > On 02/03/2013 02:52 AM, Ric Mason wrote: > > > Hi Konrad, > > > On Fri, 2013-02-01 at 15:22 -0500, Konrad Rzeszutek Wilk wrote: > > > &

Re: linux-next: build failure after merge of the xen-two tree

2013-02-05 Thread Konrad Rzeszutek Wilk
On Mon, Feb 04, 2013 at 03:14:56PM +1100, Stephen Rothwell wrote: > Hi Konrad, > > After merging the xen-two tree, today's linux-next build (x86_64 > allmodconfig) failed like this: > > arch/x86/xen/mmu.c: In function 'xen_init_mmu_ops': > arch/x86/xen/mmu.c:2213:10: error: 'struct x86_init_ops'

Re: [PATCH] PVH: remove code to map iomem from guest

2013-02-06 Thread Konrad Rzeszutek Wilk
On Wed, Jan 30, 2013 at 02:55:29PM -0800, Mukesh Rathor wrote: > It was decided during xen patch review that xen map the iomem > transparently, so remove xen_set_clr_mmio_pvh_pte() and the sub > hypercall PHYSDEVOP_map_iomem. > G.. No Signed-off-by?? > --- > arch/x86/xen/mmu.c

Re: [PATCH] PVH linux: Use ballooning to allocate grant table pages

2013-02-06 Thread Konrad Rzeszutek Wilk
On Thu, Jan 31, 2013 at 06:30:15PM -0800, Mukesh Rathor wrote: > This patch fixes a fixme in Linux to use alloc_xenballooned_pages() to > allocate pfns for grant table pages instead of kmalloc. This also > simplifies add to physmap on the xen side a bit. Pulled this. > > Signed-off-by: Mukesh Rat

Re: [PATCH] xen-pciback: notify hypervisor about devices intended to be assigned to guests

2013-02-06 Thread Konrad Rzeszutek Wilk
On Wed, Feb 06, 2013 at 04:58:01PM +, Jan Beulich wrote: > For MSI-X capable devices the hypervisor wants to write protect the > MSI-X table and PBA, yet it can't assume that resources have been > assigned to their final values at device enumeration time. Thus have > pciback do that notificatio

[GIT PULL] (xen) stable/for-linus-3.8-rc6-tag

2013-02-06 Thread Konrad Rzeszutek Wilk
Hey Linus, Please git pull the following tag: git pull git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git stable/for-linus-3.8-rc6-tag Which has two fixes. One is a security fix wherein we would spam the kernel printk buffer if one of the guests was misbehaving. The other is much tame

Re: [PATCH] staging/zcache: Fix/improve zcache writeback code, tie to a config option

2013-02-12 Thread Konrad Rzeszutek Wilk
On Mon, Feb 11, 2013 at 01:43:58PM -0800, Dan Magenheimer wrote: > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > > So, how about this, please draw up a specific plan for how you are going > > to get this code out of drivers/staging/ I want to see the steps > > involved, who is going to

Re: [PATCH v5 4/8] staging: zcache: fix pers_pageframes|_max aren't exported in debugfs

2013-04-02 Thread Konrad Rzeszutek Wilk
patch add pers_pageframes|_max back. Duh! Thanks for spotting. Reviewed-by: Konrad Rzeszutek Wilk > > Signed-off-by: Wanpeng Li > --- > drivers/staging/zcache/debug.c |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/drivers/staging/zcache/de

Re: [PATCH v5 5/8] staging: zcache: fix zcache writeback in debugfs

2013-04-02 Thread Konrad Rzeszutek Wilk
that when I did the patches, but I wonder how I missed this. Ah, now I remember - I did a silly patch by adding in #define CONFIG_ZCACHE_WRITEBACK in the zcache-main.c code, but forgot to try it out here. Thank you for spotting and fixing it. Reviewed-by: Konrad Rzeszutek Wilk > Signed-o

Re: [PATCH v5 6/8] staging: zcache: fix static variables defined in debug.h but used in mutiple C files

2013-04-02 Thread Konrad Rzeszutek Wilk
On Mon, Apr 1, 2013 at 10:46 PM, Wanpeng Li wrote: > After commit 95bdaee214 ("zcache: Move debugfs code out of zcache-main.c > file") > be merged, most of knods in zcache debugfs just export zero since these > variables > are defined in debug.h but use in multiple C files zcache-main.c and debu

Re: [PATCH v5 7/8] staging: zcache: introduce zero-filled page stat count

2013-04-02 Thread Konrad Rzeszutek Wilk
> --- a/drivers/staging/zcache/zcache-main.c > +++ b/drivers/staging/zcache/zcache-main.c > @@ -176,6 +176,8 @@ ssize_t zcache_pers_ate_eph; > ssize_t zcache_pers_ate_eph_failed; > ssize_t zcache_evicted_eph_zpages; > ssize_t zcache_evicted_eph_pageframes; > +ssize_t zcache_zero_filled_pages; >

Re: [PATCH] xen: Don't call arch_trigger_all_cpu_backtrace in dom0(pvm)

2013-04-03 Thread Konrad Rzeszutek Wilk
On Wed, Apr 03, 2013 at 08:00:37PM +0800, Zhenzhong Duan wrote: > > On 2013-04-01 20:41, Konrad Rzeszutek Wilk wrote: > >On Mon, Apr 01, 2013 at 01:26:34PM +0800, Zhenzhong Duan wrote: > >>On 2013-03-29 21:46, Konrad Rzeszutek Wilk wrote: > >>>On Fri, Mar 29, 20

Re: [PATCH v2] drivers/block/xen-blkback: preq.dev is used without initialized

2013-04-03 Thread Konrad Rzeszutek Wilk
On Wed, Apr 03, 2013 at 03:34:16PM +0100, Jan Beulich wrote: > >>> On 03.04.13 at 15:56, William Dauchy wrote: > > On Wed, Apr 3, 2013 at 3:42 PM, Jan Beulich wrote: > >> ChangeLog-3.8.3 for example has > > > > oh sorry, you are right. I wasn't looking is the 3.8.x branch. > > The thing is, the

Re: [ 34/77] xen/blkback: Dont trust the handle from the frontend.

2013-04-03 Thread Konrad Rzeszutek Wilk
On Wed, Apr 03, 2013 at 09:01:06AM -0700, Greg Kroah-Hartman wrote: > On Wed, Apr 03, 2013 at 04:01:54PM +0200, William Dauchy wrote: > > On Tue, Mar 12, 2013 at 11:10 PM, Greg Kroah-Hartman > > wrote: > > >> > >> IOW I don't see why this got proposed for stable at all. > > >> > > > > >> > > So, y

Re: [PATCH v6 2/3] staging: zcache: introduce zero-filled page stat count

2013-04-03 Thread Konrad Rzeszutek Wilk
On Wed, Apr 3, 2013 at 6:16 AM, Wanpeng Li wrote: > Introduce zero-filled page statistics to monitor the number of > zero-filled pages. > > Acked-by: Dan Magenheimer > Signed-off-by: Wanpeng Li Reviewed-by: Konrad Rzeszutek Wilk > --- > drivers/staging/zcache

Re: [PATCH v6 1/3] staging: zcache: fix static variables defined in debug.h but used in mutiple C files

2013-04-03 Thread Konrad Rzeszutek Wilk
ple C files zcache-main.c and > debug.c, > in this case variables can't be treated as shared variables. > > Signed-off-by: Wanpeng Li Reviewed-by: Konrad Rzeszutek Wilk -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message

Re: [PATCH v3 21/22] x86, mm: Make init_mem_mapping be able to be called several times

2013-04-05 Thread Konrad Rzeszutek Wilk
Still only load_cr3 one time, otherwise we would break xen 64bit again. Nope. It should be fixed now. > > Signed-off-by: Yinghai Lu > Cc: Pekka Enberg > Cc: Jacob Shin > Cc: Konrad Rzeszutek Wilk > --- > arch/x86/include/asm/pgtable.h | 2 +- > arch/x86/ke

[PATCH 3/4] x86/xen/store_gdt: Remove the pvops variant of store_gdt.

2013-04-05 Thread Konrad Rzeszutek Wilk
platforms. Signed-off-by: Konrad Rzeszutek Wilk --- arch/x86/include/asm/paravirt.h | 4 arch/x86/include/asm/paravirt_types.h | 2 +- arch/x86/kernel/acpi/sleep.c | 2 +- arch/x86/kernel/doublefault_32.c | 2 +- arch/x86/kernel/paravirt.c| 1 - arch/x86/kvm/vmx.c

[PATCH 4/4] x86/wakeup/sleep: Use pvops functions for changing GDT entries.

2013-04-05 Thread Konrad Rzeszutek Wilk
The available 32-bit TSS (09h), which is redefined as the available 64-bit TSS." Without this, on Xen, where the GDT is available as R/O (to protect the hypervisor from the guest modifying it), we end up with a pagetable fault. Signed-off-by: Konrad Rzeszutek Wilk --- arch/x86/power/cpu.c | 9

[RFC PATCH] axe the store_gdt() pvops call. (v1)

2013-04-05 Thread Konrad Rzeszutek Wilk
arch/x86/kvm/vmx.c| 2 +- arch/x86/power/cpu.c | 13 +++-- arch/x86/xen/enlighten.c | 1 - 11 files changed, 11 insertions(+), 23 deletions(-) Konrad Rzeszutek Wilk (3): x86/gdt/64-bit: store/load GDT for ACPI S3 or hibernate/resu

[PATCH 1/4] x86/gdt/64-bit: store/load GDT for ACPI S3 or hibernate/resume path is not needed.

2013-04-05 Thread Konrad Rzeszutek Wilk
the same physical location as the the restored kernel. Note that the hibernation path assumes the GDT is correct during its 'restore_registers'. The assumption in the code is that the restored image is the same as saved - meaning we are not trying to restore an different kernel in the virtual add

[PATCH 2/4] x86/gdt/i386: store/load GDT for ACPI S3 or hibernation/resume path is not needed

2013-04-05 Thread Konrad Rzeszutek Wilk
ernel has already loaded the GDT which is at the same physical location as the the restored kernel. Note that the hibernation path assumes the GDT is correct during its 'restore_registers'. The assumption in the code is that the restored image is the same as saved - meaning we are not t

Re: [PATCHv3 1/6] debugfs: add get/set for atomic types

2013-01-29 Thread Konrad Rzeszutek Wilk
On Mon, Jan 28, 2013 at 03:49:22PM -0600, Seth Jennings wrote: > debugfs currently lack the ability to create attributes > that set/get atomic_t values. > > This patch adds support for this through a new > debugfs_create_atomic_t() function. > > Signed-off-by: Seth Jennings > Acked-by: Greg Kroa

Re: [PATCH 2/8] mm: frontswap: lazy initialization to allow tmem backends to build/run as modules

2013-01-30 Thread Konrad Rzeszutek Wilk
On Tue, Nov 27, 2012 at 04:26:17PM -0500, Konrad Rzeszutek Wilk wrote: > On Mon, Nov 19, 2012 at 02:25:16PM -0800, Andrew Morton wrote: > > On Mon, 19 Nov 2012 08:53:46 +0800 > > Bob Liu wrote: > > > > > On Sat, Nov 17, 2012 at 7:16 AM, Andrew Morton > > >

Re: [PATCH 1/8] mm: cleancache: lazy initialization to allow tmem backends to build/run as modules

2013-01-30 Thread Konrad Rzeszutek Wilk
On Fri, Nov 16, 2012 at 03:10:49PM -0800, Andrew Morton wrote: > On Wed, 14 Nov 2012 13:57:05 -0500 > Konrad Rzeszutek Wilk wrote: > > > From: Dan Magenheimer > > > > With the goal of allowing tmem backends (zcache, ramster, Xen tmem) to be > > built/loaded as

Re: [PATCHv2 1/9] staging: zsmalloc: add gfp flags to zs_create_pool

2013-01-30 Thread Konrad Rzeszutek Wilk
ap backends. > > How about this? > > From 157a3edf49feb93be0595574beb153b322ddf7d2 Mon Sep 17 00:00:00 2001 > From: Minchan Kim > Date: Mon, 28 Jan 2013 11:34:00 +0900 > Subject: [PATCH] frontswap: Get rid of swap_lock dependency > > Frontswap initializa

Re: [PATCH] staging: zsmalloc: remove unused pool name

2013-01-30 Thread Konrad Rzeszutek Wilk
On Wed, Jan 30, 2013 at 06:21:59PM +0100, Greg Kroah-Hartman wrote: > On Wed, Jan 30, 2013 at 09:41:55AM -0600, Seth Jennings wrote: > > On 01/30/2013 09:36 AM, Seth Jennings wrote:> zs_create_pool() > > currently takes a name argument which is > > > never used in any useful way. > > > > > > This p

Re: [PATCH] staging: zsmalloc: remove unused pool name

2013-01-31 Thread Konrad Rzeszutek Wilk
> > > {sigh} you just made me have to edit your patch by hand, you now owe me > > > a beer... > > > > > Should we codify that :-) > > > > > > diff --git a/Documentation/SubmittingPatches > > b/Documentation/SubmittingPatches > > index c379a2a..f879c60 100644 > > --- a/Documentation/SubmittingPa

Re: [PATCH] zsmalloc: Fix TLB coherency and build problem

2013-02-01 Thread Konrad Rzeszutek Wilk
loc: add page table mapping method > [2] https://github.com/spartacus06/zsmapbench > > Cc: sta...@vger.kernel.org > Cc: Dan Magenheimer > Cc: Russell King > Cc: Konrad Rzeszutek Wilk Acked-by: Konrad Rzeszutek Wilk > Cc: Nitin Gupta > Cc: Seth Jennings > Reported-by: Mat

[PATCH v2] Make frontswap+cleancache and its friend be modularized.

2013-02-01 Thread Konrad Rzeszutek Wilk
frontswap: lazy initialization to allow tmem backends to build/run as modules staging: zcache: enable ramster to be built/loaded as a module xen: tmem: enable Xen tmem shim to be built/loaded as a module Konrad Rzeszutek Wilk (10): frontswap: Make frontswap_init use a poi

[PATCH 04/15] cleancache: Make cleancache_init use a pointer for the ops

2013-02-01 Thread Konrad Rzeszutek Wilk
Instead of using a backend_registered to determine whether a backend is enabled. This allows us to remove the backend_register check and just do 'if (cleancache_ops)' [v1: Rebase on top of b97c4b430b0a405a57c78607b520d8000329e259 (ramster->zcache move] Signed-off-by: Konrad R

[PATCH 08/15] xen/tmem: Remove the subsys call.

2013-02-01 Thread Konrad Rzeszutek Wilk
xen_selfballoon_init ends up being exported and can be called by the tmem module. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/xen/xen-selfballoon.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/xen/xen-selfballoon.c b/drivers/xen/xen-selfballoon.c index 6965e9b..f2ef569 100644 --- a

[PATCH 13/15] frontswap: Get rid of swap_lock dependency

2013-02-01 Thread Konrad Rzeszutek Wilk
ded a check for !map] Signed-off-by: Konrad Rzeszutek Wilk squash --- include/linux/frontswap.h | 6 +++--- mm/frontswap.c| 12 +--- mm/swapfile.c | 7 ++- 3 files changed, 18 insertions(+), 7 deletions(-) diff --git a/include/linux/frontswap.h b/include/lin

[PATCH 02/15] mm: frontswap: lazy initialization to allow tmem backends to build/run as modules

2013-02-01 Thread Konrad Rzeszutek Wilk
s per Seth Jennings suggestions] [v2: Removed FRONTSWAP_HAS_.. ] [v3: Fix up per Bob Liu recommendations] [v4: Fix up per Andrew's comments] Signed-off-by: Konrad Rzeszutek Wilk --- mm/frontswap.c | 95 +++--- 1 file changed, 85 insertions(+

[PATCH 07/15] xen: tmem: enable Xen tmem shim to be built/loaded as a module

2013-02-01 Thread Konrad Rzeszutek Wilk
ned-off-by: Dan Magenheimer [v1: Removed the [CLEANCACHE|FRONTSWAP]_HAS_LAZY_INIT ifdef] Signed-off-by: Konrad Rzeszutek Wilk --- drivers/xen/Kconfig | 4 ++-- drivers/xen/tmem.c| 38 +- drivers/xen/xen-selfballoon.c | 13 +++-- i

[PATCH 06/15] staging: zcache: enable zcache to be built/loaded as a module

2013-02-01 Thread Konrad Rzeszutek Wilk
|FRONTSWAP]_HAS_LAZY_INIT ifdef] [v3: Rebased on top of ramster->zcache move] [v4: Redid the Makefile] [v5: s/ZCACHE2/ZCACHE/] Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/Kconfig | 6 ++--- drivers/staging/zcache/Makefile | 11 - drivers/staging/zcache/tme

[PATCH 05/15] staging: zcache: enable ramster to be built/loaded as a module

2013-02-01 Thread Konrad Rzeszutek Wilk
should be considered deprecated. Note that module unload is explicitly not yet supported. Signed-off-by: Dan Magenheimer [v1: Fixed compile issues since ramster_init now has four arguments] [v2: Fixed rebase on ramster->zcache move] Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zca

[PATCH 10/15] frontswap: Use static_key instead of frontswap_enabled and frontswap_ops

2013-02-01 Thread Konrad Rzeszutek Wilk
being NOPs if the backend has not yet registered. Signed-off-by: Konrad Rzeszutek Wilk --- include/linux/frontswap.h | 17 -- mm/frontswap.c| 58 --- 2 files changed, 34 insertions(+), 41 deletions(-) diff --git a/include/linux

[PATCH 14/15] zcache/tmem: Better error checking on frontswap_register_ops return value.

2013-02-01 Thread Konrad Rzeszutek Wilk
In the past it either used to be NULL or the "older" backend. Now we also return -Exx error codes. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/zcache-main.c | 5 - drivers/xen/tmem.c | 5 - 2 files changed, 8 insertions(+), 2 deletion

[PATCH 11/15] cleancache: Remove the check for cleancache_enabled.

2013-02-01 Thread Konrad Rzeszutek Wilk
Rzeszutek Wilk --- include/linux/cleancache.h | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/include/linux/cleancache.h b/include/linux/cleancache.h index 3af5ea8..dfa1ccb 100644 --- a/include/linux/cleancache.h +++ b/include/linux/cleancache.h @@ -74,14 +74,12

[PATCH 09/15] frontswap: Remove the check for frontswap_enabled.

2013-02-01 Thread Konrad Rzeszutek Wilk
functions: frontswap_init and frontswap_invalidate_area can be called anytime - they queue up which of the swap devices are active and can use the frontswap API - once the backend is loaded. As such there is no need to check for 'frontswap_enabled' at all. Signed-off-by: Konrad Rzeszutek Wilk --- incl

[PATCH 12/15] cleancache: Use static_key instead of cleancache_ops and cleancache_enabled.

2013-02-01 Thread Konrad Rzeszutek Wilk
. The rest of the cleancache functions end up being NOPs when the backend has not yet registered. Signed-off-by: Konrad Rzeszutek Wilk --- include/linux/cleancache.h | 16 +- include/linux/frontswap.h | 2 +- mm/cleancache.c| 53

[PATCH 03/15] frontswap: Make frontswap_init use a pointer for the ops.

2013-02-01 Thread Konrad Rzeszutek Wilk
This simplifies the code in the frontswap - we can get rid of the 'backend_registered' test and instead check against frontswap_ops. [v1: Rebase on top of 703ba7fe5e085f2c85eeb451c2ac13cf275c7cb2 (ramster->zcache move] Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/

[PATCH 15/15] xen/tmem: Add missing %s in the printk statement.

2013-02-01 Thread Konrad Rzeszutek Wilk
Seems that it got lost. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/xen/tmem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/xen/tmem.c b/drivers/xen/tmem.c index 2f939e5..4f3ff99 100644 --- a/drivers/xen/tmem.c +++ b/drivers/xen/tmem.c @@ -401,7 +401,7

[PATCH 01/15] mm: cleancache: lazy initialization to allow tmem backends to build/run as modules

2013-02-01 Thread Konrad Rzeszutek Wilk
Signed-off-by: Dan Magenheimer [v1: Minor fixes: used #define for some values and bools] [v2: Removed CLEANCACHE_HAS_LAZY_INIT] [v3: Added more comments, added a lock for [shared_|]fs_poolid_map] Signed-off-by: Konrad Rzeszutek Wilk --- mm/cleancac

[RFC PATCH] Various fixes for zcache.

2013-02-01 Thread Konrad Rzeszutek Wilk
https://lkml.org/lkml/2013/2/1/542 This patchset is just cleaning the zcache driver a bit. I was hoping that the end goal would be less code but with the header file ('debug.h') that did not happend. But I can axe it some more - on the next iteration. Konrad Rzeszutek Wilk (16): zca

[PATCH 01/16] zcache: s/int/bool/ on the various options.

2013-02-01 Thread Konrad Rzeszutek Wilk
There are so many, but this allows us to at least have them right in as bool. [v1: Rebase on ramster->zcache move] Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/zcache-main.c | 46 ++-- 1 file changed, 23 insertions(+), 23 deletions(-) d

[PATCH 06/16] zcache: Make the debug code use pr_debug

2013-02-01 Thread Konrad Rzeszutek Wilk
as if you are debugging this driver you would be using 'debug' on the command line anyhow - and this would dump the debug data on the proper loglevel. While at it also remove the unconditional #define ZCACHE_DEBUG. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/zca

[PATCH 04/16] zcache: The last of the atomic reads has now an accessory function.

2013-02-01 Thread Konrad Rzeszutek Wilk
And now we can move the code to its own file. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/zcache-main.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/drivers/staging/zcache/zcache-main.c b/drivers/staging/zcache/zcache-main.c index

[PATCH 07/16] zcache: Move debugfs code out of zcache-main.c file.

2013-02-01 Thread Konrad Rzeszutek Wilk
Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/Makefile | 3 +- drivers/staging/zcache/debug.c | 113 drivers/staging/zcache/debug.h | 183 ++ drivers/staging/zcache/zcache-main.c | 247

[PATCH 09/16] zcache: Move the last of the debugfs counters out

2013-02-01 Thread Konrad Rzeszutek Wilk
We now have in zcache-main only the counters that are are not debugfs related. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/debug.h | 80 +++- drivers/staging/zcache/zcache-main.c | 71 2 files changed, 87

[PATCH 16/16] zcache/zbud: Fix __init mismatch

2013-02-01 Thread Konrad Rzeszutek Wilk
the annotation of zbud_init is wrong. And this fixes it. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/zbud.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/zcache/zbud.c b/drivers/staging/zcache/zbud.c index e139cd6..9aa5bd8 100644 --- a

[PATCH 15/16] ramster: Fix compile warnings due to usage of debugfs_create_size_t

2013-02-01 Thread Konrad Rzeszutek Wilk
We get tons of "note: expected ‘size_t *’ but argument is of type ‘long int *’" warnings. This fixes it. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/ramster/ramster.c | 34 1 file changed, 17 insertions(+), 17 deletions(-) di

[PATCH 05/16] zcache: Fix compile warnings due to usage of debugfs_create_size_t

2013-02-01 Thread Konrad Rzeszutek Wilk
up using 'unsigned' or 'unsigned long' instead of 'ssize_t'. So lets fix this up and use the proper type. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/zcache-main.c | 135 ++- 1 file changed, 68 insertions(+),

[PATCH 14/16] zcache/zbud: Provide the accessory functions for counter decrease.

2013-02-01 Thread Konrad Rzeszutek Wilk
Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/zbud.c | 38 -- 1 file changed, 24 insertions(+), 14 deletions(-) diff --git a/drivers/staging/zcache/zbud.c b/drivers/staging/zcache/zbud.c index cff596c..e139cd6 100644 --- a/drivers/staging

[PATCH 13/16] zcache/zbud: Add incremental accessory counters

2013-02-01 Thread Konrad Rzeszutek Wilk
that are going to be used for debug fs entries. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/zbud.c | 58 +-- 1 file changed, 39 insertions(+), 19 deletions(-) diff --git a/drivers/staging/zcache/zbud.c b/drivers/staging/zcache/zbud.c

[PATCH 12/16] zcache/zbud: Fix compiler warnings.

2013-02-01 Thread Konrad Rzeszutek Wilk
Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/zbud.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/drivers/staging/zcache/zbud.c b/drivers/staging/zcache/zbud.c index 6835fab..4654978 100644 --- a/drivers/staging/zcache

[PATCH 11/16] zcache/debug: Coalesce all debug under CONFIG_ZCACHE_DEBUG

2013-02-01 Thread Konrad Rzeszutek Wilk
and also define this extra attribute in the Kconfig entry. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/Kconfig | 8 drivers/staging/zcache/debug.c | 2 +- drivers/staging/zcache/zcache-main.c | 6 +++--- 3 files changed, 12 insertions(+), 4 deletions

[PATCH 02/16] zcache: Provide accessory functions for counter increase

2013-02-01 Thread Konrad Rzeszutek Wilk
This is the first step in moving the debugfs code out of the main file in-to another file. And also allow the code to run without CONFIG_DEBUG_FS defined. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/zcache-main.c | 103 +++ 1 file changed, 68

[PATCH 10/16] zcache: Module license is defined twice.

2013-02-01 Thread Konrad Rzeszutek Wilk
The other (same license) is at the end of the file. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/zcache-main.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/zcache/zcache-main.c b/drivers/staging/zcache/zcache-main.c index 8341d79..8830359 100644 --- a

[PATCH 08/16] zcache/debug: Use an array to initialize/use debugfs attributes.

2013-02-01 Thread Konrad Rzeszutek Wilk
It makes it neater and also allows us to piggyback on that in the zcache_dump function. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/debug.c | 141 ++--- 1 file changed, 47 insertions(+), 94 deletions(-) diff --git a/drivers/staging/zcache

[PATCH 03/16] zcache: Provide accessory functions for counter decrease.

2013-02-01 Thread Konrad Rzeszutek Wilk
This way we can have all wrapped with these functions and can disable/enable this with CONFIG_DEBUG_FS. Signed-off-by: Konrad Rzeszutek Wilk --- drivers/staging/zcache/zcache-main.c | 88 +--- 1 file changed, 51 insertions(+), 37 deletions(-) diff --git a

  1   2   3   4   5   6   7   8   9   10   >