Skip to content
Snippets Groups Projects
  1. Apr 25, 2019
    • Zach Anderson's avatar
      [zircon] Revert "[zircon] Reformat zxcrypt and minfs partitions when the partition is" · ef6dd7d3
      Zach Anderson authored
      This reverts commit 2dcfdba2.
      
      Reason for revert: Failed boot on vim2
      
      Original change's description:
      > [zircon] Reformat zxcrypt and minfs partitions when the partition is
      > missing. Added a unit test that failed without the changes.
      >
      > Change-Id: Idaf9913bf9006384e021f39047df1e8e8a855369
      
      TBR=smklein@google.com,aarongreen@google.com,abarth@google.com,zarvox@google.com,jseibert@google.com
      
      Change-Id: I0e755d84dc4839e8968308ca753aa1d9c9c5038b
      ef6dd7d3
    • Shac Ron's avatar
      [io-scheduler] Sort ops into streams · 212d3114
      Shac Ron authored
      Insert ops into streams, fetch and issue. Convert SchedulerOp from
      struct to class.
      Scheduler can balance the ordering by streams (currently dummy
      round-robin).
      
      Change-Id: I4b8f3e64eff1360f2334afa67036cbc09db105fc
      212d3114
    • Doug Evans's avatar
      [tracing docs] Move quick-start,trace-format docs to own subdirs · 585bb833
      Doug Evans authored
      ... and add link to tracing-boot.md.
      
      Change-Id: I0818ba4b0de23940b88b9b14b61f77d435077f39
      585bb833
    • Doug Evans's avatar
      [tracing docs] Pass --categories=all in example · 45ea1099
      Doug Evans authored
      Blast the reader with all the potential data that is available.
      
      Also, re-indent, improves cut-n-pastability.
      
      Change-Id: I4944aecc1616c41fc60ed7be13f55586b26b11b6
      45ea1099
    • Roland McGrath's avatar
      [build] Rename zircon/system/banjo/* directories/targets to match library names · cb9ac26e
      Roland McGrath authored
      The old - names were only ever used because of limitations of the
      old Zircon build system, which is no longer an issue.  The target
      name in banjo_library() must always match the library name.  For
      public Banjo libraries, the directory name matches that name too.
      
      Change-Id: Ifff61436ec7d15eb53dfd21349c4367047e691d4
      cb9ac26e
    • David Worsham's avatar
      [media] Remove unused method · a2e2baae
      David Worsham authored
      Migration of mediaplayer to ViewToken is complete.
      
      Tested: CQ; no behavior changed
      SCN-1018 #comment
      SCN-1291 #comment
      
      Change-Id: Iae3446ec6c7ea4eb280c985dbec9d73bfe3e5204
      a2e2baae
    • Roland McGrath's avatar
      [zircon][gn] Move scripts used in the build to where they're used · a69ac806
      Roland McGrath authored
      Test: no effect on the build
      Change-Id: I819338ff6309f7487f0ebe72b63f11031a8c631b
      a69ac806
    • Joe Brennan's avatar
      [sl4f] Made SL4F listen on both IPv6 and IPv4 interfaces · eda5d797
      Joe Brennan authored
      SL4F calls can now be made on the IPv4 and IPv6 interfaces
      
      Bug: CONN-56
      Test: Verifed SL4F could be connected on both IPv4 and IPv6
            interfaces.
      Change-Id: Ia6a68abc3c82a1484a4718f2aec3f8a88b399c77
      eda5d797
    • Arman Uguray's avatar
      [bt][gap] Distribute local identity information during pairing · 98acbbf0
      Arman Uguray authored
      LowEnergyConnectionManager now responds to local identity information
      requests during the feature exchange and key distribution phases of SMP
      pairing.
      
      Bug: BT-243 #done
      Test: 1. Run bt-pairing-tool, bt-le-battery-peripheral, and
               bt-le-peripheral -c -n <NAME> on Fuchsia. This will put Fuchsia
      	 into LE peripheral mode with one GATT service that requires
      	 security.
            2. Connect to Fuchsia from iOS using the nRF Connect app. This
               should trigger LE pairing. Respond to the authentication challenge
      	 using bt-pairing-tool (local passkey entry on ToT). The battery
      	 service characteristic should be readable and notifiable
      	 without any errors.
            3. Pipe the output of bt-snoop-cli to Wireshark and confirm that:
               a. Fuchsia configured advertising using a random address.
      	 b. Local IRK and identity address were distributed over SMP.
      	 Check that the IRK matches what has been persisted in the
      	 stash. Check that the identity address matches the address of
      	 the active adapter (confirm via bt-cli).
            4. Reboot Fuchsia, repeat step #1. Connect again using nRF
               Connect. The battery service should be accessible without
      	 additional pairing errors.
            5. Check the snoop logs to confirm that Fuchsia used a different
               RPA when advertising.
      Change-Id: Ibb78b2de3049f9f351215ed4d878acfcca5569df
      98acbbf0
    • P.Y. Laligand's avatar
      [sdk] Restore a few packages in the tools bundle. · 96cc21c1
      P.Y. Laligand authored
      While several images already contain these packages, some images don't
      and the packages need to be available in the universe set.
      
      Bug: DX-1427 #done
      Change-Id: If10223ec017312c797defca113ab21e30cf159b9
      96cc21c1
    • Pascal Perez's avatar
      [fidlc] Use std::optional for optional handle types · a9459e86
      Pascal Perez authored
      Clearer than using unique_ptr, and makes copying easier, which we need
      for templates.
      
      Change-Id: Ie851ab3c461d8b4c3b58c08c7837685e2865f96c
      a9459e86
    • Mike Voydanoff's avatar
      [imx8] Remove NXP imx8 support · 075be3db
      Mike Voydanoff authored
      There has been no active development in about a year.
      We can resurrect this from the git history if we need it in the future.
      In the mean time, lets eliminate code that is not being used or tested.
      
      TEST: builds without imx8 support
      Change-Id: Ie9b04b476e777e3311bfaec3465804a1e81c350e
      075be3db
    • Mike Voydanoff's avatar
      [board][eagle] Remove support for the eagle board · e9fd82cc
      Mike Voydanoff authored
      TEST: builds, bringup build boots normally on cleo.
      Change-Id: I94227a565ce40b8e93db79aa4457862666466bec
      e9fd82cc
    • P.Y. Laligand's avatar
      [boards] Add definitions for QEMU. · 130d7648
      P.Y. Laligand authored
      Bug: BLD-461
      Change-Id: I60372456d27eebdc5ebcb3e1b1d097b190603d1b
      130d7648
    • Dustin Green's avatar
      [codec] remove deprecated OnOutputConfig event · fd0cfc8b
      Dustin Green authored
      Now that Chromium is handling OnOutputConstraints() and
      OnOutputFormat(), we can remove OnOutputConfig().
      
      Tested: use_h264_decoder_test uses the new events on CQ
      Change-Id: I57bf213ba8fa5d225b5bfe961c5f5fc9772ce601
      fd0cfc8b
    • Mike Voydanoff's avatar
      [boards] Add myself (voydanoff) to the OWNERS file · a9197fd2
      Mike Voydanoff authored
      TEST: n/a
      Change-Id: I5e827e029e61dbc36392f1063dedf64808ff4dea
      a9197fd2
    • Pascal Perez's avatar
      [fidlc] Simplifying name modeling · cf164b41
      Pascal Perez authored
      A name is scoped (by a library), and either has a source location, or an
      anonymous name. Modeling this as a std::variant for clarity, and to
      simplify owning semantics. We can now add a copy constructor with ease.
      
      Change-Id: I7e6602e0abdc5c08535452253e324b13b8b9dade
      cf164b41
    • Christopher R. Johnson's avatar
      [libiquery] Implement reading function. · 97c06257
      Christopher R. Johnson authored
      This new function supports reading an inspect::Location into an
      inspect::Source asynchronously.
      
      CF-714: #progress
      
      Change-Id: Ia7bf90b7b9ba174ebedcabf9303ed9a29b085676
      97c06257
    • Dale Sather's avatar
      [mediaplayer] don't panic if the audio render stalls · ee83f76b
      Dale Sather authored
      This CL changes the behavior of the media player when the audio
      renderer holds on to packets too long, which generally happens due to
      scheduling issues. Because the player has allocated a limited amount
      of space for transferring packets between itself and the renderer,
      that space can be exhausted if the renderer doesn't retire packets
      quickly enough.
      
      The fix is to drop packets instead of panicking. There are a few
      paths that had to be fixed to enable this. Sometimes the memory
      allocation failures occur in a callback from the audio decoder.
      Sometimes audio has to be interleaved after the decoder, and the
      allocation failure is detected there. Sometimes (when no decoder
      is needed) audio needs to be copied from system memory into vmo
      memory, and the failure happens prior to the copy.
      
      Two tests have been added to replicate these failure modes. Those
      tests produced panics before the fixes were introduced.
      
      TEST: new CQ tests
      Change-Id: I26582d19481c6716f6a4141e6717b68670ea44f1
      ee83f76b
    • Jody Sankey's avatar
      [identity] Move account internal FIDL to /src/identity. · 0b1cef55
      Jody Sankey authored
      AUTH-196 #comment
      
      Change-Id: I213f7e4440f104ce27535b685a4da100f0526cbd
      0b1cef55
    • David Worsham's avatar
      [scenic] Fix bad formatting from previous CL · 91feb83e
      David Worsham authored
      Tested: CQ; no behavior change
      SCN-580 #comment
      
      Change-Id: I496893099aef041de0c721b4840bd104adc10781
      91feb83e
    • Alex Min's avatar
      [modular] Remove use_account_manager flag and codepaths · 97982728
      Alex Min authored
      Change-Id: I3df4cbd22b97e362c8c80bcb86e618ba0336cfe0
      97982728
    • Jody Sankey's avatar
      [identity] Remove devstorage.read_write OAuth scope · b40ea487
      Jody Sankey authored
      AUTH-202 #comment
      
      Change-Id: I52c5ed31524d5c50cde393f1c1d56f1a0e97bbe5
      b40ea487
    • Satsuki Ueno's avatar
      [identity] Enforce maximum size on token cache · de2d1d50
      Satsuki Ueno authored
      This places an upper bound on the size of the cache.
      Evicts expired tokens on get/put operations, and evicts a random token
      when needed to make room for another token.
      
      Change-Id: I8a9eb2926ab134c3e756b1f6b152c56e5bd1a199
      de2d1d50
    • P.Y. Laligand's avatar
      [scripts] Remove scripts related to JSON build packages. · ca0e1ff3
      P.Y. Laligand authored
      Bug: BLD-457 #done
      Change-Id: I615d2fb526ff8811f2c94219dbfe7fef2443f480
      ca0e1ff3
    • Rich Kadel's avatar
      [zircon] [scripts] Add continuation to simplify tun/tap after reboot · 1d9c6ccc
      Rich Kadel authored
      To make fx run -N even more friendly, I added "&&" (continue with
      the next command if the first command succeeds) and "\" (continue
      command on the next line) so users can simply copy and paste the
      helpful commands in a single copy-and-paste action.
      
      I find I have to re-enable tun/tap after each reboot of my Linux
      machine. Of course, I don't remember until I try the "fx run" command,
      but "fx run" helpfully prints the suggested commands to do that:
      
        To use qemu with networking on Linux, configure tun/tap:
        sudo ip tuntap add dev qemu mode tap user richkadel
        sudo ifconfig qemu up
      
      Without thinking about it, it might seem like you should be able to
      copy and paste both lines, but this fails because the sudo in the first
      line tries to use the second line as the password.
      
      The change simplifies running the commands, and may be less confusing.
      
      Test: Verified results locally (very minor change)
      Change-Id: I45ebfda917c96ba5704f108119b65a06aaba0ae4
      1d9c6ccc
    • Bruno Dal Bo's avatar
      [netemul] Refactor error codes · 51ae491f
      Bruno Dal Bo authored
      - Sandbox was using fuchsia::sys::TerminationReason for its termination
      codes, refactored termination process to return a netemul-specific
      status code.
      - Unit tests now check for expected status codes.
      
      Change-Id: I7bc48a34da311d8de1f5a5d6caab27f600e3c5ba
      51ae491f
    • Bruno Dal Bo's avatar
      [netemul] Fix race in Sandbox destructor · 037e90c0
      Bruno Dal Bo authored
      - Sandbox was using a default destructor, which could cause race with
      the FIDL bindings error callbacks kept in Sandbox::procs_.
      
      BUG: FLK-178 #done
      Change-Id: I46189bde74fa4d83faec27f503f9ec032c605906
      037e90c0
    • Rich Kadel's avatar
      [tools] Update build command when needed for fidl-format · 1f0a445e
      Rich Kadel authored
      The old build command is no longer working. I believe this change
      addresses the original intent.
      
      Test: tested locally to confirm it works
      
      Change-Id: Ib097f5e713b4794a66daaf9332da2db2c0158cc1
      1f0a445e
    • Yifei Teng's avatar
      [fidl] Fix decode/validate crash in case of tables of (nested) arrays of · 71aec42d
      Yifei Teng authored
      primitives.
      
      With the introduction of coding tables for boring types inside FIDL tables
      to support linearization, array coding tables may have a null element,
      when the array is in a FIDL table but the element is a boring type.
      
      TEST: fx run-host-tests fidl_cpp_unittests
      Change-Id: I56380c36369e4b501f5b82aa9cfc673c564f82a1
      71aec42d
    • Nelly Vouzoukidou's avatar
      [ledger][benchmark] Add first memory usage benchmark. · 968b7a90
      Nelly Vouzoukidou authored
      Test: put_memory.tspec
      
      This CL introduces LedgerMemoryEstimator, providing an API to capture
      Ledger's memory usage. This API is used in this CL to measure the
      memory usage after every insertion in the `put` benchmark.
      
      Change-Id: I2822f649e1382875b21327f3d5dbf215eac94fd4
      968b7a90
    • Corey Tabaka's avatar
      [kernel][sched] Tweak fair priority curve and preemption / latency. · 6af83dfd
      Corey Tabaka authored
      - Switch to an exponential priority-to-weight curve to yield a ~65%
        bandwidth increase to priority 24 (high priority) over priority 16
        (default priority), and a constant ~10% difference between adjacent
        priorities.
      - Tweak the bandwidth adjustment to use the start of the current time
        slice when changing the task rate so that preemptions happen faster
        when high priority work is available.
      - Adjust the target latency to provide a time slice more similar to the
        previous scheduler when a high priority thread competes with lower
        priority threads.
      - Minor refactor to remove weight-to-priority conversions, which would
        require a fixed-point logarithm to compute the inverse of the
        exponential term.
      - Optimize the priority-to-weight conversion using a constant table.
      
      Bug: MTWN-269
      Bug: ZX-3504
      Test: Detailed tracing of synthetic workload; observe ratios of
            bandwidth between priority 16 and priority 24 tasks.
      Change-Id: I596c45cf3d8c94f93c986508f9ec1007624ac9f3
      6af83dfd
    • Arman Uguray's avatar
      [bt][sm] Distribute local IRK if available · 1f889682
      Arman Uguray authored
      - Bearer now queries the upper layers to determine if local identity
        information is available during the feature exchange phase.
      - Local identity information is distributed during Phase 3 if provided
        by the delegate.
      
      Bug: BT-243
      Test: bt-host-unittests --gtest_filter=SMP_*
      Change-Id: Id07a799c4ae7e9cf4c3e0c80588d6d738b445709
      1f889682
    • Doug Evans's avatar
      [perfmon] Rename perfmon_config_t to perfmon_ioctl_config_t · 2201f2b7
      Doug Evans authored
      ... in preparation for splitting up config into FIDL config and client config.
      
      ZX-3490 #comment patch
      
      Change-Id: If7c0161d389fb238fc5540c3f496c32df7cc4388
      2201f2b7
    • P.Y. Laligand's avatar
      [images] Add argument to have QEMU-related images added to build archives. · fdad6093
      P.Y. Laligand authored
      Bug: BLD-461
      Change-Id: I720d29f5f201d4efa8aa9e41853008c9952ad948
      fdad6093
    • Vardhan Mudunuru's avatar
      [devcoordinator] REVERT: Fix buggy composite device behavior · d6a03883
      Vardhan Mudunuru authored
      This reverts commit 75b35cea.
      
      Reason for revert: This CL is a suspect for the currently red fuchsia-roller;  it fails the /boot/test/ddk/platform-bus-test test.
      
      Original change's description:
      > [devcoordinator] Fix buggy composite device behavior
      >
      > If a component was a device with MUST_ISOLATE set on it, we would try to
      > bind to the non-proxied side of it, which doesn't speak the device
      > protocols.
      >
      > Change-Id: I64eaccec5d51997f4374d0277b040a952f681667
      
      TBR=teisenbe@google.com,abarth@google.com,jocelyndang@google.com
      
      Change-Id: Id9d48682336ea648599cfead60444c5074c4ae1e
      No-Presubmit: true
      No-Tree-Checks: true
      No-Try: true
      d6a03883
    • Roland McGrath's avatar
      [build] Fix integration with Zircon build when it uses ASan · 1a86b25a
      Roland McGrath authored
      This entailed another substantial cleanup of environment_redirect()
      to handle more cases.  It also necessitated reorganizing the deps
      in the Zircon build for the legacy integration.  The new arrangement
      should be much easier to understand.
      
      Bug: BLD-447 #comment Fixed --variant asan build
      Test: fx set bringup.x64 --variant asan && fx build
      Change-Id: I2cd8d7aa95eba5f34af421c75d634f31d988a1f8
      1a86b25a
    • Eric Wang's avatar
      [wlantap] replace IOCTL with FIDL · f81124ad
      Eric Wang authored
      Test: $ runtests /pkgfs/packages/wlan-hw-sim-tests/0/test
      ZX-3496 #done
      
      Change-Id: I52e861590cda6720b22577cb6b9496d5ed68ddec
      f81124ad
    • Chris Fontas's avatar
      [ES-148] Write escher enum flag unit tests · 7e888a32
      Chris Fontas authored
      [Still in progress, please do not review yet]
      
      Change-Id: Ifa4641b3a5e48091aac1e10d9052dfb927aa962c
      7e888a32
    • Rob Krten's avatar
      [docs][tracing] Link to Chromium trace docs · c778a1e1
      Rob Krten authored
      Missing a link; created an external one to Chromium docs as a placeholder
      
      Change-Id: I352b559f728a8fbedda334cb72746538a5937ff3
      c778a1e1
Loading