similar to: linux-next: Tree for Dec 19 (xen)

Hi, I was reading some code and had a question in xenbus_file_write() drivers/xen/xenbus/xenbus_dev_frontend.c 461 if ((len + u->len) > sizeof(u->u.buffer)) { ^^^^^^^^^^^^ Can this addition overflow? Should the test be something like: if (len > sizeof(u->u.buffer) || len + u->len > sizeof(u->u.buffer)) { 462 /* On

Hi, I was reading some code and had a question in xenbus_file_write() drivers/xen/xenbus/xenbus_dev_frontend.c 461 if ((len + u->len) > sizeof(u->u.buffer)) { ^^^^^^^^^^^^ Can this addition overflow? Should the test be something like: if (len > sizeof(u->u.buffer) || len + u->len > sizeof(u->u.buffer)) { 462 /* On

Hi, I was reading some code and had a question in xenbus_file_write() drivers/xen/xenbus/xenbus_dev_frontend.c 461 if ((len + u->len) > sizeof(u->u.buffer)) { ^^^^^^^^^^^^ Can this addition overflow? Should the test be something like: if (len > sizeof(u->u.buffer) || len + u->len > sizeof(u->u.buffer)) { 462 /* On

On 10/23/2012 09:19 PM, Stephen Rothwell wrote: > Hi all, > > Changes since 201201023: > on x86_64: drivers/built-in.o: In function `dbgp_reset_prep': (.text+0xb96b5): undefined reference to `xen_dbgp_reset_prep' drivers/built-in.o: In function `dbgp_external_startup': (.text+0xb9d95): undefined reference to `xen_dbgp_external_startup' Full randconfig file is

On 10/23/2012 09:19 PM, Stephen Rothwell wrote: > Hi all, > > Changes since 201201023: > on x86_64: drivers/built-in.o: In function `dbgp_reset_prep': (.text+0xb96b5): undefined reference to `xen_dbgp_reset_prep' drivers/built-in.o: In function `dbgp_external_startup': (.text+0xb9d95): undefined reference to `xen_dbgp_external_startup' Full randconfig file is

On 10/23/2012 09:19 PM, Stephen Rothwell wrote: > Hi all, > > Changes since 201201023: > on x86_64: drivers/built-in.o: In function `dbgp_reset_prep': (.text+0xb96b5): undefined reference to `xen_dbgp_reset_prep' drivers/built-in.o: In function `dbgp_external_startup': (.text+0xb9d95): undefined reference to `xen_dbgp_external_startup' Full randconfig file is

Dear xen-devel, There''s a race condition in xenfs (xenstore driver) that causes userspace utility xenstore-watch to crash. Normally, the userspace process gets an "OK" from xenfs and then the watch fires immediately after. Occasionally, this happens the other way around: the watch fires before the driver sends "OK", which confuses the xenstore-watch client. It seems

So far only the watch path was checked to be zero terminated, while the watch token was merely assumed to be. Signed-off-by: Jan Beulich <jbeulich@suse.com> --- drivers/xen/xenbus/xenbus_dev_frontend.c | 4 ++++ 1 file changed, 4 insertions(+) --- 3.3-rc1/drivers/xen/xenbus/xenbus_dev_frontend.c +++ 3.3-rc1-xenbus-dev-write-watch/drivers/xen/xenbus/xenbus_dev_frontend.c @@ -369,6

On Wed, 15 Jun 2011 14:17:28 +1000 Stephen Rothwell wrote: > Hi all, > > Changes since 20110614: on x86_64: arch/x86/xen/mmu.c: In function 'xen_flush_tlb_others': arch/x86/xen/mmu.c:1234: error: 'num_processors' undeclared (first use in this function) kernel .config file is attached. (CONFIG_SMP is not enabled.) --- ~Randy *** Remember to use

On Wed, 15 Jun 2011 14:17:28 +1000 Stephen Rothwell wrote: > Hi all, > > Changes since 20110614: on x86_64: arch/x86/xen/mmu.c: In function 'xen_flush_tlb_others': arch/x86/xen/mmu.c:1234: error: 'num_processors' undeclared (first use in this function) kernel .config file is attached. (CONFIG_SMP is not enabled.) --- ~Randy *** Remember to use

On Wed, 15 Jun 2011 14:17:28 +1000 Stephen Rothwell wrote: > Hi all, > > Changes since 20110614: on x86_64: arch/x86/xen/mmu.c: In function 'xen_flush_tlb_others': arch/x86/xen/mmu.c:1234: error: 'num_processors' undeclared (first use in this function) kernel .config file is attached. (CONFIG_SMP is not enabled.) --- ~Randy *** Remember to use

Hi, This patch series fixes the S3 resume of a domain running xenstored and a frontend over xenbus (xen-netfront in my use case). As device resume is happening before process resume, the xenbus frontend resume is hanging if xenstored is not running, thus causing a deadlock. This patch series is fixing that issue by deferring the xenbus frontend resume when we are running xenstored in that same

On Fri, Dec 12, 2014 at 08:07:05AM +1100, Stephen Rothwell wrote: > Hi Michael, > > On Thu, 11 Dec 2014 14:02:48 +0200 "Michael S. Tsirkin" <mst at redhat.com> wrote: > > > > The following changes since commit b2776bf7149bddd1f4161f14f79520f17fc1d71d: > > > > Linux 3.18 (2014-12-07 14:21:05 -0800) > > hmmm ... > > > are

On Fri, Dec 12, 2014 at 08:07:05AM +1100, Stephen Rothwell wrote: > Hi Michael, > > On Thu, 11 Dec 2014 14:02:48 +0200 "Michael S. Tsirkin" <mst at redhat.com> wrote: > > > > The following changes since commit b2776bf7149bddd1f4161f14f79520f17fc1d71d: > > > > Linux 3.18 (2014-12-07 14:21:05 -0800) > > hmmm ... > > > are

Christoph Hellwig said runs sparse: 1) page_tables.c unnecessary initialization 2) Change prototype of run_lguest and do cast in caller instead (when we add __user to cast, it runs over another line). Al Viro pointed out the ugly cast in push_lguest_stack(): 3) Stick with unsigned long for arg, removes 4 casts in total. Most importantly, I now realize that Christoph's incorrect ranting

Christoph Hellwig said runs sparse: 1) page_tables.c unnecessary initialization 2) Change prototype of run_lguest and do cast in caller instead (when we add __user to cast, it runs over another line). Al Viro pointed out the ugly cast in push_lguest_stack(): 3) Stick with unsigned long for arg, removes 4 casts in total. Most importantly, I now realize that Christoph's incorrect ranting

Hi Stefano, [just casting the net a bit wider ...] On Tue, 19 Oct 2010 18:51:47 +0100 Stefano Stabellini <stefano.stabellini at eu.citrix.com> wrote: > > I forgot to CC the LKML and linux-next... > > On Tue, 19 Oct 2010, Stefano Stabellini wrote: > > Stephen, > > I have two patch series to merge in linux-next: > > > > PV on HVM: receive interrupts as

Hi Stefano, [just casting the net a bit wider ...] On Tue, 19 Oct 2010 18:51:47 +0100 Stefano Stabellini <stefano.stabellini at eu.citrix.com> wrote: > > I forgot to CC the LKML and linux-next... > > On Tue, 19 Oct 2010, Stefano Stabellini wrote: > > Stephen, > > I have two patch series to merge in linux-next: > > > > PV on HVM: receive interrupts as

Hi Stefano, [just casting the net a bit wider ...] On Tue, 19 Oct 2010 18:51:47 +0100 Stefano Stabellini <stefano.stabellini at eu.citrix.com> wrote: > > I forgot to CC the LKML and linux-next... > > On Tue, 19 Oct 2010, Stefano Stabellini wrote: > > Stephen, > > I have two patch series to merge in linux-next: > > > > PV on HVM: receive interrupts as

On 12/20/2011 10:47 PM, Stephen Rothwell wrote: > Hi all, > > Changes since 20111220: drivers/xen/xenbus/xenbus_dev_backend.c:74:2: error: implicit declaration of function ''xen_initial_domain'' Full randconfig file is attached. -- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code ***