Skip to content

Commit db2c886

Browse files
committed
devlog, logging for raspberry pi
1 parent 59d9d74 commit db2c886

File tree

3 files changed

+27
-4
lines changed

3 files changed

+27
-4
lines changed

DEVLOG.md

+26
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,13 @@
11
# devlog
22

3+
## Mon June 17, 2024
4+
5+
### nlnet updates
6+
7+
The [first nlnet milestone PR](https://github.com/gfx-rs/wgpu/pull/5775) is really close to
8+
merging. I'm already working on the [follow up PR](https://github.com/gfx-rs/wgpu/pull/5824)
9+
that adds the rest of the operators, which is milestone #2.
10+
311
## Sun June 16, 2024
412

513
### nlnet updates
@@ -12,6 +20,23 @@ What this means is that I'm actually very close to hitting the first nlnet miles
1220
With that, I've started work on the second milestone while my first set of PRs are in review,
1321
as it takes a good while to roundtrip w/ feedback.
1422

23+
#### Spiralling out
24+
25+
Previously we had talked about upgrading expressions, and how there would be a "spiralling out"
26+
or "cascade" of upgrades needed. I think we've mostly side-stepped that requirement by first
27+
realizing that atomics can really only be held in global variables since they only appear in
28+
`workgroup` and `storage` address spaces.
29+
30+
So these ops will always be accessed through a pointer to a global variable, and we can modify
31+
that global variable's type in place and then not worry about having to upgrade the expressions
32+
that contain that global variable. It's a nice simplification.
33+
34+
> The reason I think you won't need to update any expressions is that Naga IR Load expressions
35+
> and Store statements both can operate on Atomics, so everything accessing the globals whose
36+
> types you're whacking, whether Loads, Stores, or Atomics, should still be okay.
37+
>
38+
> -- Jim Blandy
39+
1540
### raspberry pi updates
1641

1742
Still getting OOM errors and I'm not sure why. There are a few changes I need to make to figure
@@ -36,6 +61,7 @@ it out:
3661
![diff](https://github.com/schell/renderling/assets/24942/4770e596-34ee-479f-ba2d-2325d8dfa282)
3762

3863
`seen.png` is nothing but wacky garbage!
64+
3965
## Fri June 14, 2024
4066

4167
### nlnet updates

crates/renderling/src/context.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ async fn device(
145145
let required_features = wanted_features.intersection(supported_features);
146146
let unsupported_features = wanted_features.difference(supported_features);
147147
if !unsupported_features.is_empty() {
148-
log::error!("required but unsupported features: {unsupported_features:#?}");
148+
log::error!("requested but unsupported features: {unsupported_features:#?}");
149149
}
150150
let limits = adapter.limits();
151151
log::info!("adapter limits: {limits:#?}");

crates/renderling/src/lib.rs

-3
Original file line numberDiff line numberDiff line change
@@ -205,9 +205,6 @@ mod test {
205205
let _ = env_logger::builder()
206206
.is_test(true)
207207
.filter_level(log::LevelFilter::Warn)
208-
.filter_module("renderling::context", log::LevelFilter::Info)
209-
.filter_module("renderling::atlas", log::LevelFilter::Trace)
210-
.filter_module("renderling", log::LevelFilter::Debug)
211208
.try_init();
212209
}
213210

0 commit comments

Comments
 (0)