Commit Graph

648 Commits

Author SHA1 Message Date
Dzmitry Malyshau
b6e7b1f204 Add missing WGPUGlobal to wgpu-remote bindings 2019-11-21 15:25:02 -05:00
bors[bot]
73b33ea76e Merge #382
382: Cleanup exports r=grovesNL a=kvark

This is a follow-up to #381 that cleans up the exports, such that `wgpu-core` doesn't deal with any raw C things.

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-11-20 01:02:41 +00:00
Dzmitry Malyshau
3908d2b5a1 Cleanup exports 2019-11-19 10:59:24 -05:00
bors[bot]
407cf69a8e Merge #378
378: Support for merged depth-stencil states r=grovesNL a=kvark

Fixes #376 
@Yatekii would you be able to test it and confirm the fix?

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-11-19 13:38:43 +00:00
Dzmitry Malyshau
98d28caffc Refactor internal texture tracking format 2019-11-18 22:07:15 -05:00
Dzmitry Malyshau
4e57c42c71 Initialize trackers with full selector - part 2 2019-11-18 09:20:16 -05:00
Dzmitry Malyshau
0569bc6956 Initialize trackers with full selector 2019-11-18 00:21:13 -05:00
bors[bot]
aa14f7f76d Merge #381
381: Separate Native from the Core r=grovesNL a=kvark

Fixes #379 (haven't figured out how to build remote static lib to test the example, but that's not as important at this stage, since we have Gecko).
Fixes #168 
Related to #2 and #8

This is a major refactor that moves out the core Rust API into a separate crate. It also changes the export semantics of wgpu-core in a way that the modules are exposed, and all the functionality is now represented as methods of `Global`. The "local" feature is transformed into a type system as a generic parameter `<F>` on the hubs.

Pros:
  - "local" feature is gone
  - we can change the core API more often, this doesn't necessarily require the `native` breaking version bump, and it would be very handy when adding Gecko-related changes
  - Gecko needs less stuff to compile (remote + core)
  - no collision of library names (native with or without the local feature)
  - less problem with crate types
  - cleaner semantics: each crate is either C or Rust, but not both

Cons:
  - more generics, I wouldn't be surprised this regresses the build times visibly

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-11-16 15:07:30 +00:00
Dzmitry Malyshau
6c632d1366 Separate native from the core 2019-11-16 00:36:23 -05:00
bors[bot]
f221a6592d Merge #380
380: More licenses for Gecko r=kvark a=kvark

These are all the last things we need to match Gecko, with an exception of #379 

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-11-15 16:53:13 +00:00
Dzmitry Malyshau
b9aed69bed More licenses for Gecko 2019-11-15 11:51:46 -05:00
bors[bot]
ae64e3aa0c Merge #375
375: Use callback for `request_adapter` r=kvark a=grovesNL

For now this is mostly a signature change which allows us to start using a callback for `request_adapter`. Once we have an event loop, the callback could be scheduled to make this asynchronous. At that point the examples and server would have to be updated, because they currently rely on the callback running immediately (inline).

The reason for this change is that `request_adapter` is supposed to return a `Promise` in JS, so we can represent it with callbacks at the C API, and `Future` in Rust (though we'll probably keep using callbacks initially).

I also changed `request_adapter` to provide an adapter with `AdapterId::ERROR` when no adapters are available, so we can add basic error handling to wgpu-rs (https://github.com/gfx-rs/wgpu-rs/issues/117).

Co-authored-by: Joshua Groves <josh@joshgroves.com>
2019-11-08 03:41:12 +00:00
Joshua Groves
7f25b8c318 Move adapter selection into separate function 2019-11-07 21:36:48 -06:00
Joshua Groves
2b1ce5d6d9 Use callback for request_adapter 2019-11-07 09:55:27 -06:00
bors[bot]
951641dcc5 Merge #373
373: Release preparations for 0.4 r=kvark a=kvark



Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
v0.4.0
2019-11-04 00:33:07 +00:00
Dzmitry Malyshau
9ffcbe5f9d Release preparations 2019-11-03 18:52:57 -05:00
bors[bot]
c33b1e81fd Merge #372
372: Followup for #356 - fix incorrect check r=kvark a=parasyte

@kvark This fixes the fix created in #356 

The original patch is checking the _requested descriptor limits_, but the backend limits are what needs to be checked for this edge case.

Co-authored-by: Jay Oster <jay@kodewerx.org>
2019-11-02 23:35:55 +00:00
Jay Oster
5a7b16076c Followup for #356 - fix incorrect check 2019-11-02 15:25:32 -07:00
bors[bot]
12d4108ca8 Merge #367
367: Support rendering to the swapchain from multiple command buffers r=jazzay a=kvark

Fixes #366

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-11-02 12:12:50 +00:00
bors[bot]
14c3583ba2 Merge #371
371: Don't depend on gfx-auxil r=straightforward a=kvark

Problem with gfx-auxil is that version 0.1 drags spirv_cross dependency (not needed on Vulkan), and the published crates haven't been switched to gfx-auxil-0.2

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-11-02 04:26:04 +00:00
Dzmitry Malyshau
cb692620ca Don't depend on gfx-auxil 2019-11-02 00:24:36 -04:00
bors[bot]
505096d027 Merge #369
369: return Result instead of panic r=kvark a=antonok-edm

Fixes https://github.com/gfx-rs/wgpu-rs/issues/106

Pretty simple implementation, let me know if this isn't sufficient

Co-authored-by: Anton Lazarev <antonok35@gmail.com>
2019-11-01 18:20:40 +00:00
Anton Lazarev
027a61a48d return Result instead of panic 2019-11-01 11:11:43 -07:00
Dzmitry Malyshau
52b9d061f4 Support rendering to the swapchain from multiple command buffers 2019-10-31 10:14:48 -04:00
bors[bot]
b8d26c9186 Merge #363
363: Extended remote example r=kvark a=kvark

The remote example is now successfully able to request an adapter and to clean up the IDs.
In the near future, we need to change the example so that the server is run on a different thread...

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-10-30 03:24:05 +00:00
Dzmitry Malyshau
384de9ca52 Properly dispose of DescriptorAllocator 2019-10-29 15:45:48 -04:00
Dzmitry Malyshau
ff1adb34d9 Fix destroy_instance warning 2019-10-29 15:30:30 -04:00
Dzmitry Malyshau
4635f1eb07 Extended remote example 2019-10-29 13:31:40 -04:00
bors[bot]
ee9fd49c1d Merge #361
361: Don't panic on the lack of adapters r=trivial a=kvark



Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-10-29 02:41:40 +00:00
Dzmitry Malyshau
41132a3762 Don't panic on the lack of adapters 2019-10-28 22:40:57 -04:00
bors[bot]
6a6517832a Merge #360
360: Update to latest rendy with gfx-0.4 r=kvark a=kvark



Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-10-28 16:00:13 +00:00
Dzmitry Malyshau
271c709645 Update to latest rendy with gfx-0.4 2019-10-28 11:59:18 -04:00
bors[bot]
b71fc33c72 Merge #359
359: Make examples work again r=kvark a=GabrielMajeri

This PR fixes the C example code to not crash and actually run.

I've also added a few assertions to ensure a warning is emitted next time somebody forgots to increase `max_bind_groups` to something non-zero on device creation.

To help with debugging the examples, I've configured CMake to include debug info in the builds. Some new Makefile targets for the examples have been added to automate running them.

Co-authored-by: Gabriel Majeri <gabriel.majeri6@gmail.com>
2019-10-26 21:08:16 +00:00
Gabriel Majeri
eb198a9c3e Fix C examples
Get the examples running again.

The compute example has been simplified to use only one buffer.

A new check has been added to ensure the `max_bind_groups`
device limit is properly set.
2019-10-26 22:13:53 +03:00
bors[bot]
94f43c1167 Merge #358
358: Remove all the IPC r=grovesNL a=kvark

Closes #146 
Closes #22

We have decided to use Gecko IPC for Firefox. `wgpu-remote` will therefore provide all the Rust glue that Gecko needs for client and server:
  - initialization/termination of client/server
  - ID management for the client
  - pass encoding blobs

In Servo, we'd need to enable `serde` feature of `wgpu-native` and potentially roll out a different remoting crate that would establish a protocol based on `ipc-channel`, as we wanted originally.

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-10-26 13:54:52 +00:00
Dzmitry Malyshau
538c23b039 Implement shutdown for the remote server 2019-10-25 22:56:52 -04:00
Dzmitry Malyshau
e37149b3a8 Remove all the IPC 2019-10-25 12:06:12 -04:00
bors[bot]
c8d0da65ad Merge #357
357: Explicit Global hubs r=grovesNL a=kvark

Fixes #354

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-10-25 02:08:56 +00:00
Dzmitry Malyshau
50997544c6 Formatting pass 2019-10-24 14:53:45 -04:00
Dzmitry Malyshau
41c433ed27 Refactor log usage 2019-10-24 14:53:07 -04:00
Dzmitry Malyshau
4851284150 Switch from remote feature to local 2019-10-24 14:52:44 -04:00
Dzmitry Malyshau
f67894c310 Explicit Global hubs 2019-10-24 14:52:19 -04:00
bors[bot]
ae5a78f828 Merge #356
356: Allow max_bind_groups to be missing, for now r=kvark a=kvark

Fixes  #355

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-10-24 14:51:05 +00:00
Dzmitry Malyshau
f3a0dd9b63 Allow max_bind_groups to be missing, for now 2019-10-24 10:50:08 -04:00
bors[bot]
a04a4b1cff Merge #353
353: Always return an Id by request_adapter r=kvark a=kvark

Requesting adapters is a bit special in a sense that it's exactly the place where the backend selection happens. It accepts a list of Ids, and we need to always return one, so that the remote client knows which Id was actually used and clean up the others.

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-10-23 22:49:34 +00:00
Dzmitry Malyshau
c4906ee36f Always return an Id by request_adapter 2019-10-23 14:31:36 -04:00
bors[bot]
50acc2909c Merge #352
352: Fix compute pass resource binding r=kvark a=kvark

Follow-up to #348 
cc @grenlight @terrence2 

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
2019-10-22 14:08:54 +00:00
Dzmitry Malyshau
cf687604b9 Fix compute pass resource binding 2019-10-22 09:09:53 -04:00
Terrence Cole
78fbbba5e9 Support Limits::max_bind_groups (#348)
* Pass max_bind_count down into Binder.
* Use the existing hub.
* Remove accidental newline.
2019-10-14 21:57:05 -04:00
bors[bot]
5ba923c79f Merge #344
344: Update deps r=kvark a=rukai

We should wait for https://github.com/rust-windowing/winit/pull/1191 to find its way into a winit release before merging this PR.

Co-authored-by: Rukai <rubickent@gmail.com>
2019-10-06 02:05:41 +00:00