From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olaf Hering Subject: Re: xentrace: broken by c/s 23050:4ebba54b666f Date: Tue, 22 Mar 2011 15:25:33 +0100 Message-ID: <20110322142533.GA17095@aepfle.de> References: <4D87841B.1000102@amd.com> <20110321173859.GA21766@aepfle.de> <20110321212320.GA15820@aepfle.de> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <20110321212320.GA15820@aepfle.de> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Christoph Egger Cc: xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org On Mon, Mar 21, Olaf Hering wrote: > On Mon, Mar 21, Olaf Hering wrote: > > > On Mon, Mar 21, Christoph Egger wrote: > > > > > > > > Hi, > > > > > > C/s 23050:4ebba54b666f broke xentrace: > > > > > > # xentrace -e 0x8f000 ./xentrace.out > > > (XEN) Xen trace buffers: cpu 0 p 0000000000000000 > > > > The system you test on has 8 cpus, like mine. > > t_info_pages got too big, so alloc_xenheap_pages() failed. > > Can you add a printk to se whats happening in calculate_tbuf_size()? > > compile tested patch: > + gdprintk(XENLOG_INFO, "requested %u t_info_pages for %u trace pages on %u cpus\n", > + t_info_pages, pages, num_online_cpus()); I can reproduce it now. For some reason the line above will print nothing. A plain printk gives: (XEN) requested 0 t_info_pages for 32 trace pages on 8 cpus I will investigate. Olaf