* [PATCH 1/2] arm64: dts: Add a virtio disk to the RTSM motherboard
@ 2013-12-20 12:52 Mark Brown
2013-12-20 12:52 ` [PATCH 2/2] arm64: dts: Reduce size of virtio block device for foundation model Mark Brown
0 siblings, 1 reply; 6+ messages in thread
From: Mark Brown @ 2013-12-20 12:52 UTC (permalink / raw)
To: linux-arm-kernel
From: Mark Hambleton <mahamble@broadcom.com>
Describe the virtio device so we can mount disk images in the simulator.
[Reduced the size of the region based on feedback from review -- broonie]
Signed-off-by: Mark Hambleton <mahamble@broadcom.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
Acked-by: Will Deacon <will.deacon@arm.com>
---
arch/arm64/boot/dts/rtsm_ve-motherboard.dtsi | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/arch/arm64/boot/dts/rtsm_ve-motherboard.dtsi b/arch/arm64/boot/dts/rtsm_ve-motherboard.dtsi
index b45e5f39f577..2f2ecd217363 100644
--- a/arch/arm64/boot/dts/rtsm_ve-motherboard.dtsi
+++ b/arch/arm64/boot/dts/rtsm_ve-motherboard.dtsi
@@ -183,6 +183,12 @@
clocks = <&v2m_oscclk1>, <&v2m_clk24mhz>;
clock-names = "clcdclk", "apb_pclk";
};
+
+ virtio_block at 0130000 {
+ compatible = "virtio,mmio";
+ reg = <0x130000 0x200>;
+ interrupts = <42>;
+ };
};
v2m_fixed_3v3: fixedregulator at 0 {
--
1.8.5.2
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 2/2] arm64: dts: Reduce size of virtio block device for foundation model
2013-12-20 12:52 [PATCH 1/2] arm64: dts: Add a virtio disk to the RTSM motherboard Mark Brown
@ 2013-12-20 12:52 ` Mark Brown
2013-12-20 14:25 ` Catalin Marinas
0 siblings, 1 reply; 6+ messages in thread
From: Mark Brown @ 2013-12-20 12:52 UTC (permalink / raw)
To: linux-arm-kernel
From: Mark Brown <broonie@linaro.org>
Will Deacon observed that kvmtool uses a size of 0x200 for virtio
block memory region and that the virtio block spec only uses 31 bytes in
the device specific region at 0x100 so reduce the region to a less
wasteful 0x200.
Signed-off-by: Mark Brown <broonie@linaro.org>
---
arch/arm64/boot/dts/foundation-v8.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/foundation-v8.dts b/arch/arm64/boot/dts/foundation-v8.dts
index 519c4b2c0687..4a060906809d 100644
--- a/arch/arm64/boot/dts/foundation-v8.dts
+++ b/arch/arm64/boot/dts/foundation-v8.dts
@@ -224,7 +224,7 @@
virtio_block at 0130000 {
compatible = "virtio,mmio";
- reg = <0x130000 0x1000>;
+ reg = <0x130000 0x200>;
interrupts = <42>;
};
};
--
1.8.5.2
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 2/2] arm64: dts: Reduce size of virtio block device for foundation model
2013-12-20 12:52 ` [PATCH 2/2] arm64: dts: Reduce size of virtio block device for foundation model Mark Brown
@ 2013-12-20 14:25 ` Catalin Marinas
2013-12-20 16:05 ` Mark Brown
0 siblings, 1 reply; 6+ messages in thread
From: Catalin Marinas @ 2013-12-20 14:25 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Dec 20, 2013 at 12:52:44PM +0000, Mark Brown wrote:
> From: Mark Brown <broonie@linaro.org>
>
> Will Deacon observed that kvmtool uses a size of 0x200 for virtio
> block memory region and that the virtio block spec only uses 31 bytes in
> the device specific region at 0x100 so reduce the region to a less
> wasteful 0x200.
What do we waste by keeping it at 4K? That's ioremap'ed, so we use a
full page anyway.
--
Catalin
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] arm64: dts: Reduce size of virtio block device for foundation model
2013-12-20 14:25 ` Catalin Marinas
@ 2013-12-20 16:05 ` Mark Brown
2013-12-20 16:34 ` Catalin Marinas
0 siblings, 1 reply; 6+ messages in thread
From: Mark Brown @ 2013-12-20 16:05 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Dec 20, 2013 at 02:25:24PM +0000, Catalin Marinas wrote:
> On Fri, Dec 20, 2013 at 12:52:44PM +0000, Mark Brown wrote:
> > From: Mark Brown <broonie@linaro.org>
> > Will Deacon observed that kvmtool uses a size of 0x200 for virtio
> > block memory region and that the virtio block spec only uses 31 bytes in
> > the device specific region at 0x100 so reduce the region to a less
> > wasteful 0x200.
> What do we waste by keeping it at 4K? That's ioremap'ed, so we use a
> full page anyway.
This was done for consistency with the same change made in RTSM
motherboard patch so that we register the same device in the same way.
That change was at Will's request, he did point out that we might use
16K pages in some systems IIRC but personally I'm not sure it makes a
huge difference either way - I do think we should keep all registrations
of the device consistent though to avoid user confusion about how to use
it.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20131220/656f6dec/attachment.sig>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] arm64: dts: Reduce size of virtio block device for foundation model
2013-12-20 16:05 ` Mark Brown
@ 2013-12-20 16:34 ` Catalin Marinas
0 siblings, 0 replies; 6+ messages in thread
From: Catalin Marinas @ 2013-12-20 16:34 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Dec 20, 2013 at 04:05:22PM +0000, Mark Brown wrote:
> On Fri, Dec 20, 2013 at 02:25:24PM +0000, Catalin Marinas wrote:
> > On Fri, Dec 20, 2013 at 12:52:44PM +0000, Mark Brown wrote:
> > > From: Mark Brown <broonie@linaro.org>
>
> > > Will Deacon observed that kvmtool uses a size of 0x200 for virtio
> > > block memory region and that the virtio block spec only uses 31 bytes in
> > > the device specific region at 0x100 so reduce the region to a less
> > > wasteful 0x200.
>
> > What do we waste by keeping it at 4K? That's ioremap'ed, so we use a
> > full page anyway.
>
> This was done for consistency with the same change made in RTSM
> motherboard patch so that we register the same device in the same way.
> That change was at Will's request, he did point out that we might use
> 16K pages in some systems IIRC but personally I'm not sure it makes a
> huge difference either way - I do think we should keep all registrations
> of the device consistent though to avoid user confusion about how to use
> it.
OK. Applied.
--
Catalin
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] arm64: dts: Add a virtio disk to the RTSM motherboard
@ 2013-12-03 19:58 Mark Brown
0 siblings, 0 replies; 6+ messages in thread
From: Mark Brown @ 2013-12-03 19:58 UTC (permalink / raw)
To: linux-arm-kernel
From: Mark Hambleton <mahamble@broadcom.com>
Describe the virtio device so we can mount disk images in the simulator.
[Reduced the size of the region based on feedback from review -- broonie]
Signed-off-by: Mark Hambleton <mahamble@broadcom.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
Acked-by: Will Deacon <will.deacon@arm.com>
---
arch/arm64/boot/dts/rtsm_ve-motherboard.dtsi | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/arch/arm64/boot/dts/rtsm_ve-motherboard.dtsi b/arch/arm64/boot/dts/rtsm_ve-motherboard.dtsi
index b45e5f39f577..2f2ecd217363 100644
--- a/arch/arm64/boot/dts/rtsm_ve-motherboard.dtsi
+++ b/arch/arm64/boot/dts/rtsm_ve-motherboard.dtsi
@@ -183,6 +183,12 @@
clocks = <&v2m_oscclk1>, <&v2m_clk24mhz>;
clock-names = "clcdclk", "apb_pclk";
};
+
+ virtio_block at 0130000 {
+ compatible = "virtio,mmio";
+ reg = <0x130000 0x200>;
+ interrupts = <42>;
+ };
};
v2m_fixed_3v3: fixedregulator at 0 {
--
1.8.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-12-20 16:34 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-20 12:52 [PATCH 1/2] arm64: dts: Add a virtio disk to the RTSM motherboard Mark Brown
2013-12-20 12:52 ` [PATCH 2/2] arm64: dts: Reduce size of virtio block device for foundation model Mark Brown
2013-12-20 14:25 ` Catalin Marinas
2013-12-20 16:05 ` Mark Brown
2013-12-20 16:34 ` Catalin Marinas
-- strict thread matches above, loose matches on Subject: below --
2013-12-03 19:58 [PATCH 1/2] arm64: dts: Add a virtio disk to the RTSM motherboard Mark Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).