.
I've just been running 4.17 since then (4.18 has some annoying i915 drm
bugs), and things have been perfectly fine in the storage/filesystem
department.
What I had noticed as being suspect back then was the following:
$ git tag --contains 744889b7cbb56a6
v4.19
v4.19.1
v4.19.2
v4.19.3
v4.19.4
v4.19.5
v4.20-rc1
v4.20-rc2
v4.20-rc3
v4.20-rc4
$ git tag --contains 1adfc5e4136f5967
v4.20-rc2
v4.20-rc3
v4.20-rc4
$
Since the 744889b7 commit message talks specifically about discard, and
1adfc5e4 claims to fix 744889b7, I assumed it was probably responsible
considering the tags profile, but did not try understand the commits or
bisect.
FYI the machine I observed this on is a SATA-attached SSD (Samsung 840
EVO 250G) X61s. I only run fstrim manually, but of course with discard
enabled all the way down the lvm+dmcrypt stack.
Maybe that's of use in hunting down this bug. If nobody else bisects in
the coming weeks I'll have to reconsider the rigamarole of backups,
repro, and attempting a bisect.
Regards,
Vito Caputo
since 4.3, that's left a
bad taste in my mouth but maybe I'm missing something.
Thanks,
Vito Caputo
. It seems to me we're
> > >> >> lacking
> > >> >> the appropriate mechanism for charging CPU resources consumed on
> > >> >> behalf of
> > >> >> user processes in kworker threads to the work-causing process.
> > &
On Thu, Jan 03, 2019 at 09:33:09PM +0100, Hussam Al-Tayeb wrote:
> > Sent: Thursday, January 03, 2019 at 10:12 PM
> > From: "Vito Caputo"
> > To: "Hussam Al-Tayeb"
> > Subject: Re: Excessive swapping under Linux 4.14.91 (no issues in 4.14.90).
> >
wanted an ability to noop
fields before where I was polling proc frequently and skipping the bulk
of what was there but syscpu was still rather high.
I'm definitely not in favor of just adding another stat file that is the
same format as the existing one with the intrs zeroed out. It's a dirty
hack; fine for your local needs but too gross for upstream IMHO.
Regards,
Vito Caputo
On Mon, Oct 29, 2018 at 05:35:15PM -0400, Waiman Long wrote:
> On 10/29/2018 05:23 PM, Vito Caputo wrote:
> > On Mon, Oct 29, 2018 at 04:59:03PM -0400, Waiman Long wrote:
> >> On 10/29/2018 04:38 PM, Davidlohr Bueso wrote:
> >>> On Mon, 29 Oct 2018, Waiman Long wrot
e used
to granularly expose all the information we currently expose in /proc,
while still being relatively efficient in terms of system calls per
sample. Sure you still have to lookup and open all the files of
interest, but that only needs to occur once at initialization.
Regards,
Vito Caputo
On Tue, Oct 30, 2018 at 11:57:56AM -0700, Davidlohr Bueso wrote:
> On Mon, 29 Oct 2018, Vito Caputo wrote:
>
> > I'm definitely not in favor of just adding another stat file that is the
> > same format as the existing one with the intrs zeroed out. It's a dirty
&
ially
long chain of diffs, but no such syscall exists. The more
differentials, the more fragmented the operation tended to be, requiring
increasing numbers of smaller reads across more files to reconstruct the
buffer.
The same thing would be useful for making reads from large numbers of
sysfs files less costly. I presume proposing such a generally
applicable VFS API addition would meet less resistance than specialized
proc interfaces, perhaps naively :).
Regards,
Vito Caputo
his
trademark abuse because MS is throwing money at them or is it a formal
licensing situation? Do we, the community, even have any right to know
what's going on?
Cheers,
Vito Caputo
mething similarly uncomfortable to explain to a
significant other as what you've been working on.
That is, assuming it's going to continue to exist in-tree...
Regards,
Vito Caputo
On Tue, Oct 23, 2018 at 04:54:25AM +0100, Al Viro wrote:
> On Mon, Oct 22, 2018 at 08:47:19PM -0700, Vito Caputo wrote:
> > On Tue, Oct 23, 2018 at 04:38:52AM +0100, Al Viro wrote:
> > > On Mon, Oct 22, 2018 at 03:19:12AM -0700, Metztli Information Technology
> > >
On Mon, May 21, 2018 at 02:57:18PM -0700, Vito Caputo wrote:
> On Mon, May 21, 2018 at 12:53:20PM -0700, Vito Caputo wrote:
> > Hello all,
> >
> > 4.17-rc4 (my latest kernel ATM) consistently fails to start xgalaga
> > without -window. I will try find time to build the
On Wed, May 23, 2018 at 04:18:05PM +0300, Ville Syrjälä wrote:
> On Wed, May 23, 2018 at 02:49:19AM -0700, Vito Caputo wrote:
> > On Mon, May 21, 2018 at 02:57:18PM -0700, Vito Caputo wrote:
> > > On Mon, May 21, 2018 at 12:53:20PM -0700, Vito Caputo wrote:
> > > > He
On Wed, May 23, 2018 at 09:20:37PM +0300, Ville Syrjälä wrote:
> On Wed, May 23, 2018 at 11:06:00AM -0700, Vito Caputo wrote:
> > On Wed, May 23, 2018 at 04:18:05PM +0300, Ville Syrjälä wrote:
> > > On Wed, May 23, 2018 at 02:49:19AM -0700, Vito Caputo wrote:
> > > >
On Wed, May 23, 2018 at 10:12:22PM +0300, Ville Syrjälä wrote:
> On Wed, May 23, 2018 at 11:39:00AM -0700, Vito Caputo wrote:
> > On Wed, May 23, 2018 at 09:20:37PM +0300, Ville Syrjälä wrote:
> > > On Wed, May 23, 2018 at 11:06:00AM -0700, Vito Caputo wrote:
> > > >
On Wed, May 23, 2018 at 11:41:25PM +0300, Ville Syrjälä wrote:
> On Wed, May 23, 2018 at 01:15:05PM -0700, Vito Caputo wrote:
> > On Wed, May 23, 2018 at 10:12:22PM +0300, Ville Syrjälä wrote:
> > > On Wed, May 23, 2018 at 11:39:00AM -0700, Vito Caputo wrote:
> > > >
On Wed, May 23, 2018 at 02:27:48PM -0700, Vito Caputo wrote:
> On Wed, May 23, 2018 at 11:41:25PM +0300, Ville Syrjälä wrote:
> > On Wed, May 23, 2018 at 01:15:05PM -0700, Vito Caputo wrote:
> > > On Wed, May 23, 2018 at 10:12:22PM +0300, Ville Syrjälä wrote:
> > > >
On Wed, May 23, 2018 at 03:27:34PM -0700, Vito Caputo wrote:
> On Wed, May 23, 2018 at 02:27:48PM -0700, Vito Caputo wrote:
> > On Wed, May 23, 2018 at 11:41:25PM +0300, Ville Syrjälä wrote:
> > > On Wed, May 23, 2018 at 01:15:05PM -0700, Vito Caputo wrote:
> > > >
On Mon, May 28, 2018 at 07:01:36PM +0200, Enric Balletbo Serra wrote:
> Hi Vito,
>
> cc: dm-devel, Alasdair and Mike Snitzer
>
> 2018-05-28 5:32 GMT+02:00 Vito Caputo :
> > On Thu, Jan 25, 2018 at 12:33:21AM -0800, vcap...@pengaru.com wrote:
> >> On Fri, Jan 19, 2
ssion implicitly calling
> getrandom() for the Xorg MIT cookie. This was shown in the strace log
> below:
>
>https://lkml.kernel.org/r/20190910173243.GA3992@darwi-home-pc
>
So did systemd-random-seed instead drain what little entropy there was
before GDM started, increasing the likelihood a subsequent getrandom()
call would block?
Regards,
Vito Caputo
Consolidate some duplicated bookkeeping from seq_read() into a
function.
Signed-off-by: Vito Caputo
---
fs/seq_file.c | 34 +-
1 file changed, 21 insertions(+), 13 deletions(-)
diff --git a/fs/seq_file.c b/fs/seq_file.c
index 1600034a929b..e5d2bccf5ac4 100644
Remove pointless return variable dance.
Appears vestigial from when the function did locking as seen in
unix_find_socket_byinode(), but locking is handled in
unix_find_socket_byname() for __unix_find_socket_byname().
Signed-off-by: Vito Caputo
---
net/unix/af_unix.c | 6 ++
1 file changed
Remove pointless use of size return variable by directly returning
sizes.
Signed-off-by: Vito Caputo
---
net/core/sock.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/net/core/sock.c b/net/core/sock.c
index fac2b4d80de5..e01ff0d3be95 100644
--- a/net/core/sock.c
Eliminate some verbosity by using min() macro and consolidating some
things, also fix inconsistent zero tests (! vs. == 0).
Signed-off-by: Vito Caputo
---
net/core/datagram.c | 44 ++--
1 file changed, 14 insertions(+), 30 deletions(-)
diff --git a/net
Return directly from all switch cases, no point in storing in err.
Signed-off-by: Vito Caputo
---
net/core/skbuff.c | 15 +++
1 file changed, 3 insertions(+), 12 deletions(-)
diff --git a/net/core/skbuff.c b/net/core/skbuff.c
index f5f904f46893..c59b68a413b5 100644
--- a/net/core
On Sun, Oct 13, 2019 at 12:30:41PM -0700, Eric Dumazet wrote:
>
>
> On 10/12/19 4:55 AM, Vito Caputo wrote:
> > Eliminate some verbosity by using min() macro and consolidating some
> > things, also fix inconsistent zero tests (! vs. == 0).
> >
> > Signed-off-
On Sun, Oct 13, 2019 at 01:17:18PM -0700, Eric Dumazet wrote:
>
>
> On 10/13/19 1:01 PM, Vito Caputo wrote:
> > On Sun, Oct 13, 2019 at 12:30:41PM -0700, Eric Dumazet wrote:
> >>
> >>
> >> On 10/12/19 4:55 AM, Vito Caputo wrote:
> >>
ote.c:261
Signed-off-by: Vito Caputo
Reported-by: syzbot
---
drivers/media/usb/dvb-usb/cxusb.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/media/usb/dvb-usb/cxusb.c
b/drivers/media/usb/dvb-usb/cxusb.c
index f02fa0a67aa4..fac19ec46089 100644
--- a/drivers/medi
't doing anything in particular.
Regards,
Vito Caputo
ake sure tty40 is allocated and empty
> openvt -c 40 -f -- true
>
> # open it and wait on read()
> cat /dev/tty40
>
> # from a second vt, simply open tty40 again
> true < /dev/tty40
>
> # come back to the first vt and watch cat bailing out with EAGAIN.
>
> Please
ped out to the persistent swap.
Administrators then decide if they want to throw some cheap storage at
enabling this coverage, or live with the existing risks.
I think it may be an interesting approach, enabling administrators to
repair primary storage while operating in a degraded mode. Then once
things are corrected, there would be a way to evict pages from
persistent swap for another shot at writeback to primary storage.
Regards,
Vito Caputo
ine is the venerable Thinkpad X61s, 1.8Ghz, Debian 9, config attached.
Regards,
Vito Caputo
4.17-rc4-config.gz
Description: application/gzip
On Mon, May 21, 2018 at 12:53:20PM -0700, Vito Caputo wrote:
> Hello all,
>
> 4.17-rc4 (my latest kernel ATM) consistently fails to start xgalaga
> without -window. I will try find time to build the latest rc this
> evening.
>
> > ~$ xgalaga
> > X Error of fail
iput()->iput_final(), which performs the shmem_unacct_size() for us.
Signed-off-by: Vito Caputo
---
This caught my eye while looking through the memfd_create() implementation.
Included patch was compile tested only...
mm/shmem.c | 15 +--
1 file changed, 9 insertions(+), 6 deletions(
return USB_STOR_XFER_ERROR;
At a glance this seems odd to me. If the caller insists on ignoring
the return value, shouldn't it just initialize partial to zero?
In my experience it's generally frowned upon for functions to store
results in error paths.
Regards,
Vito Caputo
On Sat, Jul 25, 2020 at 04:11:58AM +0100, Matthew Wilcox wrote:
> On Fri, Jul 24, 2020 at 05:22:21PM -0700, Vito Caputo wrote:
> > Prior to looking at the code, conceptually I was envisioning the pages
> > in the reflink source inode's address_space would simply get their
>
scriptors
in a single syscall to move the needle.
This syscall doesn't provide that. It doesn't really give any
advantage over what we can achieve already. It seems basically
pointless to me, from a monitoring proc files perspective.
Regards,
Vito Caputo
On Sun, Jul 05, 2020 at 01:44:54PM +0200, Greg KH wrote:
> On Sun, Jul 05, 2020 at 01:07:14AM -0700, Vito Caputo wrote:
> > On Sun, Jul 05, 2020 at 04:27:32AM +0100, Matthew Wilcox wrote:
> > > On Sun, Jul 05, 2020 at 05:18:58AM +0200, Jan Ziak wrote:
> > > > On Sun,
n terms of page cache utilization.
Dave did you ever end up going down this path?
Regards,
Vito Caputo
ke to be able to set
this on a specific user and all logins of that user are simply stuck
on that uid no matter what.
Thanks in advance,
Vito Caputo
On Tue, May 12, 2020 at 02:45:24AM -0700, Vito Caputo wrote:
> Hello folks,
>
> I'm curious if someone knows a way to do this using existing linux
> interfaces.
>
> I'd like to create a login lacking the ability to switch uid/gid.
>
> Even if the process has ac
On Tue, May 12, 2020 at 03:44:08AM -0700, Vito Caputo wrote:
> On Tue, May 12, 2020 at 02:45:24AM -0700, Vito Caputo wrote:
> > Hello folks,
> >
> > I'm curious if someone knows a way to do this using existing linux
> > interfaces.
> >
> > I'd lik
NET_VENDOR_AQUANTIA was "default y" for some reason, which seems
obviously inappropriate.
---
drivers/net/ethernet/aquantia/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/aquantia/Kconfig
b/drivers/net/ethernet/aquantia/Kconfig
index cdf78e069a39.
---
Documentation/rbtree.txt | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/Documentation/rbtree.txt b/Documentation/rbtree.txt
index c42a21b99046..3a4c23bc90b2 100644
--- a/Documentation/rbtree.txt
+++ b/Documentation/rbtree.txt
@@ -197,9 +197,9 @@ Cached rbtrees
-
Just something I noticed during my first 4.14 `make oldconfig`.
---
init/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/init/Kconfig b/init/Kconfig
index 78cb2461012e..86dac3a573de 100644
--- a/init/Kconfig
+++ b/init/Kconfig
@@ -1582,7 +1582,7 @@ config SLAB_FREELIST
46 matches
Mail list logo
