All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bart Van Assche <Bart.VanAssche@wdc.com>
To: "keescook@chromium.org" <keescook@chromium.org>,
	"Jason@zx2c4.com" <Jason@zx2c4.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"jiangyilism@gmail.com" <jiangyilism@gmail.com>,
	"nab@linux-iscsi.org" <nab@linux-iscsi.org>,
	Bart Van Assche <Bart.VanAssche@wdc.com>,
	"viro@zeniv.linux.org.uk" <viro@zeniv.linux.org.uk>,
	"linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
	"target-devel@vger.kernel.org" <target-devel@vger.kernel.org>,
	"varun@chelsio.com" <varun@chelsio.com>
Subject: Re: [PATCH] iscsi-target: Convert timers to use timer_setup()
Date: Wed, 25 Oct 2017 15:03:19 +0000	[thread overview]
Message-ID: <1508943795.4165.18.camel@wdc.com> (raw)
In-Reply-To: <CAGXu5jKNDM8NjgtgBp-XapHEiaY_ryk3pBXdaryux6spX++zmQ@mail.gmail.com>

On Wed, 2017-10-25 at 16:10 +0200, Kees Cook wrote:
> However, maintainers: sorry to send this one -- it can't be merged
> yet, this uses timer_setup_on_stack() which is only in -next right
> now. If it looks okay, though, I can carry this in the timer tree.

Hello Kees,

Can you have a look at the following patch and queue it before your
patch if you think this would be useful (this patch has been tested):
"target/iscsi: Simplify timer manipulation code"
(https://www.spinics.net/lists/target-devel/msg15385.html).

Thanks,

Bart.

WARNING: multiple messages have this Message-ID (diff)
From: Bart Van Assche <Bart.VanAssche@wdc.com>
To: "keescook@chromium.org" <keescook@chromium.org>,
	"Jason@zx2c4.com" <Jason@zx2c4.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"jiangyilism@gmail.com" <jiangyilism@gmail.com>,
	"nab@linux-iscsi.org" <nab@linux-iscsi.org>,
	Bart Van Assche <Bart.VanAssche@wdc.com>,
	"viro@zeniv.linux.org.uk" <viro@zeniv.linux.org.uk>,
	"linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
	"target-devel@vger.kernel.org" <target-devel@vger.kernel.org>,
	"varun@chelsio.com" <varun@chelsio.com>
Subject: Re: [PATCH] iscsi-target: Convert timers to use timer_setup()
Date: Wed, 25 Oct 2017 15:03:19 +0000	[thread overview]
Message-ID: <1508943795.4165.18.camel@wdc.com> (raw)
In-Reply-To: <CAGXu5jKNDM8NjgtgBp-XapHEiaY_ryk3pBXdaryux6spX++zmQ@mail.gmail.com>

T24gV2VkLCAyMDE3LTEwLTI1IGF0IDE2OjEwICswMjAwLCBLZWVzIENvb2sgd3JvdGU6DQo+IEhv
d2V2ZXIsIG1haW50YWluZXJzOiBzb3JyeSB0byBzZW5kIHRoaXMgb25lIC0tIGl0IGNhbid0IGJl
IG1lcmdlZA0KPiB5ZXQsIHRoaXMgdXNlcyB0aW1lcl9zZXR1cF9vbl9zdGFjaygpIHdoaWNoIGlz
IG9ubHkgaW4gLW5leHQgcmlnaHQNCj4gbm93LiBJZiBpdCBsb29rcyBva2F5LCB0aG91Z2gsIEkg
Y2FuIGNhcnJ5IHRoaXMgaW4gdGhlIHRpbWVyIHRyZWUuDQoNCkhlbGxvIEtlZXMsDQoNCkNhbiB5
b3UgaGF2ZSBhIGxvb2sgYXQgdGhlIGZvbGxvd2luZyBwYXRjaCBhbmQgcXVldWUgaXQgYmVmb3Jl
IHlvdXINCnBhdGNoIGlmIHlvdSB0aGluayB0aGlzIHdvdWxkIGJlIHVzZWZ1bCAodGhpcyBwYXRj
aCBoYXMgYmVlbiB0ZXN0ZWQpOg0KInRhcmdldC9pc2NzaTogU2ltcGxpZnkgdGltZXIgbWFuaXB1
bGF0aW9uIGNvZGUiDQooaHR0cHM6Ly93d3cuc3Bpbmljcy5uZXQvbGlzdHMvdGFyZ2V0LWRldmVs
L21zZzE1Mzg1Lmh0bWwpLg0KDQpUaGFua3MsDQoNCkJhcnQu

  reply	other threads:[~2017-10-25 15:03 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-25 10:01 [PATCH] iscsi-target: Convert timers to use timer_setup() Kees Cook
2017-10-25 10:01 ` Kees Cook
2017-10-25 12:41 ` Jason A. Donenfeld
2017-10-25 12:41   ` Jason A. Donenfeld
2017-10-25 14:10   ` Kees Cook
2017-10-25 14:10     ` Kees Cook
2017-10-25 15:03     ` Bart Van Assche [this message]
2017-10-25 15:03       ` Bart Van Assche
2017-10-26  8:24       ` Kees Cook
2017-10-26  8:24         ` Kees Cook
2017-10-26 14:04         ` Bart Van Assche
2017-10-26 14:04           ` Bart Van Assche
2017-10-26 14:50           ` Kees Cook
2017-10-26 14:50             ` Kees Cook

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1508943795.4165.18.camel@wdc.com \
    --to=bart.vanassche@wdc.com \
    --cc=Jason@zx2c4.com \
    --cc=jiangyilism@gmail.com \
    --cc=keescook@chromium.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=nab@linux-iscsi.org \
    --cc=target-devel@vger.kernel.org \
    --cc=varun@chelsio.com \
    --cc=viro@zeniv.linux.org.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.