profile
viewpoint
Rémy Oudompheng remyoudompheng Paris, France

remyoudompheng/go-misc 113

Miscellaneous Go toys

remyoudompheng/bigfft 41

Big integer multiplication library for Go using Fast Fourier transform

remyoudompheng/go-vectops 24

Vgo is a compiler for vector expressions. It generates SSE2 code for amd64.

remyoudompheng/gigot 21

Gigot is a Go implementation of Git storage layer.

remyoudompheng/go-netlink 15

A go library implementing parts of the Linux netlink protocol

remyoudompheng/go-liblzma 8

Go bindings for XZ Utils/liblzma

remyoudompheng/go-alpm 6

go bindings to pacman's libalpm

remyoudompheng/go-aurjson 4

Small client for Archlinux's AUR-JSON interface

remyoudompheng/dwm 3

dwm hacks: learning to use XCB

remyoudompheng/go-dns 3

Go module for DNS queries

push eventremyoudompheng/android_device_motorola_athene

Rémy Oudompheng

commit sha 695bff4eed59b34c9da88e99acc559426ccc226c

Aggressive NTP and GPS settings.

view details

Rémy Oudompheng

commit sha 3e22658f9af1d4b16f936310b45ef73350d46f27

Increase ZRAM size to 1.5GB

view details

push time in 15 days

push eventremyoudompheng/android_kernel_motorola_msm8952

Rémy Oudompheng

commit sha f6921cf5c066f9fde4759c5a5b5900aec604d3ee

sched/fair: only warn instead of panic on negative HMP big/small counts The HMP scheduler seems to generate negative counts in some conditions.

view details

Rémy Oudompheng

commit sha 40e5b6019751c314faadf144c605d97676974748

Lower loglevel of a few verbose prints.

view details

push time in a month

push eventremyoudompheng/android_device_motorola_athene

Rémy Oudompheng

commit sha d7967ed16b140cd314c9f1c8df22ab2995bd0d92

Increase ZRAM size to 1GB

view details

push time in a month

push eventremyoudompheng/android_kernel_motorola_msm8952

Rémy Oudompheng

commit sha f5e5fb0afb2a657a017273b2d46e134af8914476

sched/fair: only warn instead of panic on negative HMP big/small counts The HMP scheduler seems to generate negative counts in some conditions.

view details

push time in 2 months

push eventremyoudompheng/android_kernel_motorola_msm8952

Rémy Oudompheng

commit sha cf9ea4abe4a453dad1e3f4b5a41b8ca6e7e1bdff

sched/fair: only warn instead of panic on negative HMP big/small counts The HMP scheduler seems to generate negative counts in some conditions.

view details

push time in 2 months

issue commentremyoudompheng/bigfft

How to fix "missing function body" build for mips, thank you !

You are using an old version. See issue #6 already fixed.

hongjinlin

comment created time in 3 months

push eventremyoudompheng/android_packages_inputmethods_LatinIME

Rémy Oudompheng

commit sha cdcc41d5fe839d8c751396754987da46246b97cd

Update smileys, people&body, animals&nature to Emoji 12.0

view details

Rémy Oudompheng

commit sha 671d5743d14ba1820a78e7376308cfde18872b28

Update remaining Emoji 12.0

view details

push time in 4 months

issue openedrhboot/efivar

Unable to use efibootmgr on a NVMe system

I am unable to use efibootmgr on a NVMe system running Debian 10, making bootloader install ineffective (or at least very inconvenient). Debian 10 runs Linux 4.19.67 and efibootmgr 15 and efivar 37.

# efibootmgr --create --loader /EFI/debian/grubx64.efi --label debian -v3 --disk /dev/nvme0n1p1
Could not prepare Boot variable: No such file or directory
error trace:
linux.c:390 device_get(): readlink of /sys/lbock/(null)/device failed: No such file or directory
creator.c:155 open_disk(): couldn ot open disk: No such file or directory
...

(and same error with --disk /dev/nvme0n1 --part 1)

Trying to debug I observed the following pattern, which seems unexpected by function set_disk_and_part_name

# ls -l /dev/nvme0n1p1
brw-rw---- 1 root disk 259, 6 Sep 10 15:06 /dev/nvme0n1p1
# ls -ld /sys/dev/block/259:6
lrwxrwxrwx 1 root root 0 Sep 10 15:10 /sys/dev/block/259:6 -> ../../devices/virtual/nvme-subsystem/nvme-subsys0/nvme0n1/nvme0n1p1

created time in 5 months

push eventremyoudompheng/go

Cherry Zhang

commit sha dbe32284ff4fb96906cdb121508eba668dbc5bae

cmd/link: mmap object data This resurrects CL 121198, except that this time we map read-only. In case that we need to apply relocations to the symbol's content that is backed by read-only memory, we do our own copy- on-write. This can happen if we failed to mmap the output file, or we build for Wasm. Memory profile for building k8s.io/kubernetes/cmd/kube-apiserver on Linux/AMD64: Old (before this sequence of CLs): inuse_space 1598.75MB total 669.87MB 41.90% 41.90% 669.87MB 41.90% cmd/link/internal/objfile.(*objReader).readSlices New: inuse_space 1280.45MB total 441.18MB 34.46% 34.46% 441.18MB 34.46% cmd/link/internal/objfile.(*objReader).readSlices Change-Id: I6b4d29d6eee9828089ea3120eb38c212db21330b Reviewed-on: https://go-review.googlesource.com/c/go/+/170741 Run-TryBot: Cherry Zhang <cherryyz@google.com> Reviewed-by: Austin Clements <austin@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>

view details

Bryan C. Mills

commit sha 14b5b4a2a13c3148fdfeb4852436661791a13e2e

cmd/go/internal/modload: fix boundary conditions in matchPackages This makes the boundary logic of matchPackages consistent with modload.dirInModule. Previously, matchPackages always stopped at go.mod file, even within the vendor tree. However, we do not guarantee that the vendor tree is free of such files in general. matchPackages also issued needless stat operations for modules in the module cach, which we already know to be free of nested modules. On systems with slow filesystems (such as macOS), those extra calls could potentially slow package matching considerably. Change-Id: I71979ab752e1d3971b370b37085d30502690413b Reviewed-on: https://go-review.googlesource.com/c/go/+/172985 Run-TryBot: Bryan C. Mills <bcmills@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Jay Conrod <jayconrod@google.com>

view details

Yuval Pavel Zholkover

commit sha 059f2d4a467465183820710df43a234f02d9c255

os: disable the use of netpoll on directories as well on *BSDs Follow up CL 156379. Updates #19093 Change-Id: I5ea3177fc5911d3af71cbb32584249e419e9d4a3 Reviewed-on: https://go-review.googlesource.com/c/go/+/172937 Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>

view details

Than McIntosh

commit sha 4590abe0723cbe639f88578d9d570b136ba0850b

cmd/link: adjust whitelist for -strictdups checking for plan9 Add a couple of additional entries to the white list used to screen out errors for builtin functions; these correspond to cases that appear to come up only on the plan9 builder. Updates #31503. Change-Id: I48ab942ab2894240efe651ec7b7eace7aa5cb45e Reviewed-on: https://go-review.googlesource.com/c/go/+/172986 Reviewed-by: David du Colombier <0intro@gmail.com> Reviewed-by: Ian Lance Taylor <iant@golang.org> Run-TryBot: Than McIntosh <thanm@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>

view details

Than McIntosh

commit sha dc193dee15294e451ceaae2e50e539255f4a37b6

go/internal/gccgoimporter: improve alias handling for anonymous fields The code in the parser that deals with anonymous structure fields records the fact that a field is anonymous, then tries to install a proxy name for the field based on the name of the type used to declare the field. If that type was an alias, the current recipe for determining the proxy name was not working properly; enhance the code to recover and report the alias name used. Fixes #31540. Change-Id: I9b7369ed558a288b56d85170c6f1144daf5228eb Reviewed-on: https://go-review.googlesource.com/c/go/+/172603 Reviewed-by: Ian Lance Taylor <iant@golang.org>

view details

David Chase

commit sha 376ce8c88033eede19d6295f9a79263f73c0fddb

cmd/compile: shortcut intrinsic inlining AFTER getcallerXX check A check in inl.go to prevent inlining of functions calling either getcallerpc or getcallersp does not work when these functions are intrinsics. Swap checks to fix. Includes test. No bug, this was discovered in the course of a ridiculous experiment with inlining. Change-Id: Ie1392523bb89882d586678f2674e1a4eadc5e431 Reviewed-on: https://go-review.googlesource.com/c/go/+/172217 Run-TryBot: David Chase <drchase@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Keith Randall <khr@golang.org>

view details

Matthew Dempsky

commit sha 9f9e17a82fd7afa622424f51e458bb383cb952ce

cmd/compile: fix ICE from go/defer call to variadic function The special case logic for go/defer arguments in Escape.call was scattered around a bit and was somewhat inconsistently handled across different types of function calls and parameters. This CL pulls the logic out into a separate callStmt method that's used uniformly for all kinds of function calls and arguments. Fixes #31573. Change-Id: Icdcdf611754dc3fcf1af7cb52879fb4b73a7a31f Reviewed-on: https://go-review.googlesource.com/c/go/+/173019 Run-TryBot: Matthew Dempsky <mdempsky@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>

view details

Bryan C. Mills

commit sha a1c481d85139f77ab27210526f9dfa2f3b375ef9

cmd/go: only add a 'go' directive on 'go mod tidy' or when a conversion occurs If the go.mod file exists and is empty, we initialize it from any of various formats supported by legacy dependency-management tools. We also initialize the 'go' directive at that point: we know that the go.mod file is incomplete, because it does not reflect the information in the legacy configuration file, and since we know that the go.mod file is incomplete, we should complete it with as much information as we have — including the version of the language currently in use. However, if there is no legacy configuration file present, then we cannot infer that the go.mod file is incomplete: it may correctly specify a module without external dependencies. In that case, we should not initialize the 'go' directive either: the user will not be expecting unnecessary edits to the go.mod file, and we generally do not make unnecessary-but-helpful edits unless 'go mod tidy' is invoked explicitly. Fixes #30790 Fixes #31100 Change-Id: I05a7872bce54a917c10d910cd9a616cab52e2730 Reviewed-on: https://go-review.googlesource.com/c/go/+/169877 Run-TryBot: Bryan C. Mills <bcmills@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>

view details

Maya Rashish

commit sha 78f0de10eccf3606dbc4b854a4cd3a7e502edc07

runtime: move linux specific code into linux specific files Allows us to stop whitelisting this error on many OS/arch combinations XXX I'm not sure I am running vet correctly, and testing all platforms right. Change-Id: I29f548bd5f4a63bd13c4d0667d4209c75c886fd9 GitHub-Last-Rev: 52f6ff4a6b986e86f8b26c3d19da7707d39f1664 GitHub-Pull-Request: golang/go#31583 Reviewed-on: https://go-review.googlesource.com/c/go/+/173157 Run-TryBot: Benny Siegert <bsiegert@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Benny Siegert <bsiegert@gmail.com>

view details

Maya Rashish

commit sha d23cba683e59f6092c1e3f676c8e83e7973f27dd

all: add start of netbsd/arm64 support This works well enough to run some code natively on arm64, but not well enough for more complicated code. I've been suggested to start a pull request anyway. Updates #30824 Change-Id: Ib4f63e0e8a9edfc862cf65b5f1b0fbf9a8a1628e GitHub-Last-Rev: b01b105e0446e349c8d9895d3ac6918fa0cdc48c GitHub-Pull-Request: golang/go#29398 Reviewed-on: https://go-review.googlesource.com/c/go/+/155739 Run-TryBot: Benny Siegert <bsiegert@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Benny Siegert <bsiegert@gmail.com>

view details

Josh Bleecher Snyder

commit sha 68d4b1265ec7915dccfccf6c0e32f9ab2d9c3a86

cmd/compile: reduce bits.Div64(0, lo, y) to 64 bit division With this change, these two functions generate identical code: func f(x uint64) (uint64, uint64) { return bits.Div64(0, x, 5) } func g(x uint64) (uint64, uint64) { return x / 5, x % 5 } Updates #31582 Change-Id: Ia96c2e67f8af5dd985823afee5f155608c04a4b6 Reviewed-on: https://go-review.googlesource.com/c/go/+/173197 Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>

view details

Austin Clements

commit sha db42bb3b70fe82e9d33011fb36b4171897663ccf

runtime: error formatting fix g.m is an muintptr, but we want to print it in hex like a pointer. Change-Id: Ifc48ed77fb2e93cff7a49d98adc7b9679d26c3b1 Reviewed-on: https://go-review.googlesource.com/c/go/+/172988 Run-TryBot: Austin Clements <austin@google.com> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>

view details

erifan01

commit sha f8f265b9cfd57970b2bc8b3dd8531cedaf57ccc1

cmd/compile: intrinsify math/bits.Sub64 for arm64 This CL instrinsifies Sub64 with arm64 instruction sequence NEGS, SBCS, NGC and NEG, and optimzes the case of borrowing chains. Benchmarks: name old time/op new time/op delta Sub-64 2.500000ns +- 0% 2.048000ns +- 1% -18.08% (p=0.000 n=10+10) Sub32-64 2.500000ns +- 0% 2.500000ns +- 0% ~ (all equal) Sub64-64 2.500000ns +- 0% 2.080000ns +- 0% -16.80% (p=0.000 n=10+7) Sub64multiple-64 7.090000ns +- 0% 2.090000ns +- 0% -70.52% (p=0.000 n=10+10) Change-Id: I3d2664e009a9635e13b55d2c4567c7b34c2c0655 Reviewed-on: https://go-review.googlesource.com/c/go/+/159018 Reviewed-by: Cherry Zhang <cherryyz@google.com> Run-TryBot: Cherry Zhang <cherryyz@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>

view details

erifan01

commit sha d17d41e58d2f69d284398f1d86d93c0f31648b16

math/big: optimize mulAddVWW on arm64 for better performance Unroll the cycle 4 times to reduce load overhead. Benchmarks: name old time/op new time/op delta MulAddVWW/1-8 15.9ns ± 0% 11.9ns ± 0% -24.92% (p=0.000 n=8+8) MulAddVWW/2-8 16.1ns ± 0% 13.9ns ± 1% -13.82% (p=0.000 n=8+8) MulAddVWW/3-8 18.9ns ± 0% 17.3ns ± 0% -8.47% (p=0.000 n=8+8) MulAddVWW/4-8 21.7ns ± 0% 19.5ns ± 0% -10.14% (p=0.000 n=8+8) MulAddVWW/5-8 25.1ns ± 0% 22.5ns ± 0% -10.27% (p=0.000 n=8+8) MulAddVWW/10-8 41.6ns ± 0% 40.0ns ± 0% -3.79% (p=0.000 n=8+8) MulAddVWW/100-8 368ns ± 0% 363ns ± 0% -1.36% (p=0.000 n=8+8) MulAddVWW/1000-8 3.52µs ± 0% 3.52µs ± 0% -0.14% (p=0.000 n=8+8) MulAddVWW/10000-8 35.1µs ± 0% 35.1µs ± 0% -0.01% (p=0.000 n=7+6) MulAddVWW/100000-8 351µs ± 0% 351µs ± 0% +0.15% (p=0.038 n=8+8) Change-Id: I052a4db286ac6e4f3293289c7e9a82027da0405e Reviewed-on: https://go-review.googlesource.com/c/go/+/155780 Run-TryBot: Cherry Zhang <cherryyz@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>

view details

Daniel Martí

commit sha fe1afe8d4ca06c56f583a9296282697f39d27d97

encoding/json: avoid work when unquoting strings We can work out how many bytes can be unquoted trivially in rescanLiteral, which already iterates over a string's bytes. Removing the extra loop in unquoteBytes simplifies the function and speeds it up, especially when decoding simple strings, which are common. While at it, we can remove unnecessary checks like len(s)<2 and s[0]=='"'. Add a comment explaining why. name old time/op new time/op delta CodeDecoder-8 11.2ms ± 0% 11.1ms ± 1% -1.63% (p=0.000 n=9+10) name old speed new speed delta CodeDecoder-8 173MB/s ± 0% 175MB/s ± 1% +1.66% (p=0.000 n=9+10) Updates #28923. Change-Id: I2436a3a7f8148a2f7a6a4cdbd7dec6b32ef5e20c Reviewed-on: https://go-review.googlesource.com/c/go/+/151157 Run-TryBot: Daniel Martí <mvdan@mvdan.cc> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>

view details

Austin Clements

commit sha 744fcfec4e94d9585d2be6f9efae8480a2f2fc8d

runtime: improve mstart comments Some of the comments were unclear or outdated. Change-Id: I02e01bf60def0074c1fa760e94aa992e9e4969b9 Reviewed-on: https://go-review.googlesource.com/c/go/+/172987 Run-TryBot: Austin Clements <austin@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Michael Knyszek <mknyszek@google.com>

view details

Maya Rashish

commit sha d1f43ccef7a8285bf3bcd3518d4f38838ce5da1c

runtime: use named macros on NetBSD It will use the full names that appear in netbsd's /usr/include/sys/syscall.h names. This adds some compat-goo (sys_sigprocmask->SYS_sigprocmask14), which might not be pretty, but the information about whether the compat version is used is probably important, as Go will keep using interfaces even after they are considered compatibility, which has caused problems in the past. also, the same names appear in ktrace (with the numbers). Change-Id: Idc1bb254ee33757a39ba224d91e8fbb0331e2149 GitHub-Last-Rev: b915e8f8a323cdc2d03119c3cf18e35d08c63d18 GitHub-Pull-Request: golang/go#31594 Reviewed-on: https://go-review.googlesource.com/c/go/+/173158 Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>

view details

Keith Randall

commit sha 43001a0dc96a29f662f2782c5fb3ca998eadd623

cmd/compile: use correct package name for stack object symbol Stack object generation code was always using the local package name for its symbol. Normally that doesn't matter, as we usually only compile functions in the local package. But for wrappers, the compiler generates functions which live in other packages. When there are two other packages with identical functions to wrap, the same name appears twice, and the compiler goes boom. Fixes #31252 Change-Id: I7026eebabe562cb159b8b6046cf656afd336ba25 Reviewed-on: https://go-review.googlesource.com/c/go/+/171464 Run-TryBot: Keith Randall <khr@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Matthew Dempsky <mdempsky@google.com>

view details

Shawn Elliott

commit sha e6ae4e86ad59c45f302a8828e77e6c234307fce4

cmd/go/internal/generate: stop premature variable substitution in commands go:generate commands passed no arguments are currently subject to premature variable substitution due to mistakenly assuming append guarantees a copy. The change fixes this by forcing a slice copy at each invocation of a command. The previous code assumed that append would always generate a copy of its inputs. However, append wouldn't create a copy if there was no need to increase capacity and it would just return the original input slice. This resulted in premature variable substitutions in the "master word list" of generate commands, thus yielding incorrect results across multiple invocations of the same command when the body contained substitutions e.g. environment variables, moreover these can change during the lifetime of go:generate processing a file. Note that this behavior would not manifest itself if any arguments were passed to the command, because append would make a copy of the slice as it needed to increase its capacity. The "hacky" work-around was to always pass at least one argument to any command, even if the command ignores it. e.g., //go:generate MyNoArgsCmd ' ' This CL fixes that issue and removes the need for the hack mentioned above. Fixes #31608 Change-Id: I782ac2234bd7035a37f61c101ee4aee38ed8d29f GitHub-Last-Rev: 796d3430191f183c123c450a60b4a7987cc85e20 GitHub-Pull-Request: golang/go#31527 Reviewed-on: https://go-review.googlesource.com/c/go/+/172580 Run-TryBot: Jay Conrod <jayconrod@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Jay Conrod <jayconrod@google.com>

view details

Tyler Bui-Palsulich

commit sha 1f0c102a57f62de6f66381c75732ba486da7b904

cmd/go: expand cannot determine module path error See suggestion 2 of #31543 by thepudds. We may want to expand 'go help mod init' in the future to document what the module path should look like. Change-Id: Ia559fa96fda871e011d53f42a803175abc512202 Reviewed-on: https://go-review.googlesource.com/c/go/+/173318 Run-TryBot: Jay Conrod <jayconrod@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Jay Conrod <jayconrod@google.com>

view details

push time in 6 months

create barnchremyoudompheng/go

branch : aarch32

created branch time in 6 months

more