summaryrefslogtreecommitdiff
path: root/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* core: Remove usage of unicornGravatar Lioncash2020-11-0311-418/+15
| | | | | | | | Unicorn long-since lost most of its use, due to dynarmic gaining support for handling most instructions. At this point any further issues encountered should be used to make dynarmic better. This also allows us to remove our dependency on Python.
* Merge pull request #4873 from lioncash/common-errorGravatar bunnei2020-11-039-31/+49
|\ | | | | common: Enable warnings as errors
| * common: Enable warnings as errorsGravatar Lioncash2020-11-029-31/+49
| | | | | | | | Cleans up common so that we can enable warnings as errors.
* | Merge pull request #4878 from bunnei/unload-nrrGravatar bunnei2020-11-031-1/+15
|\ \ | | | | | | hle: service: ldr: Implement UnloadNrr.
| * | hle: service: ldr: Implement UnloadNrr.Gravatar bunnei2020-10-311-1/+15
| | | | | | | | | | | | - Used by Final Fantasy X/X-2 HD Remaster.
* | | Merge pull request #4865 from ameerj/async-threadcountGravatar bunnei2020-11-011-8/+9
|\ \ \ | | | | | | | | async_shaders: Increase Async worker thread count for >8 thread cpus
| * | | async_shaders: Increase Async worker thread count for 8+ thread cpusGravatar ameerj2020-10-291-8/+9
| | | | | | | | | | | | | | | | Adds 1 async worker thread for every 2 available threads above 8
* | | | Rename to align with switchbrew and remove gpu function (#4714)Gravatar Levi Behunin2020-11-012-16/+10
| | | | | | | | | | | | | | | | | | | | * Rename to align with switchbrew * Rename to align with switchbrew and remove gpu function that checks if clearing should be done.
* | | | Merge pull request #4853 from ReinUsesLisp/fcmp-immGravatar bunnei2020-10-312-1/+4
|\ \ \ \ | |_|/ / |/| | | shader/arithmetic: Implement FCMP immediate + register variant
| * | | shader/arithmetic: Implement FCMP immediate + register variantGravatar ReinUsesLisp2020-10-282-1/+4
| | | | | | | | | | | | | | | | Trivially add the encoding for this.
* | | | Merge pull request #4872 from jbeich/clangGravatar LC2020-10-301-1/+1
|\ \ \ \ | |_|_|/ |/| | | video_core: unbreak -Werror in NVDEC with Clang
| * | | video_core: unbreak -Werror in NVDEC with ClangGravatar Jan Beich2020-10-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | src/core/hle/service/nvdrv/devices/nvhost_nvdec_common.cpp:41:15: error: unused variable 'OutOfMemory' [-Werror,-Wunused-const-variable] constexpr u32 OutOfMemory{static_cast<u32>(-12)}; ^
* | | | CMakeLists: Resolve MSVC build failuresGravatar Lioncash2020-10-301-1/+0
|/ / / | | | | | | | | | Prevents the compiler tripping up about Windows headers.
* | | Merge pull request #4868 from lioncash/discard-errorGravatar bunnei2020-10-303-6/+21
|\ \ \ | | | | | | | | General: Make ignoring a discarded return value an error
| * | | General: Catch more expressions with no effect on MSVCGravatar Lioncash2020-10-301-0/+4
| | | | | | | | | | | | | | | | | | | | MSVC lets us fine-tune catching expressions with no side-effects a little more.
| * | | General: Make ignoring a discarded return value an errorGravatar Lioncash2020-10-303-6/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Allows our CI to catch more potential bugs. This also removes the [[nodiscard]] attribute of IOFile's Open member function. There are cases where a file may want to be opened, but have the status of it checked at a later time.
* | | | common/stream: Be explicit with copy and move operatorsGravatar Lioncash2020-10-291-3/+9
| | | |
* | | | vp9: Be explicit with copy and move operatorsGravatar Lioncash2020-10-291-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | It's deprecated in the language to autogenerate these if the destructor for a type is specified, so we can explicitly specify how we want these to be generated.
* | | | vp9: Mark functions with [[nodiscard]] where applicableGravatar Lioncash2020-10-292-13/+13
| | | | | | | | | | | | | | | | | | | | Prevents values from mistakenly being discarded in cases where it's a bug to do so.
* | | | vp9: Provide a default initializer for "hidden" memberGravatar Lioncash2020-10-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The API of VP9 exposes a WasFrameHidden() function which accesses this member. Given the constructor previously didn't initialize this member, it's a potential vector for an uninitialized read. Instead, we can initialize this to a deterministic value to prevent that from occurring.
* | | | vp9: Make some member functions internally linkedGravatar Lioncash2020-10-292-58/+54
|/ / / | | | | | | | | | | | | These helper functions don't directly modify any member state and can be hidden from view.
* | | Merge pull request #4837 from lioncash/nvdec-2Gravatar bunnei2020-10-2913-88/+81
|\ \ \ | |_|/ |/| | nvdec: Minor tidying up
| * | h264: Make WriteUe take a u32Gravatar Lioncash2020-10-272-7/+8
| | | | | | | | | | | | Enforces the type of the desired value in calling code.
| * | vp9: std::move buffer within ComposeFrameHeader()Gravatar Lioncash2020-10-271-1/+1
| | | | | | | | | | | | We can move the buffer here to avoid a heap reallocation
| * | vp9: Remove dead codeGravatar Lioncash2020-10-271-6/+0
| | |
| * | vp9: Join declarations with assignmentsGravatar Lioncash2020-10-271-7/+8
| | |
| * | vp9: Remove pessimizing movesGravatar Lioncash2020-10-271-2/+2
| | | | | | | | | | | | The move will already occur without std::move.
| * | vp9: Resolve variable shadowingGravatar Lioncash2020-10-271-4/+4
| | |
| * | nvdec: Tidy up header includesGravatar Lioncash2020-10-2713-62/+59
| | | | | | | | | | | | Prevents a few unnecessary inclusions.
* | | Merge pull request #4781 from german77/GChotplugGravatar bunnei2020-10-293-303/+433
|\ \ \ | | | | | | | | Add hotplug, rumble and fix 3rd party adapters for the GC adapter
| * | | Add hotplug, rumble and fix 3rd party adapters for the GC adapterGravatar german2020-10-283-303/+433
| | | |
* | | | kernel/process: Add missing <ctime> includeGravatar Morph2020-10-291-0/+1
| | | | | | | | | | | | | | | | Fixes compilation on MSVC
* | | | Merge pull request #4857 from liushuyu/masterGravatar LC2020-10-291-5/+0
|\ \ \ \ | | | | | | | | | | web_service: follow-up fix to #4842
| * | | | web_service: follow-up fix to #4842 ...Gravatar liushuyu2020-10-281-5/+0
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * The web_service http request is now fixed on Windows (R) platform. * The issue is due to a complicated race-condition in `httplib`, a detailed explanation is available at https://github.com/yhirose/cpp-httplib/pull/701 * A pending Pull Request on `httplib` has been applied to remedy the said race-condition. * The socket availability check is removed due to a behavioral chice of `httplib` that a socket will not be created before any actual request is sent.
* | | | Merge pull request #4835 from lat9nq/rng-default-timeGravatar bunnei2020-10-281-1/+1
|\ \ \ \ | | | | | | | | | | kernel: Use the current time as the default RNG seed
| * | | | kernel: Use the current time as the default RNG seedGravatar lat9nq2020-10-261-1/+1
| | | | | | | | | | | | | | | | | | | | Use the current time, not zero, as the default RNG seed.
* | | | | Merge pull request #4838 from lioncash/syncmgrGravatar bunnei2020-10-282-9/+9
|\ \ \ \ \ | |_|/ / / |/| | | | sync_manager: Amend parameter order of calls to SyncptIncr constructor
| * | | | sync_manager: Amend parameter order of calls to SyncptIncr constructorGravatar Lioncash2020-10-272-9/+9
| | |/ / | |/| | | | | | | | | | Corrects some cases where the arguments would be incorrectly swapped.
* | | | web_service: web_backend: Handle socket errors with GenericRequest.Gravatar bunnei2020-10-281-0/+11
| | | | | | | | | | | | | | | | - Fixes a shutdown crash when we try to submit telemetry if there is a service issue.
* | | | video_core: cdma_pusher: Add missing LOG_DEBUG field in ExecuteCommand.Gravatar bunnei2020-10-281-1/+1
| | | |
* | | | Merge pull request #4846 from lioncash/service-fnGravatar bunnei2020-10-285-1/+7
|\ \ \ \ | |_|_|/ |/| | | service: Update function tables
| * | | service: Update function tablesGravatar Lioncash2020-10-275-1/+7
| |/ / | | | | | | | | | Updates function tables according to info on SwitchBrew.
* | | Merge pull request #4851 from ReinUsesLisp/core-threads-raceGravatar LC2020-10-281-5/+0
|\ \ \ | | | | | | | | hle/kernel: Remove unused registered_core_threads to fix data races
| * | | hle/kernel: Remove unused registered_core_threads to fix data racesGravatar ReinUsesLisp2020-10-271-5/+0
| | | | | | | | | | | | | | | | | | | | This member was only used on asserts and it triggered data races. Remove it to fix them.
* | | | Merge pull request #4850 from ReinUsesLisp/fiber-ptr-refGravatar LC2020-10-282-3/+3
|\ \ \ \ | | | | | | | | | | common/fiber: Take shared_ptr<Fiber> by copy in YieldTo
| * | | | common/fiber: Take shared_ptr<Fiber> by copy in YieldToGravatar ReinUsesLisp2020-10-282-3/+3
| |/ / / | | | | | | | | | | | | | | | | YieldTo does not intend to modify the passed shared_ptrs. Pass it by copy to keep a reference count while this function executes.
* | | | Merge pull request #4849 from ReinUsesLisp/fix-fiber-testGravatar LC2020-10-281-31/+40
|\ \ \ \ | | | | | | | | | | tests: Fix data race in fibers test
| * | | | tests: Fix data race in fibers testGravatar ReinUsesLisp2020-10-281-31/+40
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previous to this commit, the tests were using operator[] from unordered_map to query elements but this silently inserts empty elements when they don't exist. If all threads were executed without concurrency, this wouldn't be an issue, but the same unordered_map could be written from two threads at the same time. This is a data race and makes some previously inserted elements invisible for a short period of time, causing them to insert and return an empty element. This default constructed element (a zero) was used to index an array of fibers that asserted when one of them was nullptr, shutting the test session off. To address this issue, lock on thread id reads and writes. This could be a shared mutex to allow concurrent reads, but the definition of std::this_thread::get_id is fuzzy when using non-standard techniques like fibers. I opted to use a standard mutex. While we are at it, fix the included headers.
* | | | Merge pull request #4848 from ReinUsesLisp/type-limitsGravatar LC2020-10-282-1/+2
|\ \ \ \ | | | | | | | | | | video_core: Enforce -Werror=type-limits
| * | | | video_core: Enforce -Werror=type-limitsGravatar ReinUsesLisp2020-10-282-1/+2
| | | | | | | | | | | | | | | | | | | | Silences one warning and avoids introducing more in the future.