lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Date:	Wed, 07 Mar 2012 09:33:26 +0000
From:	"Jan Beulich" <JBeulich@...e.com>
To:	"Santosh Jodh" <Santosh.Jodh@...rix.com>, <konrad@...nok.org>
Cc:	"David Vrabel" <david.vrabel@...rix.com>,
	"Ian Campbell" <Ian.Campbell@...rix.com>,
	"Paul Durrant" <Paul.Durrant@...rix.com>,
	"waldi@...ian.org" <waldi@...ian.org>,
	"weiyi.huang@...il.com" <weiyi.huang@...il.com>,
	"jeremy@...p.org" <jeremy@...p.org>,
	"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
	"virtualization@...ts.linux-foundation.org" 
	<virtualization@...ts.linux-foundation.org>,
	"xen-devel@...ts.xen.org" <xen-devel@...ts.xen.org>,
	"joe.jin@...cle.com" <joe.jin@...cle.com>,
	"konrad.wilk@...cle.com" <konrad.wilk@...cle.com>,
	"lersek@...hat.com" <lersek@...hat.com>,
	"rusty@...tcorp.com.au" <rusty@...tcorp.com.au>,
	"dgdegra@...ho.nsa.gov" <dgdegra@...ho.nsa.gov>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"jbarnes@...tuousgeek.org" <jbarnes@...tuousgeek.org>,
	"paul.gortmaker@...driver.com" <paul.gortmaker@...driver.com>
Subject: Re: [Xen-devel] [PATCH 0001/001] xen: multi page ring support
 for block devices

>>> On 06.03.12 at 18:20, Konrad Rzeszutek Wilk <konrad@...nok.org> wrote:
>  -> the usage of XenbusStateInitWait? Why do we introduce that? Looks
> like a fix to something.

No, this is required to get the negotiation working (the frontend must
not try to read the new nodes until it can be certain that the backend
populated them). However, as already pointed out in an earlier reply
to Santosh, the way this is done here doesn't appear to allow for the
backend to already be in InitWait state when the frontend gets
invoked.

> -> XENBUS_MAX_RING_PAGES - why 2? Why not 4? What is the optimal
> default size for SSD usage? 16?

What do SSDs have to do with a XenBus definition? Imo it's wrong (and
unnecessary) to introduce a limit at the XenBus level at all - each driver
can do this for itself.

As to the limit for SSDs in the block interface - I don't think the number
of possibly simultaneous requests has anything to do with this. Instead,
I'd expect the request number/size/segments extension that NetBSD
apparently implements to possibly have an effect.

Jan

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Powered by blists - more mailing lists