On Wed, Feb 11, 2015 at 11:26:12AM +, Daniel P. Berrange wrote:
> Add a new 'guest-set-user-password' command for changing the password
> of guest OS user accounts. This command is needed to enable OpenStack
> to support its API for changing the admin password of guests running
> on KVM/QEMU. I
On Mon, Jan 12, 2015 at 03:58:14PM +, Daniel P. Berrange wrote:
> Add a new 'guest-set-admin-password' command for changing the
> root/administrator password. This command is needed to allow
> OpenStack to support its API for changing the admin password
> on a running guest.
>
> Accepts either
On Thu, Dec 25, 2014 at 08:37:29AM +0300, Denis V. Lunev wrote:
> +#ifdef CONFIG_FALLOCATE_ZERO_RANGE
> +do {
> +if (fallocate(s->fd, CONFIG_FALLOCATE_ZERO_RANGE,
Must be a typo, FALLOC_FL_ZERO_RANGE is what you mean.
Roman.
On Thu, Dec 25, 2014 at 08:37:30AM +0300, Denis V. Lunev wrote:
> This sequence works efficiently if FALLOC_FL_ZERO_RANGE is not supported.
> The idea is that FALLOC_FL_PUNCH_HOLE could not increase file size
> but it cleans already allocated blocks inside the file. If we have to
> create something
c | 12
> 1 file changed, 8 insertions(+), 4 deletions(-)
Acked-by: Roman Kagan
Roman.
; ---
> block/parallels.c | 48
> 1 file changed, 24 insertions(+), 24 deletions(-)
Acked-by: Roman Kagan
Roman.
tefan Hajnoczi
> ---
> block/parallels.c | 19 ++-
> 1 file changed, 10 insertions(+), 9 deletions(-)
Acked-by: Roman Kagan
Roman.
On Tue, Dec 30, 2014 at 01:07:09PM +0300, Denis V. Lunev wrote:
> From the point of guest each write to real disk prior to disk barrier
> operation could be lost. Therefore there is no problem that "not synced"
> new block is lost due to not updated allocation table if QEMU is crashed.
>
> This pa
On Tue, Jan 13, 2015 at 11:16:04PM +0300, Denis V. Lunev wrote:
> On 13/01/15 18:17, Denis V. Lunev wrote:
> >On 13/01/15 17:50, Roman Kagan wrote:
> >>On Tue, Dec 30, 2014 at 01:07:09PM +0300, Denis V. Lunev wrote:
> >>>--- a/block/parallels.c
> >>>+++
On Tue, Dec 30, 2014 at 01:07:10PM +0300, Denis V. Lunev wrote:
> The idea is that we do not need to immediately sync BAT to the image as
> from the guest point of view there is a possibility that IO is lost
> even in the physical controller until flush command was finished.
> bdrv_co_flush_to_os i
On Wed, Jan 14, 2015 at 04:08:50PM +0300, Denis V. Lunev wrote:
> On 14/01/15 16:03, Roman Kagan wrote:
> >On Tue, Dec 30, 2014 at 01:07:10PM +0300, Denis V. Lunev wrote:
> >>+static int cache_bat(BlockDriverState *bs, uint32_t idx, uint32_t
> >>new_data_off)
>
On Tue, Dec 30, 2014 at 01:07:11PM +0300, Denis V. Lunev wrote:
> This is preparational commit for tweaks in Parallels image expansion.
> The idea is that enlarge via truncate by one data block is slow. It
> would be much better to use fallocate via bdrv_write_zeroes and
> expand by some significan
On Wed, Jan 14, 2015 at 05:31:20PM +0300, Denis V. Lunev wrote:
> On 14/01/15 17:26, Roman Kagan wrote:
> >On Tue, Dec 30, 2014 at 01:07:11PM +0300, Denis V. Lunev wrote:
> >>This is preparational commit for tweaks in Parallels image expansion.
> >>The idea is that enlar
On Tue, Dec 30, 2014 at 01:07:12PM +0300, Denis V. Lunev wrote:
> Plain image expansion spends a lot of time to update image file size.
> This seriously affects the performance. The following simple test
> qemu_img create -f parallels -o cluster_size=64k ./1.hds 64G
> qemu_io -n -c "write -P 0x
On Mon, Dec 15, 2014 at 01:30:03PM +0100, Kevin Wolf wrote:
> Am 15.12.2014 um 09:27 hat Denis V. Lunev geschrieben:
> > From: Roman Kagan
> >
> > Add backing file support to Parallels format driver.
> >
> > That said, I think backing file operations should end
ment about locking considerations in the driver.
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
> block/parallels.c | 54 +-
> 1 file changed, 33 insertions(+), 21 deletions(-)
Reviewed-by: Roman Kagan
On Tue, Apr 28, 2015 at 10:46:41AM +0300, Denis V. Lunev wrote:
> From the guest point of view unallocated blocks are zeroed.
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
> block/parallels.c | 1 +
> 1 file change
On Tue, Apr 28, 2015 at 10:46:59AM +0300, Denis V. Lunev wrote:
> Plain image expansion spends a lot of time to update image file size.
> This seriously affects the performance. The following simple test
> qemu_img create -f parallels -o cluster_size=64k ./1.hds 64G
> qemu_io -n -c "write -P 0x
On Tue, Apr 28, 2015 at 10:46:42AM +0300, Denis V. Lunev wrote:
> Support write on Parallels images. The code is almost the same as one
> in the previous patch implemented scatter-gather IO for read.
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
>
or the image expansion
> - tail writes are ordered, i.e. the guest IO queue could not be sent
> immediately to the host introducing additional IO delays
>
> This patch just adds proper parameters into BDRVParallelsState and
> performs options parsing in parallels_open.
>
>
On Tue, Apr 28, 2015 at 01:59:56PM +0300, Roman Kagan wrote:
> On Tue, Apr 28, 2015 at 10:46:58AM +0300, Denis V. Lunev wrote:
> > This is preparational commit for tweaks in Parallels image expansion.
> > The idea is that enlarge via truncate by one data block is slow. It
> >
On Tue, Mar 10, 2015 at 11:51:09AM +0300, Denis V. Lunev wrote:
> This will allow to use this data as buffer to BAT update directly
> without any intermediate buffers.
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
&
On Tue, Mar 10, 2015 at 11:51:08AM +0300, Denis V. Lunev wrote:
> deduplicate copy/paste arithmetcs
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
> block/parallels.c | 12 +---
> 1 file changed, 9
1.hds
> from 45 Mb/sec to 160 Mb/sec on my SSD disk. The gain on rotational media
> is much more sufficient, from 800 Kb/sec to 45 Mb/sec.
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
> block/parallels.c | 2 +-
64k ./1.hds 64G
> qemu-io -f parallels -c "write -P 0x11 0 1024k" 1.hds
> writing from 50-60 Mb/sec to 80-90 Mb/sec on rotational media and
> from 160 Mb/sec to 190 Mb/sec on SSD disk.
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin
BDRVParallelsState for this.
>
> On the other hand, nothing should be done to clear inuse in
> parallels_check. Generic close will do the job right.
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
> block/parallels.c | 5
On Tue, Mar 10, 2015 at 11:51:14AM +0300, Denis V. Lunev wrote:
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
> tests/qemu-iotests/115 | 9 +
> tests/qemu-iotests/115.out | 19 ++-
> 2 file
pand by some significant amount at once.
>
> This patch just adds proper parameters into BDRVParallelsState and
> performs options parsing in parallels_open.
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajno
mage are zeroed. Nothing else could be done here
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
> block/parallels.c | 82
> +++
> 1 file changed, 82 insertions(+)
&
V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
> block/parallels.c | 24 +---
> 1 file changed, 17 insertions(+), 7 deletions(-)
Reviewed-by: Roman Kagan
Roman.
On Tue, Mar 10, 2015 at 11:51:17AM +0300, Denis V. Lunev wrote:
> calculate offset of the BAT entry in the image file.
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
> block/parallels.c | 15 +--
> 1
On Tue, Mar 10, 2015 at 05:44:19PM +0300, Denis V. Lunev wrote:
> On 10/03/15 17:38, Roman Kagan wrote:
> >On Tue, Mar 10, 2015 at 11:51:13AM +0300, Denis V. Lunev wrote:
> >>The software driver must set inuse field in Parallels header to
> >>0x746F6E59 when the image i
On Tue, Mar 10, 2015 at 11:51:11AM +0300, Denis V. Lunev wrote:
> This will help to avoid forward declarations for upcoming parallels_check
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
>
On Tue, Mar 10, 2015 at 11:51:16AM +0300, Denis V. Lunev wrote:
> Try to perform IO for the biggest continuous block possible.
> The performance for sequential read is increased from 220 Gb/sec to
> 360 Gb/sec for continous image on my SSD HDD.
>
> Signed-off-by: Denis V. Lunev
&g
ential write is increased from 200 Gb/sec to
> 235 Gb/sec on my SSD HDD.
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
> block/parallels.c | 44
> 1 file changed, 24 insertions(+),
mage are zeroed. Nothing else could be done here
>
> Signed-off-by: Denis V. Lunev
> CC: Roman Kagan
> CC: Kevin Wolf
> CC: Stefan Hajnoczi
> ---
> block/parallels.c | 85
> +++
> 1 file changed, 85 insertions(+)
Reviewed-by: Roman Kagan
Roman.
On Fri, Oct 09, 2015 at 04:42:33PM +0200, Paolo Bonzini wrote:
> You need to add SYNIC vectors to the EOI exit bitmap, so that APICv
> (Xeon E5 or higher, Ivy Bridge or newer) is handled correctly. You also
> need to check the auto EOI exit bitmap in __apic_accept_irq, and avoid
> going through kv
On Fri, Oct 09, 2015 at 04:41:15PM +0200, Paolo Bonzini wrote:
> On 09/10/2015 15:39, Denis V. Lunev wrote:
> > A new vcpu exit is introduced to notify the userspace of the
> > changes in Hyper-V synic configuraion triggered by guest writing to the
> > corresponding MSRs.
>
> Why is this exit nece
On Mon, Oct 12, 2015 at 10:58:36AM +0200, Paolo Bonzini wrote:
> On 12/10/2015 10:48, Cornelia Huck wrote:
> > Going back to Paolo's original question, I think changing the check
> > to !KVM_IRQ_ROUTING_IRQCHIP makes sense, if I understand the code
> > correctly. They seem to be the only special on
On Mon, Oct 12, 2015 at 07:42:42AM -0600, Eric Blake wrote:
> On 10/09/2015 07:39 AM, Denis V. Lunev wrote:
> > From: Andrey Smetanin
> >
> > A new vcpu exit is introduced to notify the userspace of the
> > changes in Hyper-V synic configuraion triggered by guest writing to the
> Again, is 'synic
On Wed, Oct 28, 2015 at 06:41:45PM +0100, Paolo Bonzini wrote:
> Hi Andrey,
>
> just one question. Is kvm_arch_set_irq actually needed? I think
> everything should work fine without it. Can you check? If so, I can
> remove it myself and revert the patch that introduced the hook.
While Andrey
r-V SynIC MSR's
> > * setup Hyper-V SynIC evt/msg pages
> > * setup SINT's routing
> > * inject SINT's into destination vCPU by 'hyperv-synic-test-device'
> > * wait for SINT's isr's completion
> > * clear Hyper-V SynIC evt/msg pages and
On Tue, Nov 03, 2015 at 03:51:16PM +0100, Paolo Bonzini wrote:
>
>
> On 03/11/2015 15:36, Andrey Smetanin wrote:
> >>
> >>
> >> if I run a patched QEMU but I *do not* enable the synthetic interrupt
> >> controller. I can fix it by wrapping the calls to synic_exit with "if
> >> (!host)", but I ha
On Sun, Feb 07, 2016 at 11:08:07AM +0200, Michael S. Tsirkin wrote:
> On Tue, Jan 26, 2016 at 02:50:25PM +0100, Igor Mammedov wrote:
> > From: Roman Kagan
> > -static Aml *build_fdc_device_aml(void)
> > +static Aml *build_fdinfo_aml(int idx, uint8_t ty
On Fri, Feb 05, 2016 at 07:25:07PM +0100, Igor Mammedov wrote:
> On Thu, 4 Feb 2016 23:54:13 +0200
> "Michael S. Tsirkin" wrote:
> > -static Aml *build_fdc_device_aml(void)
> > +static Aml *build_fdinfo_aml(int idx, uint8_t type, uint8_t cylinders,
> > + uint8_t heads,
On Mon, Feb 08, 2016 at 03:20:47PM -0500, John Snow wrote:
> On 02/08/2016 08:14 AM, Roman Kagan wrote:
> > On Fri, Feb 05, 2016 at 07:25:07PM +0100, Igor Mammedov wrote:
> >>> +aml_append(fdi,
> >>> +aml_int(cylinders - 1)); /* Maximum Cylinder Num
On Tue, Feb 09, 2016 at 11:22:01AM -0500, John Snow wrote:
> > I don't. At the time the patch was developed there basically were no
> > mechanisms to update the geometry at all (and this was what you patchset
> > addressed, in particular, wasn't it?) so I didn't care.
>
> That's not true.
>
> Yo
On Tue, Feb 09, 2016 at 07:36:12PM +0100, Laszlo Ersek wrote:
> In my opinion, the real mess in this case is in the ACPI spec itself. If
> you re-read the _FDI control method's description, the Package that it
> returns contains *dynamic* geometry data, about the *disk* (not *drive*):
>
> - Maximu
On Wed, Feb 10, 2016 at 11:14:30AM -0500, John Snow wrote:
> On 02/09/2016 01:48 PM, Michael S. Tsirkin wrote:
> > On Tue, Feb 09, 2016 at 07:36:12PM +0100, Laszlo Ersek wrote:
> >> Implementing this in QEMU would require:
> >> - inventing virt-only registers for the FDC that provide the current
>
On Wed, Feb 10, 2016 at 12:16:32PM -0500, John Snow wrote:
> On 02/10/2016 12:10 PM, Roman Kagan wrote:
> > Well, as I wrote in another mail, SeaBIOS, which is supposed to provide
> > the same information to int 0x13/0x08, populates it with static data
> > based only on the
On Sun, Feb 14, 2016 at 05:02:14PM +0200, Michael S. Tsirkin wrote:
> On Sat, Feb 13, 2016 at 12:26:53PM -0500, Kevin O'Connor wrote:
> > On Tue, Feb 09, 2016 at 07:36:12PM +0100, Laszlo Ersek wrote:
> > > In my opinion, the real mess in this case is in the ACPI spec itself. If
> > > you re-read th
description into runtime-generated SSDT.
Roman Kagan (4):
i386/acpi: make floppy controller object dynamic
i386: expose floppy drive CMOS type
fdc: add function to determine drive chs limits
i386: populate floppy drive information in DSDT
Signed-off-by: Roman Kagan
Cc: Igor Mammedov
Cc: "Mich
matching the given floppy drive type.
Signed-off-by: Roman Kagan
Cc: Igor Mammedov
Cc: "Michael S. Tsirkin"
Cc: Marcel Apfelbaum
Cc: John Snow
Cc: Laszlo Ersek
Cc: Kevin O'Connor
---
changes since v7:
- use drive max c,h,s rather than the current diskette geometry
Make it possible to query the CMOS type of a floppy drive outside of the
source file where it's defined.
It will allow to properly populate the corresponding ACPI objects and
thus enable Windows on BIOS-less systems to access the floppy drives.
Signed-off-by: Roman Kagan
Cc: Igor Mammedo
VMF.
This patch adds those objects to the floppy controller in DSDT,
populating them with the information from respective QEMU objects.
Signed-off-by: Roman Kagan
Cc: Igor Mammedov
Cc: "Michael S. Tsirkin"
Cc: Marcel Apfelbaum
Cc: John Snow
Cc: Laszlo Ersek
Cc: Kevin O'Connor
---
the declarations of the fields indicating whether the contoller is
enabled.
Signed-off-by: Roman Kagan
Signed-off-by: Igor Mammedov
Reviewed-by: Marcel Apfelbaum
Cc: "Michael S. Tsirkin"
Cc: John Snow
Cc: Laszlo Ersek
Cc: Kevin O'Connor
---
changes since v7:
- remove unn
On Wed, Feb 17, 2016 at 01:58:47PM -0700, Eric Blake wrote:
> On 02/17/2016 11:10 AM, Denis V. Lunev wrote:
> > @@ -446,6 +448,11 @@ The following request types exist:
> > about the contents of the export affected by this command, until
> > overwriting it again with `NBD_CMD_WRITE`.
> >
s of the corresponding configuration
> MSR. If deemed necessary, the "catch up" mode (the timer period is
> shortened until the timer catches up) will be implemented later.
>
> Signed-off-by: Andrey Smetanin
> CC: Gleb Natapov
> CC: Paolo Bonzini
> CC: "K.
and let the message sources compete for the slot again.
>
> Signed-off-by: Andrey Smetanin
> CC: Gleb Natapov
> CC: Paolo Bonzini
> CC: "K. Y. Srinivasan"
> CC: Haiyang Zhang
> CC: Vitaly Kuznetsov
> CC: Roman Kagan
> CC: Denis V. Lunev
> CC: qemu-devel@nongnu.org
Reviewed-by: Roman Kagan
Roman.
On Fri, Nov 27, 2015 at 11:49:40AM +0100, Paolo Bonzini wrote:
[ sorry missed your message on Friday, replying now ]
> On 27/11/2015 09:12, Roman Kagan wrote:
> >> > +n = div64_u64(time_now - stimer->exp_time, stimer->count) + 1;
> >> > +stim
On Mon, Dec 14, 2015 at 04:05:03PM +0100, Igor Mammedov wrote:
> On Mon, 14 Dec 2015 11:22:39 +0300
> "Denis V. Lunev" wrote:
>
> > From: Roman Kagan
> >
> > On x86-based systems Linux determines the presence and the type of
> > floppy drives via a q
On Wed, Dec 16, 2015 at 05:46:57PM +0100, Igor Mammedov wrote:
> On Wed, 16 Dec 2015 10:45:09 +0300 "Denis V. Lunev"
> wrote:
> > @@ -1125,6 +1183,26 @@ build_ssdt(GArray *table_data, GArray *linker,
> > aml_append(ssdt, scope);
> > }
> >
> > +{
> add this block only if floppy
On Mon, Jan 11, 2016 at 02:51:15PM +0100, Igor Mammedov wrote:
> On Wed, 30 Dec 2015 23:11:50 +0300
> Roman Kagan wrote:
>
> > Windows on UEFI systems is only capable of detecting the presence and
> > the type of floppy drives via corresponding ACPI objects.
> >
>
On Mon, Jan 11, 2016 at 03:47:24PM +0100, Igor Mammedov wrote:
> On Mon, 11 Jan 2016 17:26:26 +0300
> Roman Kagan wrote:
>
> > On Mon, Jan 11, 2016 at 02:51:15PM +0100, Igor Mammedov wrote:
> > > I've rebased/rewrote this series on top of current PCI tree.
> &
onnecting
either 1 or 2 floopy drives of different size. Each time the floopies
were correctly identified and their contents was visibile in the guest.
So
Tested-by: Roman Kagan
Roman.
On Wed, Jan 13, 2016 at 03:36:18PM +0100, Laszlo Ersek wrote:
> On 12/30/15 21:11, Roman Kagan wrote:
> > Windows on UEFI systems is only capable of detecting the presence and
> > the type of floppy drives via corresponding ACPI objects.
>
> I'm late to the party, but
On Wed, Jan 13, 2016 at 06:49:44PM +0300, Roman Kagan wrote:
> On Wed, Jan 13, 2016 at 03:36:18PM +0100, Laszlo Ersek wrote:
> > On 12/30/15 21:11, Roman Kagan wrote:
> > > Windows on UEFI systems is only capable of detecting the presence and
> > > the type of floppy dr
On Thu, Jan 14, 2016 at 11:30:43AM +0300, Pavel Fedin wrote:
> > --- a/Documentation/virtual/kvm/api.txt
> > +++ b/Documentation/virtual/kvm/api.txt
> > @@ -3359,6 +3359,14 @@ Hyper-V SynIC state change. Notification is used to
> > remap SynIC
> > event/message pages and to enable/disable SynIC m
On Thu, Jan 14, 2016 at 01:50:20PM +0300, Pavel Fedin wrote:
> > We thought reusing KVM_EXIT_HYPERCALL was a bad idea exactly because of
> > that. Hypercalls are not universal, the calling and return conventions
> > are hypervisor-specific.
>
> Treatment of them is hypervisor-specific, but from
On Wed, Jan 20, 2016 at 05:44:49PM +0300, Denis V. Lunev wrote:
> On 01/20/2016 05:05 PM, Paolo Bonzini wrote:
> >
> >On 19/01/2016 08:48, Denis V. Lunev wrote:
> >>>diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h
> >>>index 6877b4d7e..93c9e25 100644
> >>>--- a/include/linux/kvm_ho
On Wed, Jan 20, 2016 at 02:59:08PM +0100, Paolo Bonzini wrote:
> >> --- a/Documentation/virtual/kvm/api.txt
> >> +++ b/Documentation/virtual/kvm/api.txt
> >> @@ -3359,6 +3359,14 @@ Hyper-V SynIC state change. Notification is used to
> >> remap SynIC
> >> event/message pages and to enable/disable
; >
> > Signed-off-by: Andrey Smetanin
> > Reviewed-by: Roman Kagan
> > CC: Gleb Natapov
> > CC: Paolo Bonzini
> > CC: Joerg Roedel
> > CC: "K. Y. Srinivasan"
> > CC: Haiyang Zhang
> > CC: Roman Kagan
> > CC: Denis V. Lunev
>
On Wed, Jan 20, 2016 at 06:02:25PM +0100, Paolo Bonzini wrote:
>
>
> On 20/01/2016 16:20, 'Roman Kagan' wrote:
> >> > So we should not add a new exit
> > Why? VCPU exit codes are not a scarse resource.
>
> Indeed, but grouping makes things easier t
On Wed, Jan 20, 2016 at 02:40:14PM -0500, John Snow wrote:
> On 01/20/2016 02:55 AM, Denis V. Lunev wrote:
> > should we recreate ACPI tables after geometry switch?
> > This would be especially interesting for the case of
> > Win2k12 (or Win8.1 if you prefer) under OVMF.
> >
> > Den
>
> This seri
On Thu, Jan 21, 2016 at 09:59:09AM -0500, John Snow wrote:
> > Now that is a bit trickier: the currently submitted code queries the
> > floppy properties at SSDT build time, and sticks static objects into
> > AML; if that really needs updating at runtime it'll require certain
> > refactoring.
> >
On Wed, Dec 16, 2015 at 11:15:55PM +0100, Igor Mammedov wrote:
> On Wed, 16 Dec 2015 20:34:55 +0300
> Roman Kagan wrote:
>
> > On Wed, Dec 16, 2015 at 05:46:57PM +0100, Igor Mammedov wrote:
> > > On Wed, 16 Dec 2015 10:45:09 +0300 "Denis V. Lunev"
> &
On Fri, Dec 18, 2015 at 06:19:25PM +0300, Pavel Fedin wrote:
> I dislike implementing architecture-dependent exit code where we could
> implement an architecture-independent one.
>
> As far as i understand this code, KVM_EXIT_HYPERV is called when one of
> three MSRs are accessed. But, shouldn
On Fri, Dec 18, 2015 at 05:01:59PM +0100, Paolo Bonzini wrote:
> On 18/12/2015 16:19, Pavel Fedin wrote:
> > As far as i understand this code, KVM_EXIT_HYPERV is called when one
> > of three MSRs are accessed. But, shouldn't we have implemented
> > instead something more generic, like KVM_EXIT_REG
On Fri, Dec 18, 2015 at 10:10:11AM -0800, Peter Hornyack wrote:
> On Fri, Dec 18, 2015 at 8:01 AM, Paolo Bonzini wrote:
> > On 18/12/2015 16:19, Pavel Fedin wrote:
> >> As far as i understand this code, KVM_EXIT_HYPERV is called when one
> >> of three MSRs are accessed. But, shouldn't we have impl
Roman Kagan (2):
i386: expose floppy-related objects in SSDT
tests: update expected SSDT for floppy changes
Signed-off-by: Roman Kagan
Signed-off-by: Denis V. Lunev
CC: Michael S. Tsirkin
CC: Igor Mammedov
CC: Paolo Bonzini
CC: Richard Henderson
CC: Eduardo Habkost
CC: John Snow
CC
Update the expected SSDTs to reflect the changes introduced in the
previous patch.
Signed-off-by: Roman Kagan
Signed-off-by: Denis V. Lunev
CC: Michael S. Tsirkin
CC: Igor Mammedov
CC: Paolo Bonzini
CC: Richard Henderson
CC: Eduardo Habkost
CC: John Snow
CC: Kevin Wolf
---
changes since
SSDT, populating them via AML
API. For that, a couple of functions are extern-ified to facilitate
populating those objects in acpi-build.c.
Signed-off-by: Roman Kagan
Signed-off-by: Denis V. Lunev
CC: Michael S. Tsirkin
CC: Igor Mammedov
CC: Paolo Bonzini
CC: Richard Henderson
CC: Eduardo Habkos
On Tue, Dec 22, 2015 at 05:07:16PM +0200, Michael S. Tsirkin wrote:
> On Fri, Dec 18, 2015 at 10:32:29PM +0300, Roman Kagan wrote:
> > On x86-based systems Linux determines the presence and the type of
> > floppy drives via a query of a CMOS field. So does SeaBIOS when
> >
On Wed, Dec 23, 2015 at 10:47:22AM +, Daniel P. Berrange wrote:
> On Wed, Dec 23, 2015 at 11:14:12AM +0800, Fam Zheng wrote:
> > As an alternative, can we introduce .bdrv_flock() in protocol drivers, with
> > similar semantics to flock(2) or lockf(3)? That way all formats can benefit,
> > and a
On Tue, Dec 22, 2015 at 06:41:47PM +0200, Michael S. Tsirkin wrote:
> On Fri, Dec 18, 2015 at 10:32:30PM +0300, Roman Kagan wrote:
> > Update the expected SSDTs to reflect the changes introduced in the
> > previous patch.
> >
> > Signed-off-by: Roman Kagan
> &
On Wed, Dec 23, 2015 at 03:45:29PM +0200, Michael S. Tsirkin wrote:
> This is the actual vs expected diff with both patches applied.
Interesting... The diff suggests that qemu running in your test
environment has no floppy drives, while in mine ...
> +Scope (\_SB.PCI0.ISA.FDC0)
> +{
> +
On Wed, Dec 23, 2015 at 06:06:31PM +0300, Roman Kagan wrote:
> On Wed, Dec 23, 2015 at 03:45:29PM +0200, Michael S. Tsirkin wrote:
> > This is the actual vs expected diff with both patches applied.
>
> Interesting... The diff suggests that qemu running in your test
> environ
On Wed, Dec 23, 2015 at 06:47:16PM +0100, Igor Mammedov wrote:
> On Wed, 23 Dec 2015 20:20:54 +0300
> Roman Kagan wrote:
> > > ... two 1.44M drives with bogus geometry for q35.
> >
> > This one is a bug in my patch, indeed: I was tricked by FDRIVE_DRV_NONE
> &
On Thu, Dec 24, 2015 at 12:30:26PM +0300, Andrey Smetanin wrote:
> Currently on x86 arch we has already 32 requests defined
> so the newer request bits can't be placed inside
> vcpu->requests(unsigned long) inside x86 32 bit system.
> But we are going to add a new request in x86 arch
> for Hyper-V
tup
> * timer cleanup
>
> Signed-off-by: Andrey Smetanin
> CC: Gleb Natapov
> CC: Paolo Bonzini
> CC: Roman Kagan
> CC: Denis V. Lunev
> CC: qemu-devel@nongnu.org
> ---
> arch/x86/kvm/hyperv.c | 27 +++-
> arch/x86/kvm/trace.h | 170
> +
On Wed, Dec 23, 2015 at 04:53:59PM +0300, Andrey Smetanin wrote:
> Trace the following Hyper SynIC events:
> * set msr
> * set sint irq
> * ack sint
> * sint irq eoi
>
> Signed-off-by: Andrey Smetanin
> CC: Gleb Natapov
> CC: Paolo Bonzini
> CC: Roman Kagan
&g
: James Hogan
> CC: Paolo Bonzini
> CC: Gleb Natapov
> CC: James Hogan
> CC: Paul Burton
> CC: Ralf Baechle
> CC: Alexander Graf
> CC: Christian Borntraeger
> CC: Cornelia Huck
> CC: linux-m...@linux-mips.org
> CC: kvm-...@vger.kernel.org
> CC: linux-
the declarations of the fields indicating whether the contoller is
enabled.
Signed-off-by: Roman Kagan
Cc: "Michael S. Tsirkin"
Cc: Eduardo Habkost
Cc: Igor Mammedov
Cc: John Snow
Cc: Kevin Wolf
Cc: Paolo Bonzini
Cc: Richard Henderson
Cc: qemu-bl...@nongnu.org
Cc: qemu-sta...@
drive controller description into
runtime-generated SSDT.
Note that the series conflicts with Igor's patchset for dynamic DSDT, in
particular, with "[PATCH 50/74] pc: acpi: move FDC0 device from DSDT
to SSDT"; I haven't managed to avoid that while trying to meet
maintainer'
access the floppy drives.
Signed-off-by: Roman Kagan
Cc: "Michael S. Tsirkin"
Cc: Eduardo Habkost
Cc: Igor Mammedov
Cc: John Snow
Cc: Kevin Wolf
Cc: Paolo Bonzini
Cc: Richard Henderson
Cc: qemu-bl...@nongnu.org
Cc: qemu-sta...@nongnu.org
---
changes since v3:
- split out into a sepa
Factor out and expose the function to locate the floppy controller in
the system.
It will be useful when dynamically populating the relevant objects in
the ACPI tables.
Signed-off-by: Roman Kagan
Cc: "Michael S. Tsirkin"
Cc: Eduardo Habkost
Cc: Igor Mammedov
Cc: John Snow
Cc: Kevi
adds those objects to the floppy controller in SSDT,
populating them with the information from the respective QEMU objects.
Signed-off-by: Roman Kagan
Cc: "Michael S. Tsirkin"
Cc: Eduardo Habkost
Cc: Igor Mammedov
Cc: John Snow
Cc: Kevin Wolf
Cc: Paolo Bonzini
Cc: Richard Henders
On Thu, Dec 24, 2015 at 08:17:45AM +0200, Michael S. Tsirkin wrote:
> On Wed, Dec 23, 2015 at 08:51:45PM +0300, Roman Kagan wrote:
> > On Wed, Dec 23, 2015 at 06:47:16PM +0100, Igor Mammedov wrote:
> > > On Wed, 23 Dec 2015 20:20:54 +0300
> > > Roman Kagan wrote:
>
On Tue, Dec 29, 2015 at 03:09:36PM +0100, Igor Mammedov wrote:
> On Fri, 25 Dec 2015 18:04:08 +0300
> Roman Kagan wrote:
>
> > Windows on UEFI systems is only capable of detecting the presence and
> > the type of floppy drives via corresponding ACPI objects.
> >
>
to match the preceding commit "i386/acpi: make floppy controller object
dynamic".
Signed-off-by: Roman Kagan
Cc: "Michael S. Tsirkin"
Cc: Eduardo Habkost
Cc: Igor Mammedov
Cc: John Snow
Cc: Kevin Wolf
Cc: Paolo Bonzini
Cc: Richard Henderson
Cc: qemu-bl...@non
1 - 100 of 673 matches
Mail list logo
