VMCI Context code maintains state for vmci and allows the driver to communicate
with multiple VMs.
Signed-off-by: George Zhang
Acked-by: Andy king
Acked-by: Dmitry Torokhov
---
drivers/misc/vmw_vmci/vmci_context.c | 1214 ++
drivers/misc/vmw_vmci/vmci_context.h
VMCI Context code maintains state for vmci and allows the driver to communicate
with multiple VMs.
Signed-off-by: George Zhang georgezh...@vmware.com
Acked-by: Andy king ack...@vmware.com
Acked-by: Dmitry Torokhov d...@vmware.com
---
drivers/misc/vmw_vmci/vmci_context.c | 1214
On Wed, 2012-11-21 at 13:29 -0800, Dmitry Torokhov wrote:
> Hi Joe,
Howdy.
> On Wed, Nov 21, 2012 at 01:04:46PM -0800, Joe Perches wrote:
> > On Wed, 2012-11-21 at 12:31 -0800, George Zhang wrote:
> > > + context = kzalloc(sizeof(*context), GFP_KERNEL);
> > > + if (!context) {
> > > +
Hi Joe,
On Wed, Nov 21, 2012 at 01:04:46PM -0800, Joe Perches wrote:
> On Wed, 2012-11-21 at 12:31 -0800, George Zhang wrote:
> > + context = kzalloc(sizeof(*context), GFP_KERNEL);
> > + if (!context) {
> > + pr_warn("Failed to allocate memory for VMCI context.\n");
>
> OOM logging
Hi Joe,
On Wed, Nov 21, 2012 at 01:04:46PM -0800, Joe Perches wrote:
On Wed, 2012-11-21 at 12:31 -0800, George Zhang wrote:
+ context = kzalloc(sizeof(*context), GFP_KERNEL);
+ if (!context) {
+ pr_warn(Failed to allocate memory for VMCI context.\n);
OOM logging messages
On Wed, 2012-11-21 at 13:29 -0800, Dmitry Torokhov wrote:
Hi Joe,
Howdy.
On Wed, Nov 21, 2012 at 01:04:46PM -0800, Joe Perches wrote:
On Wed, 2012-11-21 at 12:31 -0800, George Zhang wrote:
+ context = kzalloc(sizeof(*context), GFP_KERNEL);
+ if (!context) {
+ pr_warn(Failed
Hi Joe,
> Just some trivial notes.
Thanks for taking a look!
> > + pr_warn("Failed to allocate memory for VMCI context.\n");
>
> OOM logging messages aren't necessary as alloc failures
> are already logged with a stack trace.
Noted, we'll remove all such occurrences.
> Maybe just
On Wed, 2012-11-21 at 12:31 -0800, George Zhang wrote:
> VMCI Context code maintains state for vmci and allows the driver to
> communicate
> with multiple VMs
Just some trivial notes.
> diff --git a/drivers/misc/vmw_vmci/vmci_context.c
> b/drivers/misc/vmw_vmci/vmci_context.c
[]
It'd be nicer
VMCI Context code maintains state for vmci and allows the driver to communicate
with multiple VMs.
Signed-off-by: George Zhang
Signed-off-by: Dmitry Torokhov
Signed-off-by: Andy King
---
drivers/misc/vmw_vmci/vmci_context.c | 1223 ++
VMCI Context code maintains state for vmci and allows the driver to communicate
with multiple VMs.
Signed-off-by: George Zhang georgezh...@vmware.com
Signed-off-by: Dmitry Torokhov d...@vmware.com
Signed-off-by: Andy King ack...@vmware.com
---
drivers/misc/vmw_vmci/vmci_context.c | 1223
On Wed, 2012-11-21 at 12:31 -0800, George Zhang wrote:
VMCI Context code maintains state for vmci and allows the driver to
communicate
with multiple VMs
Just some trivial notes.
diff --git a/drivers/misc/vmw_vmci/vmci_context.c
b/drivers/misc/vmw_vmci/vmci_context.c
[]
It'd be nicer if
Hi Joe,
Just some trivial notes.
Thanks for taking a look!
+ pr_warn(Failed to allocate memory for VMCI context.\n);
OOM logging messages aren't necessary as alloc failures
are already logged with a stack trace.
Noted, we'll remove all such occurrences.
Maybe just use
VMCI Context code maintains state for vmci and allows the driver to communicate
with
multiple VMs.
Signed-off-by: George Zhang
---
drivers/misc/vmw_vmci/vmci_context.c | 1246 ++
drivers/misc/vmw_vmci/vmci_context.h | 183 +
2 files changed, 1429
VMCI Context code maintains state for vmci and allows the driver to communicate
with
multiple VMs.
Signed-off-by: George Zhang georgezh...@vmware.com
---
drivers/misc/vmw_vmci/vmci_context.c | 1246 ++
drivers/misc/vmw_vmci/vmci_context.h | 183 +
2 files
VMCI Context code maintains state for vmci and allows the driver to communicate
with multiple VMs.
Signed-off-by: George Zhang
---
drivers/misc/vmw_vmci/vmci_context.c | 1247 ++
drivers/misc/vmw_vmci/vmci_context.h | 183 +
2 files changed, 1430
VMCI Context code maintains state for vmci and allows the driver to communicate
with multiple VMs.
Signed-off-by: George Zhang georgezh...@vmware.com
---
drivers/misc/vmw_vmci/vmci_context.c | 1247 ++
drivers/misc/vmw_vmci/vmci_context.h | 183 +
2 files
On Tuesday, October 30, 2012 08:46:52 AM Greg KH wrote:
> On Mon, Oct 29, 2012 at 09:01:40PM -0700, Dmitry Torokhov wrote:
> > Hi Greg,
> >
> > On Mon, Oct 29, 2012 at 07:10:58PM -0700, Greg KH wrote:
> > > On Mon, Oct 29, 2012 at 06:03:42PM -0700, George Zhang wrote:
> > > > +/*
> > > > + *
On Mon, Oct 29, 2012 at 09:01:40PM -0700, Dmitry Torokhov wrote:
> Hi Greg,
>
> On Mon, Oct 29, 2012 at 07:10:58PM -0700, Greg KH wrote:
> > On Mon, Oct 29, 2012 at 06:03:42PM -0700, George Zhang wrote:
> > > +/*
> > > + * Releases the VMCI context. If this is the last reference to
> > > + * the
On Mon, Oct 29, 2012 at 09:01:40PM -0700, Dmitry Torokhov wrote:
Hi Greg,
On Mon, Oct 29, 2012 at 07:10:58PM -0700, Greg KH wrote:
On Mon, Oct 29, 2012 at 06:03:42PM -0700, George Zhang wrote:
+/*
+ * Releases the VMCI context. If this is the last reference to
+ * the context it
On Tuesday, October 30, 2012 08:46:52 AM Greg KH wrote:
On Mon, Oct 29, 2012 at 09:01:40PM -0700, Dmitry Torokhov wrote:
Hi Greg,
On Mon, Oct 29, 2012 at 07:10:58PM -0700, Greg KH wrote:
On Mon, Oct 29, 2012 at 06:03:42PM -0700, George Zhang wrote:
+/*
+ * Releases the VMCI
Hi Greg,
On Mon, Oct 29, 2012 at 07:10:58PM -0700, Greg KH wrote:
> On Mon, Oct 29, 2012 at 06:03:42PM -0700, George Zhang wrote:
> > +/*
> > + * Releases the VMCI context. If this is the last reference to
> > + * the context it will be deallocated. A context is created with
> > + * a reference
On Mon, Oct 29, 2012 at 06:03:42PM -0700, George Zhang wrote:
> +/*
> + * Releases the VMCI context. If this is the last reference to
> + * the context it will be deallocated. A context is created with
> + * a reference count of one, and on destroy, it is removed from
> + * the context list before
On Mon, Oct 29, 2012 at 06:03:42PM -0700, George Zhang wrote:
> VMCI Context code maintains state for vmci and allows the driver to
> communicate with multiple VMs.
>
Very minor nit, wrap your changelog comments at 72 columns or at the
least, 80. Didn't checkpatch complain about that?
> +/*
>
VMCI Context code maintains state for vmci and allows the driver to communicate
with multiple VMs.
Signed-off-by: George Zhang
---
drivers/misc/vmw_vmci/vmci_context.c | 1290 ++
drivers/misc/vmw_vmci/vmci_context.h | 177 +
2 files changed, 1467
VMCI Context code maintains state for vmci and allows the driver to communicate
with multiple VMs.
Signed-off-by: George Zhang georgezh...@vmware.com
---
drivers/misc/vmw_vmci/vmci_context.c | 1290 ++
drivers/misc/vmw_vmci/vmci_context.h | 177 +
2 files
On Mon, Oct 29, 2012 at 06:03:42PM -0700, George Zhang wrote:
VMCI Context code maintains state for vmci and allows the driver to
communicate with multiple VMs.
Very minor nit, wrap your changelog comments at 72 columns or at the
least, 80. Didn't checkpatch complain about that?
+/*
+ *
On Mon, Oct 29, 2012 at 06:03:42PM -0700, George Zhang wrote:
+/*
+ * Releases the VMCI context. If this is the last reference to
+ * the context it will be deallocated. A context is created with
+ * a reference count of one, and on destroy, it is removed from
+ * the context list before its
Hi Greg,
On Mon, Oct 29, 2012 at 07:10:58PM -0700, Greg KH wrote:
On Mon, Oct 29, 2012 at 06:03:42PM -0700, George Zhang wrote:
+/*
+ * Releases the VMCI context. If this is the last reference to
+ * the context it will be deallocated. A context is created with
+ * a reference count of
28 matches
Mail list logo