Skip to content
Snippets Groups Projects
  1. May 12, 2019
    • docs-roller's avatar
      [gndoc] Update GN build arguments documentation · 0d4917c4
      docs-roller authored
      Test: CQ
      
      Change-Id: I6ae5d995f2e9c824b82aff9c9c985d604e7a6cbf
      0d4917c4
    • Rich Kadel's avatar
      [fidl-lint] Sort test functions · cf4e1a01
      Rich Kadel authored
      Test functions are now ordered alphabetically, just like the calling
      sequence at the bottom of lint_finding_tests.cpp.
      
      Test: fidl-compiler-test
      Change-Id: Ic0c60a83ad530f8cf3caf3d50c83c1649e4f4861
      cf4e1a01
    • Rich Kadel's avatar
      [fidl-lint] Adding simpler checks · 3932ab3a
      Rich Kadel authored
      All "case" checks are complete.
      
      Test: fidl-compiler-test
      Change-Id: Ie773e757177e82a76e30e1bfafaf1de97010e49b
      3932ab3a
    • Roland McGrath's avatar
      [fidl] Start transition from - to . names for //zircon/public/fidl · 0731683a
      Roland McGrath authored
      This is part of a soft transition to replace fuchsia-foo-bar names
      with fuchsia.foo.bar names for the directory and GN target names for
      FIDL libraries defined in //zircon/system/fidl and accessed in legacy
      GN via //zircon/public/fidl.  After the transition, the directory and
      GN target names will exactly match the FIDL library names everywhere.
       1. Make //zircon/public/fidl/a.b.c alias //zircon/public/fidl/a-b-c
       2. Convert all //zircon/public/fidl uses outside fuchsia.git & integrate
       3. Convert all //zircon/public/fidl uses inside fuchsia.git
       4. Remove transition hacks
       5. Enforce naming conventions in GN templates
      This is step 1.
      
      Bug: ZX-3365 #comment Start soft transition for //zircon/public/fidl
      Change-Id: Iabbbee27a2a442bf80da88efb9bd9ddeb729215b
      0731683a
    • Roland McGrath's avatar
      [build] Allow //zircon/public/lib/foo vs //zircon/public/tool/foo name reuse · c9d487fe
      Roland McGrath authored
      Subdirectories in different parts of the Zircon tree reusing the same
      name is not a problem, and neither is //zircon/public/*/NAME reuse.
      Fix the metadata-based target generation magic to distinguish the
      category (i.e. subdirectory under //zircon/public/) so that names
      can be reused across categories.
      
      Bug: BLD-472 #done
      Test: local build with //zircon/public/lib/blobfs export enabled
      Change-Id: I592e52f524471a006bec2ff5942215ec277705dd
      c9d487fe
  2. May 11, 2019
    • Christopher Tam's avatar
      [tee][optee][rpc] Support file system commands · 4eb6a488
      Christopher Tam authored
      This change brings support for file-related file system RPC requests
      from the trusted world into the OP-TEE driver. In a later change,
      directory-related file system RPC requests will be supported.
      
      Issue: MTWN-169
      
      Test: Integration testing on astro with optee_test. Be sure to have
      the latest patch from upstream that fixes a storage bug in BL32 and the
      test storage TA signed/encrypted with the same keys as those in BL32.
      
      Change-Id: I8073c2ea83702f641ce6c3fd8647f3607c52a52a
      4eb6a488
    • docs-roller's avatar
      [gndoc] Update GN build arguments documentation · 0ed5270e
      docs-roller authored
      Test: CQ
      
      Change-Id: I530eb77e338a992583fd82719e4d9c099203e99e
      0ed5270e
    • Bryan Henry's avatar
      [component_manager] Don't assume /svc/fuchsia.pkg.PackageResolver always exists · 85d19081
      Bryan Henry authored
      This is currently only true in tests of component_manager that run as v1
      Components, but will not be true of the component_manager started from
      bootsvc.
      
      CF-750 #done
      
      Test: No package resolver case is covered by new no_pkg_resolver_test.
      Package resolver present case is covered by existing integration tests
      like routing_integration_test.
      
      Change-Id: Ide07ab3d87ee5c31ea8f036c7dbb3c3b59c7fc00
      85d19081
    • Doug Evans's avatar
      [trace-reader] Add FileReader class · a87715f5
      Doug Evans authored
      Change-Id: Ic00e280eaaa5028ee2fd2c16a8cdcf7172ea6aca
      a87715f5
    • Doug Evans's avatar
      [kernel perfmon] Remove status result from Stop/Fini handlers · 6393ecad
      Doug Evans authored
      ... they always succeed, no result necessary.
      
      Tested: No functional change
      fx run-test cpuperf_tests
      
      Change-Id: I2bcbf7b55a2ef8491b37bbee67736f7cb9410c72
      6393ecad
    • Francois Rousseau's avatar
      [feedback] spawn a new DataProvider on every request · 1ba56adc
      Francois Rousseau authored
      by having feedback_agent spawn a new process on every fuchsia.feedback.DataProvider
      request it gets, we can better isolate each request handling, in
      particular if it has dangling threads due to some attachment collection
      hanging and make sure the process exits upon request fulfillment
      
      DX-1497 #comment
      
      TESTED=`fx run-test feedback_agent_tests`
      
      Change-Id: I103b36ef2cfc4df6a7eef5a5783ddc22e93357e8
      1ba56adc
    • Bryan Henry's avatar
      [component_manager] Split into lib & bin targets to improve testability · 05321628
      Bryan Henry authored
      A future change will move some of the startup code out of main() to
      enable testing it in multiple different environments.
      
      Test: Refactor, no change in behavior. Covered by existing tests.
      
      Change-Id: I1fb82522268acdc0265aba6490230fb4404e995f
      05321628
    • Doug Evans's avatar
      [trace-reader tests] Move to live with source, convert to zxtest · fa21ba04
      Doug Evans authored
      Change-Id: I54af54cc510d2feef1fdb2575268ddb8fc25b4fb
      fa21ba04
    • Ariana Mott's avatar
      [scenic] Add FramePredictor, to replace hard-coded 8ms prediction · 48386b28
      Ariana Mott authored
      Forked from fxr/256673. This introduces the frame prediction logic
      that will be used by the default_frame_scheduler.
      
      SCN-960 #comment
      FL-255 #comment
      
      Tested: added unittests for introduced classes.
      Change-Id: Ieee6426c6d58c69e65612ad7cf587a5cca8c8167
      48386b28
    • Roland McGrath's avatar
      [docs] Update advice about standard C++ library, hermetic C++ · ad7e4d6e
      Roland McGrath authored
      Bug: ZX-1751 #done
      Test: docs-only change
      Change-Id: I07d59f9b5179b2faa357dba306d3e13c80c8e14e
      ad7e4d6e
    • Roland McGrath's avatar
      [zircon] Use shared libc++ in Zircon · 9f565bdd
      Roland McGrath authored
      Bug: ZX-1751 #comment Full libc++ use now available in Zircon.
      Change-Id: I3f2f7e2e9a8f81129d27374e6e396a75586f6774
      9f565bdd
    • Aaron Green's avatar
      [zircon][gn] Add fuzzer environments · a125d184
      Aaron Green authored
      This CL adds environments derived from the standard environments that
      instrument code with '-fsanitize=fuzzer' and link against
      libclang_rt.fuzzer.a.
      
      Bug: SEC-251
      Test: noop-test; see zircon/public/gn/toolchain/environment.gni
      Change-Id: I7d25134b78eb2fbdead551599e15218b68ecd5ba
      a125d184
    • docs-roller's avatar
      [gndoc] Update GN build arguments documentation · 7d4acd96
      docs-roller authored
      Test: CQ
      
      Change-Id: I47754d1308745f71fffe2197150e4e66c64b850d
      7d4acd96
    • Roland McGrath's avatar
      [kernel] Remove zxcpp entirely · ba4b4e22
      Roland McGrath authored
      The kernel supplies its own C++ ABI functions directly.
      
      Bug: ZX-1751 #comment Removed zxcpp entirely
      Change-Id: I3707353210d26aa98682d2b29e22dbbf3b44078e
      ba4b4e22
    • Roland McGrath's avatar
      [zircon] Use static (hermetic) libc++ instead of zxcpp in userland · e9bd2477
      Roland McGrath authored
      Bug: ZX-1751 #comment Removed zxcpp from userland
      Change-Id: Ic4fedc8e0c2362c1f26864629078cd86c6a59cac
      e9bd2477
    • Doug Evans's avatar
      [trace-manager] Pass on verbosity to configured providers · 5a615f0f
      Doug Evans authored
      ... and add VLOG message to perfmon event processing
      
      Change-Id: I3029a9c97a8c2710025d5e435fccf508e168f156
      5a615f0f
    • Sean Klein's avatar
      [test][digest] Relocate unit tests to exist alongside source · cd9c7326
      Sean Klein authored
      Change-Id: Ib244d3afefbad49c5534bf71882c7f21ceff3b9b
      cd9c7326
    • Marie Janssen 💖's avatar
      [bt] Eliminate ::common:: namespace. · f6de06b4
      Marie Janssen 💖 authored
      Remove the ::common:: namespace for sapphire-wide objects.
      
      fx format-code
      
      Test: bt-host-unittests, manual test of supported platforms (no
          behavior change)
      BT-870
      
      Change-Id: I810f29852e0da0d3d5bec25d57b2706d39be4e61
      f6de06b4
    • Erick Tryzelaar's avatar
      [pkg-uri] Implement PkgUri::repo() · 930b3afc
      Erick Tryzelaar authored
      This allows for a zero copy conversion of a package uri into
      a repo uri.
      
      Change-Id: I6281b1a323ac7fe8b65c0556f1d638cd3cd8d114
      930b3afc
    • Jeremy Manson's avatar
      [fidlcat] Add XUnion support · bc135434
      Jeremy Manson authored
      DX-1013 #comment Add XUnion support
      
      Change-Id: I778175f975f9783824531018dab47fab8cadba7f
      bc135434
    • Roland McGrath's avatar
      [zircon][gn][libc] Add ubsan-sancov-full variant · c556aa40
      Roland McGrath authored
      This new variant enables all the kinds of sancov instrumentation,
      not just the basic one that the standard runtime handles.  This
      exercises more of the instrumentation features that e.g. fuzzing
      also uses.  The special magic in the dynamic linker to cope with
      the instrumentation is extended to handle all the cases.
      
      Bug: ZX-3986 #comment ubsan-sancov-full variant
      Test: /boot/test/sys/noop-test.ubsan-sancov-full
      Change-Id: I2ca2bbaca732526ee10e0d60822410f48d8eb1ef
      c556aa40
    • Roland McGrath's avatar
      [zircon][gn] Fix legacy export of trace-handler library · d7a53561
      Roland McGrath authored
      This makes it possible for Fuchsia GN targets to depend
      on //zircon/public/lib/trace-handler or on a static library
      like //zircon/public/lib/trace-test-utils that uses it.
      
      Bug: BLD-473 #done
      Test: no immediate effect on the build
      Change-Id: If29ed149c178c2aae0d03316d518de6e2d3c22a0
      d7a53561
    • Sean Klein's avatar
      [test][async-loop] Relocate unit tests to exist alongside source · c9d95b25
      Sean Klein authored
      Change-Id: I572c24dbcaf13e0197037499e379f94956cc5aa7
      c9d95b25
    • Adam Barth's avatar
      [sdk] Add sys and vfs to the SDK · 10fb4a7c
      Adam Barth authored
      Several customers have started copying these libraries into their tree.
      We're now at a point where we can add these libraries to the SDK.
      
      DX-387 #done
      
      Test: Captured by the "api" files.
      Change-Id: I0d2ecb6a259a145a9563ed5d3c39e28b5b467c89
      10fb4a7c
    • Jinwoo Lee's avatar
      Revert "[amberctl] Mirror source edits to pkg_resolver" · 34034708
      Jinwoo Lee authored
      This reverts commit 96f91478.
      
      Reason for revert: Breaks tests of SDK consumers.
      
      Original change's description:
      > [amberctl] Mirror source edits to pkg_resolver
      > 
      > To ease the migration from using amber source configs to using
      > pkg_resolver repository configs, this change modifies the source
      > management commands in amberctl to perform roughly equivalent operations
      > on the RepositoryManager and rewrite.Engine.
      > 
      > Test: modified amberctl test to verify pkg_resolver state
      > 
      > PKG-704 #done
      > 
      > Change-Id: I92853a61b59a60693014853862a9494c9d227caa
      
      TBR=raggi@google.com,etryzelaar@google.com,kevinwells@google.com,computerdruid@google.com,galbanum@google.com
      
      Change-Id: Ie0621d15882b5f71bc737fa2a9558b6d965cc361
      No-Presubmit: true
      No-Tree-Checks: true
      No-Try: true
      34034708
    • Casey Dahlin's avatar
      [debugger][elflib] Move Build ID extraction to elflib · 5b27d23c
      Casey Dahlin authored
      Change-Id: Ie2919f76b09db6f7419fc3be598af72f6f10ef24
      5b27d23c
    • Rich Kadel's avatar
      [fidl][docs] Add exception for repeating names if ambiguous · 513cd4f5
      Rich Kadel authored
      Test: No behavior change
      Change-Id: Ibd3183ee8337a77d30701a4e75cf635458dc10f4
      513cd4f5
    • Francois Rousseau's avatar
      [crash] switch to using fsl::GetObjectName to get the process name · 16d75f60
      Francois Rousseau authored
      the default name (from when the lookup fails) changes from "unknown-package" to "", which is fine and package was a bit of a misnomer anyway
      
      TESTED=`fx run-test crashpad_agent_tests`
      
      Change-Id: I51e4b86e2ccc74bc3e2d8751480211139f68aab2
      16d75f60
    • Francois Rousseau's avatar
      [feedback] add utilies to check the number of data provider processes in tests · 786d36b7
      Francois Rousseau authored
      for now, there are no data provider processes, but in the future there
      will be one per connection to fuchsia.feedback.DataProvider. This CL is
      just to introduce the test utilities.
      
      DX-1497 #comment
      
      TESTED=`fx run-test feedback_agent_tests -t feedback_agent_integration_test`
      
      Change-Id: Ia5cb07334d3d27874318cd6b54fcf4afaa336da0
      786d36b7
    • Joshua Seaton's avatar
      [build-id] Reland "[build-id] pass extended ids.txt to dump_breakpad_symbols" · 1a4a0325
      Joshua Seaton authored
      This is a reland of e52f9b4c
      
      Original change's description:
      > [build-id] pass extended ids.txt to dump_breakpad_symbols
      >
      > This change creates a concatenation of the ids.txt produced by the build
      > that that produced by the .build-id archives downloaded from CIPD - and
      > passes that to dump_breakpad_symbols.
      >
      > Tested locally.
      >
      > Bug: INTK-884 #comment
      > Bug: INTK-969 #comment
      > Change-Id: I9d96f8098239ca6a1c828be0a44dbe36a0c3b14a
      
      Bug: INTK-884 #comment, INTK-969 #comment
      Change-Id: I90adfc83c16bc00e80ff97f8bfe7190ae4607e21
      1a4a0325
    • Marie Janssen 💖's avatar
      [bt][host] Rename DeviceId => PeerId · 94ae69c8
      Marie Janssen 💖 authored
      DeviceId only referred to peers and is renamed to be more consistent
      with the idea that it is an identifier for a bluetooth peer.
      
      fx format-code because "Peer" is 2 characters shorter
      
      Test: bt-host-unittests (no behavior change)
      BT-870
      
      Change-Id: Ib97223e106bf9a9efb506de343ce218ba0b50c62
      94ae69c8
    • Yifei Teng's avatar
      [fidlc] Emit transitive dependencies in "library_dependencies" JSON IR · f76d76f3
      Yifei Teng authored
      When library A depends on B and B depends on C, declarations from C are
      not emitted as part of the JSON IR for library A. This would not work in
      backend generation if A references declarations from C.
      
      TEST:
      ./scripts/build-zircon-x64 -A
      ./scripts/ninja -C ./build-asan system/utest:host
      build-asan/host-x64-linux-clang/obj/system/utest/fidl-compiler/fidl-compiler-test
      
      Change-Id: Ic48ec8b2e55280ac6bedbd78853c2bd1303df288
      f76d76f3
    • Yifei Teng's avatar
      [fdio][llcpp] Start of migration to llcpp · 9c7feb81
      Yifei Teng authored
      Begin in remoteio.cpp, by rewriting calls to the C binding as llcpp.
      
      Public APIs converted:
      - fdio_open_at
      - fdio_service_connect_at
      - fdio_service_clone
      - fdio_service_clone_to
      
      As of this patch, some implementations are less than ideal: most
      helper functions take zx_handle_t as arguments as opposed to zx::channel
      or zx::unowned_channel, which encodes ownership in the type. As a
      result, there are frequent casting from zx_handle_t to zx::channel etc.
      and vice versa. As greater proportion of the library is converted to
      C++, these conversions may be removed.
      
      Change-Id: I9a8bc4df228610daafa64eec7ebc99ba194a3489
      9c7feb81
    • Roland McGrath's avatar
      [zircon][gn] Add build-tests target · 60950397
      Roland McGrath authored
      Instantiate the noop-test target in each variant of each environment()
      defined by standard_environments().  The top-level build-tests target
      collects all of these and is in the default set.  This ensures that
      each supported variant gets exercised and can successfully build an
      executable.
      
      Bug: BLD-430
      Change-Id: Iccd823f7711c9cff04e0c6bf891f7d4541023b27
      60950397
    • Tom Turney's avatar
      [sl4f] Rustfmt all of SL4F · ebca4e0f
      Tom Turney authored
      Bug: NONE
      Test: fx build-push sl4f
      Change-Id: If2b2f90cd059f82122c98b1ca73f99ad743b1512
      ebca4e0f
Loading