From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olaf Hering Subject: Re: [PATCH 2/3] Handle GNTST_eagain in kernel drivers Date: Tue, 3 Jan 2012 19:40:45 +0100 Message-ID: <20120103184045.GA26720@aepfle.de> References: <1324092141-9730-1-git-send-email-adin@scannell.ca> <1324092141-9730-3-git-send-email-adin@scannell.ca> <20111217143015.GD14816@konrad-lan> <20120102160611.GB12529@aepfle.de> <20120103181953.GH749@andromeda.dapyr.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20120103181953.GH749@andromeda.dapyr.net> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Konrad Rzeszutek Wilk Cc: andres@gridcentric.ca, xen-devel@lists.xensource.com, Adin Scannell , JBeulich@suse.com, adin@gridcentric.com List-Id: xen-devel@lists.xenproject.org On Tue, Jan 03, Konrad Rzeszutek Wilk wrote: > If this problem does occur (the swapper died in dom0) should the > printk at least use printk_ratelimited so that we don't cause too much > noise? I remember there was no flood because the guest was stuck anyway. But see below. > > The caller has to deal with the various !GNTST_okay states anyway, this > > patch wont change that fact. > > Ok, so then we don't really need the printk right? As the caller > would presumarily do the right thing and also print the error? I think its more a debug thing, so that I knew something bad happend. And at that time it was just helpful to get me some understanding of the code flow. Since now that part of the paging code is reasonable debugged, the printk is not really needed anymore. Instead the code who uses these new functionality should have proper error handling and print reasonable diagnostic messages. Olaf