Re: CVS commit: src/sys/arch/xen/xen

2020-06-24 Thread Taylor R Campbell
> Module Name:src > Committed By: jdolecek > Date: Fri Apr 10 18:03:06 UTC 2020 > > Modified Files: > src/sys/arch/xen/xen: if_xennet_xenbus.c > > Log Message: > convert to bus_dma(9), remove now not necessary XENPVHVM redefines > [...] > + KASSERT(req->rxreq

Re: CVS commit: src/sys/arch/xen/xen

2020-04-13 Thread Manuel Bouyer
On Mon, Apr 13, 2020 at 08:09:13PM +, Jaromir Dolecek wrote: > Module Name: src > Committed By: jdolecek > Date: Mon Apr 13 20:09:13 UTC 2020 > > Modified Files: > src/sys/arch/xen/xen: xbd_xenbus.c > > Log Message: > KASSERT() that requested I/O size is <= XBD_MAX_XFER - this

Re: CVS commit: src/sys/arch/xen/xen

2020-04-09 Thread Frank Kardel
HI, I am not sur whether it is the commit below, but 2 out 4 times my xen-DOMU from today (20200409/9.99.55) panics with following locking botch: [ 29.9301379] panic: kernel diagnostic assertion "IFNET_LOCKED(ifp)" failed: file "/usr/src/sys/arch/xen/xen/if_xennet_xenbus.c", line 1120 [ 2

Re: CVS commit: src/sys/arch/xen/xen

2020-04-05 Thread Manuel Bouyer
On Sun, Apr 05, 2020 at 05:26:47PM +, Jaromir Dolecek wrote: > Module Name: src > Committed By: jdolecek > Date: Sun Apr 5 17:26:47 UTC 2020 > > Modified Files: > src/sys/arch/xen/xen: if_xennet_xenbus.c xennetback_xenbus.c > > Log Message: > remove support for legacy rx-flip

Re: CVS commit: src/sys/arch/xen/xen

2014-10-17 Thread Maxime Villard
Le 10/10/2014 18:42, Maxime Villard a écrit : Le 03/10/2014 22:56, Christos Zoulas a écrit : Module Name:src Committed By:christos Date:Fri Oct 3 20:56:24 UTC 2014 Modified Files: src/sys/arch/xen/xen: privcmd.c Log Message: correct error paths; still need to verify that

Re: CVS commit: src/sys/arch/xen/xen

2014-10-17 Thread Christos Zoulas
On Oct 17, 6:02pm, m...@m00nbsd.net (Maxime Villard) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | > Don't you think that you should also revert r1.46 [1]? Now that | > privcmd_map_obj() frees everything correctly... Yes, I was going to and I forgot; done now. christos

Re: CVS commit: src/sys/arch/xen/xen

2014-10-10 Thread Maxime Villard
Le 03/10/2014 22:56, Christos Zoulas a écrit : Module Name:src Committed By: christos Date: Fri Oct 3 20:56:24 UTC 2014 Modified Files: src/sys/arch/xen/xen: privcmd.c Log Message: correct error paths; still need to verify that the "didn't give us back..." case is corr

Re: CVS commit: src/sys/arch/xen/xen

2014-10-03 Thread Maxime Villard
Le 27/09/2014 17:51, Christos Zoulas a écrit : On Sep 27, 8:36am, m...@m00nbsd.net (Maxime Villard) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | One however returns an error without freeing: | | if (newstart != start) { | printf("uvm_map didn't give u

Re: CVS commit: src/sys/arch/xen/xen

2014-09-27 Thread Christos Zoulas
On Sep 27, 10:15pm, bou...@antioche.eu.org (Manuel Bouyer) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | > if (newstart != start) { | > printf("uvm_map didn't give us back our vm space\n"); | > + uvm_unmap1(map, news

Re: CVS commit: src/sys/arch/xen/xen

2014-09-27 Thread Manuel Bouyer
On Sat, Sep 27, 2014 at 11:51:59AM -0400, Christos Zoulas wrote: > On Sep 27, 8:36am, m...@m00nbsd.net (Maxime Villard) wrote: > -- Subject: Re: CVS commit: src/sys/arch/xen/xen > > | One however returns an error without freeing: > | > | if (newstart != start) { >

Re: CVS commit: src/sys/arch/xen/xen

2014-09-27 Thread Christos Zoulas
On Sep 27, 8:36am, m...@m00nbsd.net (Maxime Villard) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | One however returns an error without freeing: | | if (newstart != start) { | printf("uvm_map didn't give us back our vm space\n"); |

Re: CVS commit: src/sys/arch/xen/xen

2014-09-27 Thread Maxime Villard
Le 21/09/2014 20:22, Christos Zoulas a écrit : On Sep 21, 7:57pm, m...@m00nbsd.net (Maxime Villard) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | Did you test this change? Verily I'm not sure it's a proper bug, but I | kept it in my list so that someone (bouyer@?) could i

Re: CVS commit: src/sys/arch/xen/xen

2014-09-21 Thread Maxime Villard
Did you test this change? Verily I'm not sure it's a proper bug, but I kept it in my list so that someone (bouyer@?) could investigate. Le 21/09/2014 18:56, Christos Zoulas a écrit : > > Module Name: src > Committed By: christos > Date: Sun Sep 21 16:56:44 UTC 2014 > > Modified Files: >

Re: CVS commit: src/sys/arch/xen/xen

2014-09-21 Thread Christos Zoulas
On Sep 21, 7:57pm, m...@m00nbsd.net (Maxime Villard) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | Did you test this change? Verily I'm not sure it's a proper bug, but I | kept it in my list so that someone (bouyer@?) could investigate. This is why I committed the fix in

Re: CVS commit: src/sys/arch/xen/xen

2014-02-02 Thread Manuel Bouyer
On Sun, Feb 02, 2014 at 08:24:08PM +, David Laight wrote: > On Sun, Feb 02, 2014 at 08:41:29PM +0100, Manuel Bouyer wrote: > > On Sun, Feb 02, 2014 at 06:53:55PM +, David Laight wrote: > > > Something needs to set the TS (task switched) flag when a new cpu > > > is added. Both amd64 and i38

Re: CVS commit: src/sys/arch/xen/xen

2014-02-02 Thread David Laight
On Sun, Feb 02, 2014 at 08:41:29PM +0100, Manuel Bouyer wrote: > On Sun, Feb 02, 2014 at 06:53:55PM +, David Laight wrote: > > Something needs to set the TS (task switched) flag when a new cpu > > is added. Both amd64 and i386 'bare metal' have direct calls to > > fpuinit() to do this. > > > >

Re: CVS commit: src/sys/arch/xen/xen

2014-02-02 Thread Manuel Bouyer
On Sun, Feb 02, 2014 at 06:53:55PM +, David Laight wrote: > > It's been a while since I looked at this and I don't remmeber the details, > > but I don't think anything of fpuinit() is neeed for Xen. in netbsd-6 > > npxattach() doens't do anything either for Xen (only set i386_fpu_present to > >

Re: CVS commit: src/sys/arch/xen/xen

2014-02-02 Thread David Laight
On Sun, Feb 02, 2014 at 01:27:45PM +0100, Manuel Bouyer wrote: > On Sat, Feb 01, 2014 at 10:49:44PM +, David Laight wrote: > > On Sat, Feb 01, 2014 at 08:07:07PM +, Manuel Bouyer wrote: > > > Module Name: src > > > Committed By: bouyer > > > Date: Sat Feb 1 20:07:07 UT

Re: CVS commit: src/sys/arch/xen/xen

2014-02-02 Thread Manuel Bouyer
On Sat, Feb 01, 2014 at 10:49:44PM +, David Laight wrote: > On Sat, Feb 01, 2014 at 08:07:07PM +, Manuel Bouyer wrote: > > Module Name:src > > Committed By: bouyer > > Date: Sat Feb 1 20:07:07 UTC 2014 > > > > Modified Files: > > src/sys/arch/xen/xen: hyper

Re: CVS commit: src/sys/arch/xen/xen

2014-02-02 Thread David Laight
On Sat, Feb 01, 2014 at 06:09:04PM -0800, John Nemeth wrote: > } > } XEN might need the bits of fpuinit() that don't play with cr0. > } In particular a fninit and setting TS. > } Although I'm not 100% sure the TS is needed on a single cpu system. > > Xen domU is SMP capable. NetBSD dom0 cur

Re: CVS commit: src/sys/arch/xen/xen

2014-02-01 Thread John Nemeth
On Feb 1, 10:49pm, David Laight wrote: } On Sat, Feb 01, 2014 at 08:07:07PM +, Manuel Bouyer wrote: } > Module Name:src } > Committed By: bouyer } > Date: Sat Feb 1 20:07:07 UTC 2014 } > } > Modified Files: } > src/sys/arch/xen/xen: hypervisor.c } > } > Log Me

Re: CVS commit: src/sys/arch/xen/xen

2014-02-01 Thread Jeff Rizzo
On 2/1/14, 2:49 PM, David Laight wrote: On Sat, Feb 01, 2014 at 08:07:07PM +, Manuel Bouyer wrote: I can't say I've actuallly looked a xen. Presumably we somewhere compile the hypervisor as part of a kernel? The hypervisor is in pkgsrc - sysutils/xenkernel42 (and the accompanying xento

Re: CVS commit: src/sys/arch/xen/xen

2014-02-01 Thread David Laight
On Sat, Feb 01, 2014 at 08:07:07PM +, Manuel Bouyer wrote: > Module Name: src > Committed By: bouyer > Date: Sat Feb 1 20:07:07 UTC 2014 > > Modified Files: > src/sys/arch/xen/xen: hypervisor.c > > Log Message: > Revert previous: calling fpuinit() leads to a panic, as a domU i

Re: CVS commit: src/sys/arch/xen/xen

2014-02-01 Thread Manuel Bouyer
On Sat, Feb 01, 2014 at 05:48:04PM +, David Laight wrote: > Module Name: src > Committed By: dsl > Date: Sat Feb 1 17:48:04 UTC 2014 > > Modified Files: > src/sys/arch/xen/xen: hypervisor.c > > Log Message: > Add a direct call to fpuinit(). > I'm not sure this is architectural

Re: CVS commit: src/sys/arch/xen/xen

2013-12-03 Thread David Laight
On Tue, Dec 03, 2013 at 08:51:00PM +, Manuel Bouyer wrote: > Module Name: src > Committed By: bouyer > Date: Tue Dec 3 20:51:00 UTC 2013 > > Modified Files: > src/sys/arch/xen/xen: evtchn.c > > Log Message: > Remove the "evtchn_do_event: handler %p didn't lower ipl %d %d\n" pr

Re: CVS commit: src/sys/arch/xen/xen

2013-12-03 Thread Jeff Rizzo
On 12/3/13 12:51 PM, Manuel Bouyer wrote: Module Name:src Committed By: bouyer Date: Tue Dec 3 20:51:00 UTC 2013 Modified Files: src/sys/arch/xen/xen: evtchn.c Log Message: Remove the "evtchn_do_event: handler %p didn't lower ipl %d %d\n" printf. With help from Robert E

Re: CVS commit: src/sys/arch/xen/xen

2012-02-21 Thread Manuel Bouyer
On Tue, Feb 21, 2012 at 09:49:40AM +0100, J. Hannken-Illjes wrote: > On Feb 21, 2012, at 2:47 AM, Jonathan A. Kollasch wrote: > > > Module Name:src > > Committed By: jakllsch > > Date: Tue Feb 21 01:47:50 UTC 2012 > > > > Modified Files: > > src/sys/arch/xen/xen: x

Re: CVS commit: src/sys/arch/xen/xen

2012-02-21 Thread J. Hannken-Illjes
On Feb 21, 2012, at 2:47 AM, Jonathan A. Kollasch wrote: > Module Name: src > Committed By: jakllsch > Date: Tue Feb 21 01:47:50 UTC 2012 > > Modified Files: > src/sys/arch/xen/xen: xbd_xenbus.c > > Log Message: > Add and use xbdminphys() to handle transfer segmentation/size limit

Re: CVS commit: src/sys/arch/xen/xen

2011-12-08 Thread Manuel Bouyer
On Thu, Dec 08, 2011 at 11:09:19AM +0100, Christoph Egger wrote: > On 12/08/11 04:34, Cherry G. Mathew wrote: > >Module Name: src > >Committed By:cherry > >Date:Thu Dec 8 03:34:48 UTC 2011 > > > >Modified Files: > > src/sys/arch/xen/xen: evtchn.c > > > >Log Message: > >

Re: CVS commit: src/sys/arch/xen/xen

2011-12-08 Thread Christoph Egger
On 12/08/11 04:34, Cherry G. Mathew wrote: Module Name:src Committed By: cherry Date: Thu Dec 8 03:34:48 UTC 2011 Modified Files: src/sys/arch/xen/xen: evtchn.c Log Message: kmem_free() the appropriate size. Thanks cegger@ This is still not enough: In pirq_establish(

Re: CVS commit: src/sys/arch/xen/xen

2011-12-07 Thread Cherry G. Mathew
On 7 December 2011 19:19, Christoph Egger wrote: > Module Name:    src > Committed By:   cegger > Date:           Wed Dec  7 13:49:04 UTC 2011 > > Modified Files: >        src/sys/arch/xen/xen: evtchn.c > > Log Message: > build fix: add back . malloc(9) is still in use. > Hi, sorry about this, I

Re: CVS commit: src/sys/arch/xen/xen

2011-05-15 Thread Jean-Yves Migeon
On 15.05.2011 21:11, Mindaugas Rasiukevicius wrote: > This is not correct. Atomic op might decrease the reference count to > 1 while other thread executes xbdi_put() before xbdi_refcnt is fetched, > thus decreasing it to 0. In such case, both threads would fetch 0. > > The following is a correct

Re: CVS commit: src/sys/arch/xen/xen

2011-05-15 Thread Mindaugas Rasiukevicius
Hello, "Jean-Yves Migeon" wrote: > Module Name: src > Committed By: jym > Date: Sun May 15 07:24:15 UTC 2011 > > Modified Files: > src/sys/arch/xen/xen: xbdback_xenbus.c > > Log Message: > Use atomic_ops(3) for ref counting. > > + atomic_dec_uint(&(xbdip)->xbdi_refcnt);

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Mindaugas Rasiukevicius
Jean-Yves Migeon wrote: > On 18.04.2011 10:35, Mindaugas Rasiukevicius wrote: > > We used to check the return of big size allocations, when kmem(9) could > > fail even with KM_SLEEP due to KVA starvation. However, pretty much all > > kernel does not perform checks for smaller allocations and sinc

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Adam Hamsik
On Apr,Monday 18 2011, at 10:26 AM, Mindaugas Rasiukevicius wrote: > "Cherry G. Mathew" wrote: > On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasiukevicius > wrote: >> Module Name:src >> Committed By: rmind >> Date: Mon Apr 18 03:04:31 UTC 2

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Jean-Yves Migeon
On 18.04.2011 10:35, Mindaugas Rasiukevicius wrote: > We used to check the return of big size allocations, when kmem(9) could > fail even with KM_SLEEP due to KVA starvation. However, pretty much all > kernel does not perform checks for smaller allocations and since the bug > was fixed - we are no

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Mindaugas Rasiukevicius
Jean-Yves Migeon wrote: > On 18.04.2011 05:04, Mindaugas Rasiukevicius wrote: > > Module Name:src > > Committed By: rmind > > Date: Mon Apr 18 03:04:31 UTC 2011 > > > > Modified Files: > > src/sys/arch/xen/xen: balloon.c > > > > Log Message: > > balloon_xenbus_att

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Mindaugas Rasiukevicius
"Cherry G. Mathew" wrote: > >>> On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasiukevicius > >>> wrote: > >>> > Module Name:        src > >>> > Committed By:       rmind > >>> > Date:               Mon Apr 18 03:04:31 UTC 2011 > >>> > > >>> > Modified Files: > >>> >     src/sys/arch/xen/xen

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Jean-Yves Migeon
On 18.04.2011 05:04, Mindaugas Rasiukevicius wrote: > Module Name: src > Committed By: rmind > Date: Mon Apr 18 03:04:31 UTC 2011 > > Modified Files: > src/sys/arch/xen/xen: balloon.c > > Log Message: > balloon_xenbus_attach: use KM_SLEEP for allocation. > > Note: please do not us

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Cherry G. Mathew
On 18 April 2011 09:04, Jean-Yves Migeon wrote: > On 18.04.2011 09:23, Cherry G. Mathew wrote: >> On 18 April 2011 08:00, Cherry G. Mathew wrote: >>> On 18 April 2011 04:21, Mindaugas Rasiukevicius wrote: Masao Uebayashi wrote: > On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasi

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Jean-Yves Migeon
On 18.04.2011 09:23, Cherry G. Mathew wrote: > On 18 April 2011 08:00, Cherry G. Mathew wrote: >> On 18 April 2011 04:21, Mindaugas Rasiukevicius wrote: >>> Masao Uebayashi wrote: On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasiukevicius wrote: > Module Name:src > Co

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Cherry G. Mathew
On 18 April 2011 08:00, Cherry G. Mathew wrote: > On 18 April 2011 04:21, Mindaugas Rasiukevicius wrote: >> Masao Uebayashi wrote: >>> On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasiukevicius wrote: >>> > Module Name:        src >>> > Committed By:       rmind >>> > Date:              

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Cherry G. Mathew
On 18 April 2011 04:21, Mindaugas Rasiukevicius wrote: > Masao Uebayashi wrote: >> On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasiukevicius wrote: >> > Module Name:        src >> > Committed By:       rmind >> > Date:               Mon Apr 18 03:04:31 UTC 2011 >> > >> > Modified Files: >

Re: CVS commit: src/sys/arch/xen/xen

2011-04-17 Thread Mindaugas Rasiukevicius
Masao Uebayashi wrote: > On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasiukevicius wrote: > > Module Name:src > > Committed By: rmind > > Date: Mon Apr 18 03:04:31 UTC 2011 > > > > Modified Files: > > src/sys/arch/xen/xen: balloon.c > > > > Log Message: >

Re: CVS commit: src/sys/arch/xen/xen

2011-04-17 Thread Masao Uebayashi
On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasiukevicius wrote: > Module Name: src > Committed By: rmind > Date: Mon Apr 18 03:04:31 UTC 2011 > > Modified Files: > src/sys/arch/xen/xen: balloon.c > > Log Message: > balloon_xenbus_attach: use KM_SLEEP for allocation. > > N

Re: CVS commit: src/sys/arch/xen/xen

2011-04-06 Thread Jean-Yves Migeon
On 06.04.2011 20:01, Manuel Bouyer wrote: > You could also use > xvifxiy (e.g. xvif5i2, where i stands for 'index'). > or any other letter ... Huh hmm indeed... I wonder why I did not think about this approach before... -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/xen/xen

2011-04-06 Thread Manuel Bouyer
On Wed, Apr 06, 2011 at 05:41:37PM +0100, Jean-Yves Migeon wrote: > Yep, but there are other places where this will get tricky. For > example, rc.conf(5) is parsed as a classic shell script [1]. Given > that someone has two possibilities to configure interfaces: > > /etc/ifconfig.xxx > or > ifconf

Re: CVS commit: src/sys/arch/xen/xen

2011-04-06 Thread Jean-Yves Migeon
On Wed, 6 Apr 2011 16:29:22 +, Taylor R Campbell wrote: Date: Mon, 04 Apr 2011 23:46:19 +0200 From: Jean-Yves Migeon The newer scripts for Xen read the interface value from the vifname entry in Xenstore, so changing the name is not that critical. But this should be stabilized

Re: CVS commit: src/sys/arch/xen/xen

2011-04-06 Thread Taylor R Campbell
Date: Mon, 04 Apr 2011 23:46:19 +0200 From: Jean-Yves Migeon The newer scripts for Xen read the interface value from the vifname entry in Xenstore, so changing the name is not that critical. But this should be stabilized sooner than later. Personally, I find '_' rather ugly in a

Re: CVS commit: src/sys/arch/xen/xen

2011-04-04 Thread Jean-Yves Migeon
On 04.04.2011 23:21, Taylor R Campbell wrote: >Date: Sun, 3 Apr 2011 23:21:39 + >From: "Jean-Yves Migeon" > >Now that pkgsrc-2011Q1 has arrived, and before -6 chimes in, change >ifxname for xvif(4) from xvif%d.%d to xvif%d-%d. This is needed >to avoid sysctl(9) EINVAL erro

Re: CVS commit: src/sys/arch/xen/xen

2011-04-04 Thread Taylor R Campbell
Date: Sun, 3 Apr 2011 23:21:39 + From: "Jean-Yves Migeon" Now that pkgsrc-2011Q1 has arrived, and before -6 chimes in, change ifxname for xvif(4) from xvif%d.%d to xvif%d-%d. This is needed to avoid sysctl(9) EINVAL errors when creating interface nodes. This change came awfull

Re: CVS commit: src/sys/arch/xen/xen

2011-03-29 Thread David Young
On Wed, Mar 30, 2011 at 12:17:05AM +, Jean-Yves Migeon wrote: > Module Name: src > Committed By: jym > Date: Wed Mar 30 00:17:04 UTC 2011 > > Modified Files: > src/sys/arch/xen/xen: if_xennet_xenbus.c > > Log Message: > printf("xennet: ...") => aprint_error_ifnet() It's silly,