- Apr 17, 2019
-
-
Kiet Tran authored
WLAN-1050 #done Test: fx full-build Change-Id: I083db44fadaad0d8e77179103b4ab47ae65d9e70
-
Taylor Cramer authored
Change-Id: Id9c2406b7fb715b5bbb91ab999be687692569e29
-
Scott Graham authored
This is intended as a transitional step on replacing abigen with banjo. The goal of this backend is for it to be a relatively short-lived as a migration path: 1. Create abigen backend for banjo 2. Replace hand-written abigen with output of equivalent .banjo file as a Zircon build step 3. Create additional backend(s) that generate the current outputs of abigen, but as there are currently 13, proceed one or a handful at a time 4. Once the build is consuming files generated directly from banjo, remove the abigen backend (and the abigen tool). ZX-3882 #comment [abigen][banjo] Add beginnings of abigen backend for banjo Test: new tests added in tools/banjo/banjo/test/abigen Change-Id: I96a332e96c6d6332cfe74b62fbffd852f8a9e9b7
-
- Apr 16, 2019
-
-
Dylan Swiggett authored
CLion has an incomplete index w/o these additions. Test: Verified index is now complete. Change-Id: Ifd3638e9197af515c3396a4373034c0eb9bea6af
-
Forrest Reiling authored
Change-Id: I631566d0d9c5ecc96f7d45238fe5c094b4ef2842
-
Roland McGrath authored
The old gen/build_arguments.md files from the pre-fuchsia.git repos have not been updated in a long time. The docs-roller bot now updates //docs/gen/build_arguments.md instead. But since that file didn't exist, bot runs have not been updating *any* file for the Fuchsia GN build arguments documentation, only the new Zircon file. Remove the defunct peridot/ file and move the garnet/ file to //docs where the next bot run should update it properly. Change-Id: I6430b4af4ca796fa12ae366ce899c1c3afc0856b
-
P.Y. Laligand authored
Bug: DX-1201 Bug: DX-1269 Change-Id: I19936ae5489f485c745fff86c464c879c897cfb7
-
Travis Geiselbrecht authored
Change-Id: I8260bb84c136fa4f088c99934cacad575b8c4df3
-
Dylan Swiggett authored
This fixes the huge volume of bad readings seen due to unimplemented wlan_rx_info for data frames. Subsequent work should resolve these instances of missing info. WLAN-1018 Test=ran existing unit tests and verified histogram fixed in `wlan stats` Change-Id: I3395b65c4060664f20a3bdff0d08b03fbc8458cd
-
Mark Dittmer authored
- actions: convertTrace() accepts multiple files - Pass additional zedmon file when -zedmon specified - Temporary: Local copy of minimal tracing event structs required for zedmon trace - Hardcoded zedmon pseudoprocess PID: "zedm" ASCII - Channel-based zedmon interface - On run(): Stream CSV into structs, then JSON bytes - Return channels for data and errors - On stop(): Stop streaming and cleanup Change-Id: Ibf3e9693714d310f62c99aec7f10474a571fc829
-
Taylor Cramer authored
Change-Id: I7d190d7293ebb77184fd36bfed934c2c1cb27eda
-
Taylor Cramer authored
Change-Id: I512517aa1d0f76a07a6c54dc504db8f4da1366e2
-
Suraj Malhotra authored
As part of migrating towards a unified A/B/R boot image configuration for all supported boards the following changes were made: * A chromebook board file was added. ** VBOOT images are only built for the new target. ** VBOOT images are treated similarly to how signed images on other boards are treated. * The EFI image was deduplicated into bootloader. * The KERNC image was deduplicated into ZIRCON-A. * Support for a VB_META_R partition was added. * zedboot version was updated (due to breaking changes) ZX-3861 Tested: Paved pixelbook, NUC, astro, and vim2 Change-Id: I2344048ef69dd7b5e53004c707a261f5e0e0bb93
-
Erick Tryzelaar authored
This removes some repetition in the items `fuchsia_uri::FuchsiaBootUri` and `fuchsia_uri::FuchsiaPkgUri`. Change-Id: I400a15b98a8fc6e476fa15c3d920c1d5a13780ab
-
Adam Gousetis authored
Change-Id: Icd85ede1a0f4f9492e91907039ba012a908db456
-
Robert Lord authored
Previously, since a waker was reused between loops in recv_all, a deadlock was possible: TASK 1 -> poll_recv_event <- pending, will wake task 1 on next chan.recv_from msg TASK 2 -> send MyRequest, poll_recv_msg_response <- pending, will wake task 2 on next chan.recv_from msg **event comes in, chan.recv_from's waker is called** <- wake task 2 TASK 2 -> poll_recv_msg_response <- pending, will wake task 2 on next chan.recv_from msg <- but hey also event is ready so wake task 1 TASK 1 -> poll_recv_event <- ready, will wake task 1 on next chan.recv_from msg **task 1 calls await!(Timer(1 year))** **MyRequest reply comes in, chan.recv_from's waker is called** <- wake task 1 ^ this will not poll or get recv_all called, since task 1 is waiting on a timer, not a FIDL event, so TASK 2 will never be woken up with its now-ready response. Eventually, after one year, task 1 will stop awaiting and either drop the EventReceiver or await!() on it again, either of which will call recv_all and wake task 2. This may be a contrived example, but I ran into it when Task 1 requested a (futures-aware) lock that task 2 was holding. Task 1 was blocked waiting on the lock, and task 2 never got woken because of the logic above. This CL makes several changes: - recv_all now uses the same logic as wake_any to pick a Waker, instead of using the initially-used waker, which may not be polling if the Future is Ready after recv_all. In the example above, the last poll_recv_event would cause the still-waiting task 2 to be registered as the recv_from msg waker, instead of the no-longer-waiting task 1. - poll_recv_msg_response and poll_recv_event now update their waker *before* calling recv_all, so that recv_all can correctly select their waker to be the next chan.recv_from waker if they'll still be Pending after recv_all is called. - When the EventListener is woken, it is now set to EventListener::WillPoll to match the Received behavior in responses. This means the comment in wake_any() is not necessary, since already-called events that are now awaiting on something else will not be selected in get_pending_worker(). EventListener::New was also renamed to EventListener::WillPoll to clarify this new use of the state. This also means the previous comment in wake_any is no longer relevant, since an event receiver will only be the waker if got Pending on its last poll. Change-Id: Id3e23fd4575d220117207e43b6a5ce6b1e419f33
-
Taylor Cramer authored
Change-Id: Ic129292339bda65c5a59e98a72cca5e257e40953
-
Ruchira Ravoori authored
This change adds skeleton for 2 protocol ops. One for requesting the hardware voltage constraints of a regulator, the other for requesting for a particular voltage. Test: fx full-build. No functionality added here. Will do more tets when functionality is added to these protocols. Change-Id: I499ea4790eb86cb17ab216e9c64446e9b5df4ca3
-
Dale Sather authored
This CL fixes a null dereference in mediaplayer that occurs when the player cannot access a URL. TEST: fx shell present_view \ fuchsia-pkg://fuchsia.com/mediaplayer_test_util#meta/mediaplayer_test_util.cmx \ http://ia800201.us.archive.org/12/items/BigBuckBunny_328/BigBuckBunny.ogv Change-Id: Ib95a0f4c0dff81ec8ef05b6600d0abef03679b9c
-
docs-roller authored
Test: CQ Change-Id: I992bebbb45c0b5f5316b32bd6eebe5762eb33c1c
-
Adam Barth authored
Change-Id: Ide75beb0e1f6b35137a5103019ef0219dfbb27cc
-
Dale Sather authored
This CL addresses some misunderstandings concerning constraints on various DNS/mDNS-related string values (chiefly around max length) and makes string checking more thorough. Also added unit tests. TEST: fx run-test mdns_tests Change-Id: I4ca0bbbab463378fb11381bbb01fb59b9f4779b6
-
Adam MacBeth authored
Change-Id: Id63faf19219bdec97bc2c63e81e24b7f5e10d0cd
-
Nick Van der Auwermeulen authored
Change-Id: I2ea40a4ea46dedc08f429707ca79d1611de5f89f
-
Marco Vanotti authored
This reverts commit 6be4d762. Reason for revert: Looks like the build errors were unrelated to this commit. Original change's description: > Revert "[thinfs] fix nil pointer dereference." > > This reverts commit 36b9d349. > > Reason for revert: I received a LUCI email saying that this breaks asan builds. > > Original change's description: > > [thinfs] fix nil pointer dereference. > > > > This CL fixes a nil pointer dereference that happens in the > > thinfs/zircon/rpc package. > > > > The issue is that the `directoryWrapper is not correctly initialized, > > not setting the cookies map. Upon calls to GetToken(), the dictionary is > > dereferenced and that causes a panic. > > > > The commit just makes sure that the cookies map is initialized > > correctly, and adds a test for it. > > > > ZX-3928: Done > > > > TEST=/pkfs/packages/go_thinfs_tests/0/test/go_thinfs_test > > > > Change-Id: I4686714f547f8a759d11dcc2e9414ce60e37707d > > TBR=kulakowski@google.com,smklein@google.com,mvanotti@google.com > > Change-Id: I8bd13bf33f3ee744afed06d4e5d43e74b4349041 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true TBR=kulakowski@google.com,smklein@google.com,mvanotti@google.com Change-Id: I96f2afca116fac4a8f83dfbc15113234907cac17 No-Presubmit: true No-Tree-Checks: true No-Try: true
-
Ilya Bobyr authored
ZX-3581 #done Change-Id: Id55af08c814ec2921e31dfd9f707281a0d6b830f
-
Bert Muthalaly authored
I additionally grepped the tree for instances of NetErr that weren't StatusOk that left the Message field empty; this is the only one. Change-Id: I607748dd6a91083e0f74decc16743be3879b8633
-
P.Y. Laligand authored
Bug: DX-1045 #done Change-Id: I277dd292b4f35a5cf3953928fab0d6bd72770803
-
Mitch Rudominer authored
TEST=Manually ran and observed that warning in console went away. Change-Id: I548e20d7df618d4cf031bff12237233a7f8bd493
-
Taylor Cramer authored
Change-Id: Idf4fbd28106e45c982fee6f024e27f6d30469052
-
Claudio Cherubino authored
This was approved in OSRB-168. Change-Id: Ib3f0b14ebb2c39e37e0d968ab810d6b4a55db002
-
Aaron Green authored
CL 267663 was a bit too aggressive in excluding //test_data. Test: fd && mkdir -p test_data zircon/test_data touch test_data/foo zircon/test_data/bar git status # Shows zircon/test_data/bar but nor test_data/foo Change-Id: Ie6a2512eb368e9a9ffe21796e53ef4520977670b
-
Doug Evans authored
Tested: Apply repro-inducer from bug report, note that test no longer fails due to timing out. PT-133 #done Change-Id: I78e1a34f77fe02075512bca75e89222cfae3293f
-
Zach Bush authored
Change-Id: I8ee983db6b3ca88113264c86af8c6ec4ceeb0d39
-
Kevin Wells authored
The arguments of switches were ending up in targets arrays instead of the switches arrays. Also remove some unused local variables. Test: fx build; fx build -t targets Change-Id: I07582facadf19b9300cc96f27f12b857dfb808d7
-
Dale Sather authored
This CL adds fuchsia.net.mdns, which will replace fuchsia.mdns. This library definition has been reviewed and approved. TEST: no behavior change Change-Id: I9df801befe3466ccda59f308d4739eef6cb26e0f
-
Taylor Cramer authored
Change-Id: If463cd17fff48aaa53ca027c594a1c8107e2154e
-
Yifei Teng authored
Change-Id: Idddba28f06a49623f50c1a29bca22c48912a32e2
-
Adam Barth authored
Now that oldset is no longer supported, we can migrate these product configurations to use the new GN label syntax. Change-Id: Ieacdd19f38a655f8ff3c6c59a8ff2be1f12f5034
-
Marty Faltesek authored
ZX_WAIT_ASYNC_REPEATING is deprecated so convert code to use ZX_WAIT_ASYNC_ONCE instead. Bug: ZX-3090 Test: /boot/test/sys/debugger-test and /boot/test/sys/exception-test Change-Id: I134ce421dccaf85b23a657f92855055552a3d32b
-