an Jackson.
- Got rid of the global variable aer_watch.
- Added documentation (comments in code) for the new API calls.
- Removed the unnecessary writes to xenstore.
Changes in v2:
- Instead of killing the guest and hiding the device, forcibly remove
the device from the guest.
Venu Bu
Implement the callback function to handle unrecoverable AER errors, and
also the public APIs that can be used to register/unregister the handler.
When an AER error occurs, the handler will forcibly remove the erring
PCIe device from the guest.
Signed-off-by: Venu Busireddy
---
tools/libxl
When a guest is created, register the AER event handler to handle the
AER errors. When an AER error occurs, the handler will forcibly remove
the erring PCIe device from the guest.
Signed-off-by: Venu Busireddy
---
tools/xl/xl_vmcontrol.c | 9 +
1 file changed, 9 insertions(+)
diff
When a guest is created, register the AER event handler to handle the
AER errors. When an AER error occurs, the handler will forcibly remove
the erring PCIe device from the guest.
Signed-off-by: Venu Busireddy
---
tools/xl/xl_vmcontrol.c | 9 +
1 file changed, 9 insertions(+)
diff
an Jackson.
- Got rid of the global variable aer_watch.
- Added documentation (comments in code) for the new API calls.
- Removed the unnecessary writes to xenstore.
Changes in v2:
- Instead of killing the guest and hiding the device, forcibly remove
the device from the guest.
Venu Bu
Implement the callback function to handle unrecoverable AER errors, and
also the public APIs that can be used to register/unregister the handler.
When an AER error occurs, the handler will forcibly remove the erring
PCIe device from the guest.
Signed-off-by: Venu Busireddy
---
tools/libxl
On 2017-08-08 15:33:01 +0100, Wei Liu wrote:
> On Mon, Aug 07, 2017 at 06:54:56PM -0500, Venu Busireddy wrote:
> > Implement the callback function to handle unrecoverable AER errors, and
> > also the public APIs that can be used to register/unregister the handler.
> > When an
When a guest is created, register the AER event handler to handle the
AER errors. When an AER error occurs, the handler will forcibly remove
the erring PCIe device from the guest.
Signed-off-by: Venu Busireddy
---
tools/xl/xl_vmcontrol.c | 9 +
1 file changed, 9 insertions(+)
diff
Implement the callback function to handle unrecoverable AER errors, and
also the public APIs that can be used to register/unregister the handler.
When an AER error occurs, the handler will forcibly remove the erring
PCIe device from the guest.
Signed-off-by: Venu Busireddy
---
tools/libxl
nges in v2:
- Instead of killing the guest and hiding the device, forcibly remove
the device from the guest.
Venu Busireddy (2):
libxl: Implement the handler to handle unrecoverable AER errors.
xl: Register the AER event handler to handle AER errors.
tools/libxl/libxl.h
When a guest is created, register the AER event handler to handle the
AER errors. When an AER error occurs, the handler will forcibly remove
the erring PCIe device from the guest.
Signed-off-by: Venu Busireddy
---
tools/xl/xl_vmcontrol.c | 9 +
1 file changed, 9 insertions(+)
diff
Implement the callback function to handle unrecoverable AER errors, and
also the public APIs that can be used to register/unregister the handler.
When an AER error occurs, the handler will forcibly remove the erring
PCIe device from the guest.
Signed-off-by: Venu Busireddy
---
tools/libxl
nges in v2:
- Instead of killing the guest and hiding the device, forcibly remove
the device from the guest.
Venu Busireddy (2):
libxl: Implement the handler to handle unrecoverable AER errors.
xl: Register the AER event handler to handle AER errors.
tools/libxl/libxl.h
- Added documentation (comments in code) for the new API calls.
- Removed the unnecessary writes to xenstore.
Changes in v2:
- Instead of killing the guest and hiding the device, forcibly remove
the device from the guest.
Venu Busireddy (2):
libxl: Implement the handler to handle unrecoverable
When a guest is created, register the AER event handler to handle the
AER errors. When an AER error occurs, the handler will forcibly remove
the erring PCIe device from the guest.
Signed-off-by: Venu Busireddy
---
tools/xl/xl_vmcontrol.c | 9 +
1 file changed, 9 insertions(+)
diff
Implement the callback function to handle unrecoverable AER errors, and
also the public APIs that can be used to register/unregister the handler.
When an AER error occurs, the handler will forcibly remove the erring
PCIe device from the guest.
Signed-off-by: Venu Busireddy
---
tools/libxl
On 2017-07-28 17:39:52 +0100, Ian Jackson wrote:
> Venu Busireddy writes ("[PATCH v2 1/2] libxl: Implement the handler to handle
> unrecoverable AER errors."):
> > Implement the callback function to handle unrecoverable AER errors, and
> > also the public APIs
On 2017-07-28 16:58:13 +0100, Wei Liu wrote:
> On Wed, Jul 26, 2017 at 07:16:38PM -0500, Venu Busireddy wrote:
> > Implement the callback function to handle unrecoverable AER errors, and
> > also the public APIs that can be used to register/unregister the handler.
> > When an
On 2017-07-28 16:58:16 +0100, Wei Liu wrote:
> On Wed, Jul 26, 2017 at 07:16:39PM -0500, Venu Busireddy wrote:
> > When a guest is created, register the AER event handler to handle the
> > AER errors. When an AER error occurs, the handler will forcibly remove
> > the erring
Implement the callback function to handle unrecoverable AER errors, and
also the public APIs that can be used to register/unregister the handler.
When an AER error occurs, the handler will forcibly remove the erring
PCIe device from the guest.
Signed-off-by: Venu Busireddy
---
tools/libxl
When a guest is created, register the AER event handler to handle the
AER errors. When an AER error occurs, the handler will forcibly remove
the erring PCIe device from the guest.
Signed-off-by: Venu Busireddy
---
tools/xl/xl_vmcontrol.c | 11 +++
1 file changed, 11 insertions(+)
diff
patch set is:
https://lists.xen.org/archives/html/xen-devel/2017-06/msg03274.html
Changes in v2:
- Instead of killing the guest and hiding the device, forcibly remove
the device from the guest.
Venu Busireddy (2):
libxl: Implement the handler to handle unrecoverable AER errors.
xl: Register
On 2017-07-10 01:52:27 -0600, Jan Beulich wrote:
> >>> On 07.07.17 at 20:11, wrote:
> > On 2017-07-06 02:45:18 -0600, Jan Beulich wrote:
> >> I think so, but I may be missing parts of your reasoning as to why
> >> hiding the device may be a good thing.
> >
> > Here is the rationale behind hiding
On 2017-07-07 12:00:26 +0100, Wei Liu wrote:
> On Thu, Jul 06, 2017 at 02:45:18AM -0600, Jan Beulich wrote:
> > >>> On 05.07.17 at 21:38, wrote:
> > > On 2017-07-04 09:46:58 -0600, Jan Beulich wrote:
> > >> >>> On 27.06.17 at 19:14, wrote:
> > >>
> > >> First of all, please Cc all maintainers of
On 2017-07-06 02:45:18 -0600, Jan Beulich wrote:
> >>> On 05.07.17 at 21:38, wrote:
> > On 2017-07-04 09:46:58 -0600, Jan Beulich wrote:
> >> >>> On 27.06.17 at 19:14, wrote:
> >>
> >> First of all, please Cc all maintainers of code you modify.
> >
> > I was using the names spit out by the scri
On 2017-06-30 11:18:15 +0100, Wei Liu wrote:
> On Tue, Jun 27, 2017 at 12:14:56PM -0500, Venu Busireddy wrote:
> > libxl: Add wrappers for new commands and add AER error handler
>
> Extraneous line.
My mistake. Will remove it.
> >
> > Add wrappers for the new
On 2017-06-30 11:18:10 +0100, Wei Liu wrote:
> I haven't reviewed the code in detail, but I have some questions
> regarding the design. See the end of this email.
>
> On Tue, Jun 27, 2017 at 12:14:54PM -0500, Venu Busireddy wrote:
> >
> > +static void p
On 2017-07-04 09:46:58 -0600, Jan Beulich wrote:
> >>> On 27.06.17 at 19:14, wrote:
>
> First of all, please Cc all maintainers of code you modify.
I was using the names spit out by the scripts/get_maintainer.pl script
for the patch file. I didn't know that the script had a "-f" option, and
with
On 2017-06-20 12:56:34 +0100, Wei Liu wrote:
> On Wed, Jun 07, 2017 at 02:24:32PM -0500, Venu Busireddy wrote:
> >
> > Hi,
> >
> > I am working on creating a patch to aid in containing the unrecoverable
> > AER errors generated by PCI devices assigned
error handling needs to be delegated to the host operating system.
Signed-off-by: Venu Busireddy
Signed-off-by: Elena Ufimtseva
---
drivers/xen/xen-pciback/pci_stub.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/drivers/xen/xen-pciback/pci_stub.c
b/drivers/xen/xen-pciback/p
docs: Document the new commands.
Add documentation for the newly added commands "pci-assignable-hide",
"pci-assignable-unhide", and "pci-assignable-list-hidden".
Signed-off-by: Venu Busireddy
---
docs/man/xl.pod.1.in | 24
1 file changed,
libxl: Add wrappers for new commands and add AER error handler
Add wrappers for the newly introduced commands "pci-assignable-hide",
"pci-assignable-unhide", and "pci-assignable-list-hidden".
Implement the callback function to handle unrecoverable AER errors.
tools/python/xc: Update pyxc_methods with new commands
Add pyxc_unhide_device() and pyxc_hide_device(), and update pyxc_methods.
Signed-off-by: Venu Busireddy
---
tools/python/xen/lowlevel/xc/xc.c | 84 +++
1 file changed, 84 insertions(+)
diff --git a
s.
Changed create_domain() to register a handler for unrecoverable AER
errors.
Signed-off-by: Venu Busireddy
---
tools/xl/xl.h | 3 ++
tools/xl/xl_cmdtable.c | 17 +++
tools/xl/xl_pci.c | 125 +++-
tools/xl/xl_vmcontrol.c |
assign_device, deassign_device, and test_assign_device hooks.
Signed-off-by: Venu Busireddy
Signed-off-by: Elena Ufimtseva
---
xen/common/domctl.c | 6 ++
xen/drivers/passthrough/pci.c | 161 ++--
xen/include/public/domctl.h | 3 +
xen
patch set to work,
the AER error handling needs to be delegated to the host operating system.
Venu Busireddy (6):
xen: Add support for hiding and unhiding pcie passthrough devices
xl: Add commands for hiding and unhiding pcie passthrough devices
libxc: Add wrappers for new commands
libxl
libxc: Add wrappers for new commands
Add wrappers for the newly introduced commands "pci-assignable-hide",
"pci-assignable-unhide", and "pci-assignable-list-hidden".
Signed-off-by: Venu Busireddy
---
tools/libxc/include/xenctrl.h | 4
tool
On 2017-06-22 11:52:50 -0400, Konrad Rzeszutek Wilk wrote:
> On Thu, Jun 22, 2017 at 09:31:50AM -0600, Jan Beulich wrote:
> > >>> On 22.06.17 at 16:21, wrote:
> > > On Thu, Jun 22, 2017 at 03:26:04AM -0600, Jan Beulich wrote:
> > > On 21.06.17 at 12:47, wrote:
> > >>> The problem is a VF of R
Hi,
I am working on creating a patch to aid in containing the unrecoverable
AER errors generated by PCI devices assigned to guests in passthrough
mode.
The overall approach is as follows:
1. Change the BIOS settings such that the AER error handling is delegated
to the host.
2. Change the xe
> -Original Message-
> From: Konrad Rzeszutek Wilk
> Sent: Tuesday, March 21, 2017 08:19 AM
> To: Xuquan (Quan Xu); Venu Busireddy
> Cc: Jan Beulich; anthony.per...@citrix.com; george.dun...@eu.citrix.com;
> ian.jack...@eu.citrix.com; Fanhenglong; Kevin Tian; Stefan
On Wed, Mar 15, 2017 at 04:38:39PM +, Roger Pau Monn? wrote:
> On Wed, Mar 15, 2017 at 10:11:35AM -0500, Venu Busireddy wrote:
> > On Wed, Mar 15, 2017 at 12:56:50PM +, Roger Pau Monn? wrote:
> > > On Wed, Mar 15, 2017 at 08:42:04AM -0400, Konrad Rzeszutek Wilk wrote:
&
On Wed, Mar 15, 2017 at 12:56:50PM +, Roger Pau Monn? wrote:
> On Wed, Mar 15, 2017 at 08:42:04AM -0400, Konrad Rzeszutek Wilk wrote:
> > On Wed, Mar 15, 2017 at 12:07:28PM +, Roger Pau Monn? wrote:
> > > On Fri, Mar 10, 2017 at 10:28:43AM -0500, Konrad Rzeszutek Wilk wrote:
> > > > On Fri,
As some users have suggested, elaborate the usage of RMRR specification
on the command line, and provide a usage example.
Also, always treat the specified page numbers as hexadecimal values.
Signed-off-by: Venu Busireddy
---
Changes in v2:
- Provide a better usage example.
- Treat input
On Wed, Feb 22, 2017 at 01:52:02AM -0700, Jan Beulich wrote:
> >>> On 21.02.17 at 21:42, wrote:
> > --- a/docs/misc/xen-command-line.markdown
> > +++ b/docs/misc/xen-command-line.markdown
> > @@ -1406,6 +1406,15 @@ If segment of the first device is not specified,
> > segment zero will be used.
>
As some users have suggested, elaborate the usage of RMRR specification
on the command line, and provide a usage example.
Signed-off-by: Venu Busireddy
---
docs/misc/xen-command-line.markdown |9 +
1 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/docs/misc/xen-command
On Fri, Feb 17, 2017 at 02:29:32PM -0500, Konrad Rzeszutek Wilk wrote:
> On Fri, Feb 17, 2017 at 12:27:25PM -0700, Tamas K Lengyel wrote:
> > On Fri, Feb 17, 2017 at 11:56 AM, Konrad Rzeszutek Wilk
> > wrote:
> > > . snip..
> > >> >> > Given this commit is pretty old, I'm also curious why it's onl
Adjust/manage the return values of register_one_rmrr() such that new
callers log errors for non-debug builds too, while not affecting the
behavior of the original callers.
Signed-off-by: Venu Busireddy
---
xen/drivers/passthrough/vtd/dmar.c | 11 +++
1 file changed, 11 insertions
Adjust/manage the return values of register_one_rmrr() such that new
callers log errors for non-debug builds too, while not affecting the
behavior of original callers.
Signed-off-by: Venu Busireddy
---
xen/drivers/passthrough/vtd/dmar.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions
On Tue, Jan 31, 2017 at 08:56:09AM -0700, Jan Beulich wrote:
> >>> On 31.01.17 at 16:22, wrote:
> > On Tue, Jan 31, 2017 at 02:55:50AM -0700, Jan Beulich wrote:
> >> >>> On 30.01.17 at 22:09, wrote:
> >> > On Mon, Jan 30, 2017 at 03:39:23AM -0700, Jan Beulich wrote:
> >> >> I notice, however, tha
Jan,
Sure. I will look in to it.
Venu
> -Original Message-
> From: Jan Beulich [mailto:jbeul...@suse.com]
> Sent: Monday, January 30, 2017 04:39 AM
> To: Andrew Cooper; Elena Ufimtseva; Venu Busireddy
> Cc: Xen-devel
> Subject: Re: [PATCH] VT-d/RMRR: Avoid me
> -Original Message-
> From: Jan Beulich [mailto:jbeul...@suse.com]
> Sent: Tuesday, January 24, 2017 10:08 AM
> To: Venu Busireddy
> Cc: Feng Wu; Kevin Tian; xen-devel@lists.xen.org; Elena Ufimtseva; Konrad
> Rzeszutek Wilk
> Subject: Re: [PATCH v14 3/3] iommu: add
On Tue, Jan 24, 2017 at 08:52:50AM -0700, Jan Beulich wrote:
> >>> On 24.01.17 at 16:35, wrote:
> > On Tue, Jan 24, 2017 at 01:46:44AM -0700, Jan Beulich wrote:
> >> >>> On 23.01.17 at 19:20, wrote:
> >> > +overlap = false;
> >> > +list_for_each_entry(rmrru, &acpi_rmrr_units, list
On Tue, Jan 24, 2017 at 01:46:44AM -0700, Jan Beulich wrote:
> >>> On 23.01.17 at 19:20, wrote:
> > +overlap = false;
> > +list_for_each_entry(rmrru, &acpi_rmrr_units, list)
> > +{
> > +if ( pfn_to_paddr(base) <= rmrru->end_address &&
> > + rmrru
, refer to grub2 manual for more information.
Signed-off-by: Elena Ufimtseva
Signed-off-by: Venu Busireddy
Acked-by: Kevin Tian
---
docs/misc/xen-command-line.markdown | 13 +++
xen/drivers/passthrough/vtd/dmar.c | 203 +++-
2 files changed, 215 insertions(+
Elena Ufimtseva
Signed-off-by: Venu Busireddy
Acked-by: Kevin Tian
---
xen/drivers/pci/pci.c | 11 +++
xen/include/xen/pci.h | 3 +++
2 files changed, 14 insertions(+)
Changes in v14:
- No new changes. Included this patch file for the sake of completeness.
diff --git a/xen/drivers/pci/pc
In preparation for auxiliary RMRR data provided on Xen command line,
make RMRR adding a separate function.
Also free memery for rmrr device scope in error path.
Signed-off-by: Elena Ufimtseva
Signed-off-by: Venu Busireddy
Acked-by: Kevin Tian
---
xen/drivers/passthrough/vtd/dmar.c | 123
Add Xen command line option rmrr to specify RMRR regions that are not
defined in ACPI thus causing IO Page Faults and prevent dom0 from booting
if "iommu=dom0-strict" option is specified on the Xen command line.
These additional regions will be added to the list of RMRR regions parsed
from ACPI.
C
On Wed, Jan 11, 2017 at 05:55:58AM +, Tian, Kevin wrote:
> > From: Venu Busireddy
> > Sent: Wednesday, January 11, 2017 6:58 AM
> >
> > From: Elena Ufimtseva
> >
> > Add Xen command line option rmrr to specify RMRR regions that are not
> > defin
From: Elena Ufimtseva
Add Xen command line option rmrr to specify RMRR regions that are not
defined in ACPI thus causing IO Page Faults and prevent dom0 from booting
if "iommu=dom0-strict" option is specified on the Xen command line.
These additional regions will be added to the list of RMRR regi
uld be
quoted/escaped, refer to grub2 manual for more information.
Signed-off-by: Elena Ufimtseva
Signed-off-by: Venu Busireddy
---
docs/misc/xen-command-line.markdown | 13 +++
xen/drivers/passthrough/vtd/dmar.c | 201 +++-
2 files changed, 213 insertions(+
ected.
Signed-off-by: Elena Ufimtseva
Signed-off-by: Venu Busireddy
---
xen/drivers/pci/pci.c | 11 +++
xen/include/xen/pci.h | 3 +++
2 files changed, 14 insertions(+)
v13:
- Renamed __parse_pci() to parse_pci_seg() as suggested by Konrad Wilk.
diff --git a/xen/drivers/pci/pci.c
From: Elena Ufimtseva
In preparation for auxiliary RMRR data provided on Xen command line,
make RMRR adding a separate function.
Also free memery for rmrr device scope in error path.
Signed-off-by: Elena Ufimtseva
Signed-off-by: Venu Busireddy
Acked-by: Kevin Tian
---
xen/drivers
62 matches
Mail list logo