mlarkin discovered that if the treasure lands on the snake, you won't see it.
it's still there, but invisible. this fixes the display to draw the correct
character.
Index: snake.c
===
RCS file: /cvs/src/games/snake/snake.c,v
retrievi
On 2019-01-19 16:09:05, Ted Unangst wrote:
> Bryan Linton wrote:
> > Hello tech@,
> >
> > I'd appreciate it if someone could review both this patch and my
> > analysis.
>
> There is no reason for etime to be a global. We can make it a local with the
> correct value in the two functions that use
On Sat, Jan 19, 2019 at 05:37:34PM +0100, Stefan Fritsch wrote:
> Add a sc_driver_features field that is automatically used by
> virtio_negotiate_features() and during reinit.
>
> Make virtio_negotiate_features() return an error code. Virtio 1.0 has a
> special status bit for feature negotiation
On Sat, Jan 19, 2019 at 05:37:35PM +0100, Stefan Fritsch wrote:
> virtio 1.0 for virtio_mmio it not yet implemented, but 0.9 devices
> continue to work.
> ---
This one also should have someone who is well versed in PCI take a look.
-ml
> share/man/man4/virtio.4 | 11 +-
> sys/dev/pci/virti
On Sat, Jan 19, 2019 at 05:37:32PM +0100, Stefan Fritsch wrote:
> In virtio_pci 1.0, different parts of the register set may be located in
> different BARs. Use subregions to make the access independent of the
> virtio version.
> ---
This one will need someone more well versed in PCI BARs/etc to c
On Sat, Jan 19, 2019 at 05:37:33PM +0100, Stefan Fritsch wrote:
> ---
> sys/dev/pci/virtio_pci.c | 38 --
> 1 file changed, 24 insertions(+), 14 deletions(-)
>
ok mlarkin
> diff --git a/sys/dev/pci/virtio_pci.c b/sys/dev/pci/virtio_pci.c
> index d69db1968d0..
On Sat, Jan 19, 2019 at 05:37:31PM +0100, Stefan Fritsch wrote:
> ---
> sys/dev/pv/if_vio.c| 84 +++---
> sys/dev/pv/vioblk.c| 3 ++
> sys/dev/pv/vioblkreg.h | 21 ++-
> sys/dev/pv/virtio.c| 1 +
> 4 files changed, 62 insertions(+), 47 dele
On Fri, Jan 18, 2019 at 07:34:35PM +0100, Anton Lindqvist wrote:
> You're right, updated diff.
OK bluhm@
> Index: share/man/man4/kcov.4
> ===
> RCS file: /cvs/src/share/man/man4/kcov.4,v
> retrieving revision 1.6
> diff -u -p -r1.6 k
On Tue, Jan 15, 2019 at 10:01:42PM +0100, Stefan Sperling wrote:
> This diff makes 'ifconfig bwfm0' display whether 11n is active or not,
> what the current Tx rate is, and show up-to-date RSSI measurements.
> I'm leaving display of 11ac Tx rates for future work because that will
> require a change
Bryan Linton wrote:
> Hello tech@,
>
> I'd appreciate it if someone could review both this patch and my
> analysis.
There is no reason for etime to be a global. We can make it a local with the
correct value in the two functions that use it, avoiding future problems as
well.
Index: vmstat.c
On Sat, Jan 19, 2019 at 05:37:30PM +0100, Stefan Fritsch wrote:
> virtio 1.0 supports an arbitrary number of feature bits. However, so far
> no more than 64 are used (compared to 32 in virtio 0.9). Adjust data
> types to support 64 feature bits.
>
> Later, we may want to use bitmaps and setbit(),
On Sat, Jan 19, 2019 at 05:37:29PM +0100, Stefan Fritsch wrote:
> Make it take an address instead of a PFN.
> Pass the virtqueue pointer. In virtio 1.0, more information has to be
> configured in the device. Also call virtio_setup_queue() after the
> information has been filled in.
> ---
> sys/dev
Hi,
The i_lockf field in `struct iso_node' looks unused since the initial
import. Also, vop_advlock is mapped to eopnotsupp() in cd9660_vops.
Comments? OK?
Index: isofs/cd9660/cd9660_node.h
===
RCS file: /cvs/src/sys/isofs/cd9660/cd9
virtio 1.0 supports an arbitrary number of feature bits. However, so far
no more than 64 are used (compared to 32 in virtio 0.9). Adjust data
types to support 64 feature bits.
Later, we may want to use bitmaps and setbit(), ... to support even more
feature bits.
---
sys/dev/fdt/virtio_mmio.c | 8
Add a sc_driver_features field that is automatically used by
virtio_negotiate_features() and during reinit.
Make virtio_negotiate_features() return an error code. Virtio 1.0 has a
special status bit for feature negotiation that means that negotiation
can fail. Make virtio_negotiate_features() ret
virtio 1.0 for virtio_mmio it not yet implemented, but 0.9 devices
continue to work.
---
share/man/man4/virtio.4 | 11 +-
sys/dev/pci/virtio_pci.c| 518 +++-
sys/dev/pci/virtio_pcireg.h | 57
sys/dev/pv/if_vio.c | 9 +-
sys/dev/pv/virtiovar.
In virtio_pci 1.0, different parts of the register set may be located in
different BARs. Use subregions to make the access independent of the
virtio version.
---
sys/dev/pci/virtio_pci.c | 114 +++
1 file changed, 80 insertions(+), 34 deletions(-)
diff --git a/
---
sys/dev/pci/virtio_pci.c | 38 --
1 file changed, 24 insertions(+), 14 deletions(-)
diff --git a/sys/dev/pci/virtio_pci.c b/sys/dev/pci/virtio_pci.c
index d69db1968d0..7be93684a68 100644
--- a/sys/dev/pci/virtio_pci.c
+++ b/sys/dev/pci/virtio_pci.c
@@ -69,6
Make it take an address instead of a PFN.
Pass the virtqueue pointer. In virtio 1.0, more information has to be
configured in the device. Also call virtio_setup_queue() after the
information has been filled in.
---
sys/dev/fdt/virtio_mmio.c | 11 +++
sys/dev/pci/virtio_pci.c | 11 ++--
Hi,
here comes the split up diff for virtio 1.0 support. Compared to the previous
diff, I have omitted some changes in how the feature bits and negotiation
works. This also means that the feature bit defines in the headers stay the
same and vmd is not affected. I have tested that virtio_mmio on a
---
sys/dev/pv/if_vio.c| 84 +++---
sys/dev/pv/vioblk.c| 3 ++
sys/dev/pv/vioblkreg.h | 21 ++-
sys/dev/pv/virtio.c| 1 +
4 files changed, 62 insertions(+), 47 deletions(-)
diff --git a/sys/dev/pv/if_vio.c b/sys/dev/pv/if_vio.c
index a4cd8
On 2019-01-19 15:00:20, Otto Moerbeek wrote:
> On Sat, Jan 19, 2019 at 09:30:12PM +0900, Bryan Linton wrote:
>
> > Hello tech@,
> >
> > I'd appreciate it if someone could review both this patch and my
> > analysis.
> >
> > [...]
> >
> > ADDENDUM
> >
> >
> > I'm curious why iostat.c u
Hi Theo and Theo,
Theo de Raadt wrote on Fri, Jan 18, 2019 at 08:06:32PM -0700:
> Ted Unangst wrote:
>> Theo Buehler wrote:
>>> According to our documentation and all the standards I checked,
>>> snprintf() returns a negative value on error, not necessarily -1.
>>> This confused me quite a bit
On Saturday, 12 January 2019 01:49:09 CET Jonathan Gray wrote:
> On Fri, Jan 11, 2019 at 03:21:11PM -0800, William Ahern wrote:
> > On Fri, Jan 11, 2019 at 10:43:25AM +0100, Stefan Fritsch wrote:
> >
> >
> > > /* only used for sizeof, not actually allocated */
> > > extern struct virtio_pci_commo
Hi,
Comments below,
> On 19/01/2019, at 2:32 PM, Klemens Nanni wrote:
>
> A few assorted nits for consistency and proper format, no object change.
>
> OK?
>
> Index: pfctl.c
> ===
> RCS file: /cvs/src/sbin/pfctl/pfctl.c,v
> retr
On Sat, Jan 19, 2019 at 09:30:12PM +0900, Bryan Linton wrote:
> Hello tech@,
>
> I'd appreciate it if someone could review both this patch and my
> analysis.
>
>
> PROBLEM
> ---
> When running systat, the vmstat page shows inaccurate numbers for
> disk i/o speed (likely only on multiprocess
Hello tech@,
I'd appreciate it if someone could review both this patch and my
analysis.
PROBLEM
---
When running systat, the vmstat page shows inaccurate numbers for
disk i/o speed (likely only on multiprocessor systems) when
compared to both the iostat page, as well as the statistics
report
On Sat, Jan 19, 2019 at 05:14:56PM +1300, Richard Procter wrote:
> > +#define PF_OPTIMIZE_NONE 0x
>
> these PF_OPTIMIZE_* are bit-field definitions,
> see e.g. pfctl_optimize.c:299.
While I'm aware of this,
> But PF_OPTIMIZE_NONE is not, as pf->optimize & PF_OPTIMIZE_NONE
> is never true,
28 matches
Mail list logo
