Re: [PATCH] virtio_rng: drop extra empty line

2015-01-15 Thread Michael S. Tsirkin
On Fri, Jan 16, 2015 at 10:21:09AM +1100, Herbert Xu wrote: > On Thu, Jan 15, 2015 at 01:50:42PM +0200, Michael S. Tsirkin wrote: > > makes code look a bit prettier. > > > > Signed-off-by: Michael S. Tsirkin > > Please resend this patch with a cc to linux-crypto@v

[PATCH repost] virtio_rng: drop extra empty line

2015-01-20 Thread Michael S. Tsirkin
makes code look a bit prettier. Cc: linux-crypto@vger.kernel.org. Signed-off-by: Michael S. Tsirkin --- drivers/char/hw_random/virtio-rng.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/char/hw_random/virtio-rng.c b/drivers/char/hw_random/virtio-rng.c index 72295ea..3fa2f8a 100644

[PATCH 14/86] crypto/ccp: use uapi/linux/pci_ids.h directly

2015-03-29 Thread Michael S. Tsirkin
Header moved from linux/pci_ids.h to uapi/linux/pci_ids.h, use the new header directly so we can drop the wrapper in include/linux/pci_ids.h. Signed-off-by: Michael S. Tsirkin --- drivers/crypto/ccp/ccp-ops.c | 2 +- drivers/crypto/ccp/ccp-pci.c | 2 +- 2 files changed, 2 insertions(+), 2

[PATCH 80/86] crypto/ccp: drop linux/pci dependencies

2015-03-29 Thread Michael S. Tsirkin
pci code is in ccp-pci.c, don't include pci headers from ccp/ccp-ops.c. Signed-off-by: Michael S. Tsirkin --- drivers/crypto/ccp/ccp-ops.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/crypto/ccp/ccp-ops.c b/drivers/crypto/ccp/ccp-ops.c index fd5491d..3da1140 100644 --- a/dr

[PATCH 15/86] crypto/geode: use uapi/linux/pci_ids.h directly

2015-03-29 Thread Michael S. Tsirkin
Header moved from linux/pci_ids.h to uapi/linux/pci_ids.h, use the new header directly so we can drop the wrapper in include/linux/pci_ids.h. Signed-off-by: Michael S. Tsirkin --- drivers/crypto/geode-aes.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/geode

[PATCH v2 1/6] crypto/ccp: drop linux/pci dependencies

2015-03-30 Thread Michael S. Tsirkin
pci code is in ccp-pci.c, don't include pci headers from ccp/ccp-ops.c. Signed-off-by: Michael S. Tsirkin --- drivers/crypto/ccp/ccp-ops.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/crypto/ccp/ccp-ops.c b/drivers/crypto/ccp/ccp-ops.c index 8729364..3da1140 100644 --- a/dr

Re: [PATCH v2 2/2] crypto: add virtio-crypto driver

2016-11-26 Thread Michael S. Tsirkin
device provides the following > cryptoservices: CIPHER, MAC, HASH, and AEAD. > > For more information about virtio-crypto device, please see: > http://qemu-project.org/Features/VirtioCrypto > > CC: Michael S. Tsirkin > CC: Cornelia Huck > CC: Stefan Hajnoczi >

Re: [PATCH v2 1/2] virtio: introduce little edian functions for virtio_cread/write# family

2016-11-26 Thread Michael S. Tsirkin
On Tue, Nov 22, 2016 at 04:10:22PM +0800, Gonglei wrote: > Virtio modern devices are always little edian, let's introduce > the LE functions for read/write configuration space for > virtio modern devices, which avoid complaint by Sparse when > we use the virtio_creaed/virtio_cwrite in VIRTIO_1 devi

Re: [virtio-dev] Re: [PATCH v2 1/2] virtio: introduce little edian functions for virtio_cread/write# family

2016-11-27 Thread Michael S. Tsirkin
On Mon, Nov 28, 2016 at 01:56:04AM +, Gonglei (Arei) wrote: > Hi Michael, > > Thanks for your feedback firstly! > > > -Original Message- > > From: virtio-...@lists.oasis-open.org > > [mailto:virtio-...@lists.oasis-open.org] > > On Behalf Of

Re: [virtio-dev] Re: [PATCH v2 1/2] virtio: introduce little edian functions for virtio_cread/write# family

2016-11-27 Thread Michael S. Tsirkin
On Mon, Nov 28, 2016 at 04:34:56AM +0200, Michael S. Tsirkin wrote: > On Mon, Nov 28, 2016 at 01:56:04AM +, Gonglei (Arei) wrote: > > Hi Michael, > > > > Thanks for your feedback firstly! > > > > > -Original Message- > > > From: virtio-.

Re: [virtio-dev] Re: [PATCH v2 2/2] crypto: add virtio-crypto driver

2016-11-27 Thread Michael S. Tsirkin
On Mon, Nov 28, 2016 at 04:47:21AM +, Gonglei (Arei) wrote: > Michael, I'd like to add virtio-crypto stuff to your maintaining part likes > the virtio-net/blk parts so that the corresponding patches > can be CC'ed to you too because the virtio-crypto doesn't lay in > driver/virtio directory. W

Re: [PATCH v3] crypto: add virtio-crypto driver

2016-11-28 Thread Michael S. Tsirkin
> http://qemu-project.org/Features/VirtioCrypto > > I've left some comments below. > > > > > CC: Michael S. Tsirkin > > CC: Cornelia Huck > > CC: Stefan Hajnoczi > > CC: Herbert Xu > > CC: Halil Pasic > > CC: David S. Mill

[PATCH 00/10] virtio: sparse fixes

2016-12-06 Thread Michael S. Tsirkin
sparse builds. Michael S. Tsirkin (10): virtio_console: drop unused config fields drm/virtio: fix endianness in primary_plane_update drm/virtio: fix lock context imbalance drm/virtio: annotate virtio_gpu_queue_ctrl_buffer_locked vhost: make interval tree static inline vhost: add missing

[PATCH 10/10] virtio: enable endian checks for sparse builds

2016-12-06 Thread Michael S. Tsirkin
__CHECK_ENDIAN__ isn't on by default presumably because it triggers too many sparse warnings for correct code. But virtio is now clean of these warnings, and we want to keep it this way - enable this for sparse builds. Signed-off-by: Michael S. Tsirkin --- It seems that there should be a b

Re: [PATCH 10/10] virtio: enable endian checks for sparse builds

2016-12-07 Thread Michael S. Tsirkin
On Wed, Dec 07, 2016 at 07:25:51AM +0100, Johannes Berg wrote: > On Tue, 2016-12-06 at 17:41 +0200, Michael S. Tsirkin wrote: > > > It seems that there should be a better way to do it, > > but this works too. > > In some cases there might be: > > > --- a/drive

[PATCH] linux/types.h: enable endian checks for all sparse builds

2016-12-07 Thread Michael S. Tsirkin
drop __CHECK_ENDIAN__. Follow-up patches can drop distinction between __bitwise and __bitwise__. Cc: Linus Torvalds Suggested-by: Christoph Hellwig Signed-off-by: Michael S. Tsirkin --- Linus, could you ack this for upstream? If yes I'll merge through my tree as a replacement for enabling thi

Re: [PATCH] linux/types.h: enable endian checks for all sparse builds

2016-12-07 Thread Michael S. Tsirkin
On Thu, Dec 08, 2016 at 05:21:47AM +, Bart Van Assche wrote: > On 12/07/16 18:29, Michael S. Tsirkin wrote: > > By now, linux is mostly endian-clean. Enabling endian-ness > > checks for everyone produces about 200 new sparse warnings for me - > > less than 10% over the

Re: [PATCH] linux/types.h: enable endian checks for all sparse builds

2016-12-08 Thread Michael S. Tsirkin
On Thu, Dec 08, 2016 at 06:38:11AM +, Bart Van Assche wrote: > On 12/07/16 21:54, Michael S. Tsirkin wrote: > > On Thu, Dec 08, 2016 at 05:21:47AM +, Bart Van Assche wrote: > >> Additionally, there are notable exceptions to the rule that most drivers > >> are

Re: [PATCH] linux/types.h: enable endian checks for all sparse builds

2016-12-09 Thread Michael S. Tsirkin
On Fri, Dec 09, 2016 at 03:18:02PM +, Bart Van Assche wrote: > On 12/08/16 22:40, Madhani, Himanshu wrote: > > We’ll take a look and send patches to resolve these warnings. > > Thanks! > > Bart. > Sounds good. I posted what I have so far so that you can start from that. -- MST -- To unsub

Re: [PATCH v6 2/2] crypto: add virtio-crypto driver

2016-12-12 Thread Michael S. Tsirkin
On Mon, Dec 12, 2016 at 06:54:07PM +0800, Herbert Xu wrote: > On Mon, Dec 12, 2016 at 06:25:12AM +, Gonglei (Arei) wrote: > > Hi, Michael & Herbert > > > > Because the virtio-crypto device emulation had been in QEMU 2.8, > > would you please merge the virtio-crypto driver for 4.10 if no other

[PATCH 0/8] enable endian checks for all sparse builds

2016-12-14 Thread Michael S. Tsirkin
This is just a reposting of the patch that enables endian checks, with addition of trivial patches that drop __bitwise__ and __CHECK_ENDIAN__ everywhere. I plan to include this in my pull request unless I hear otherwise. Michael S. Tsirkin (8): linux/types.h: enable endian checks for all

Re: [Qemu-devel] [PATCH v7 1/1] crypto: add virtio-crypto driver

2016-12-15 Thread Michael S. Tsirkin
On Thu, Dec 15, 2016 at 01:08:51AM +, Gonglei (Arei) wrote: > > > > > Regards, > -Gonglei > > > > -Original Message- > > From: Zeng, Xin [mailto:xin.z...@intel.com] > > Sent: Thursday, December 15, 2016 8:59 AM > > To: Gonglei (Arei); Halil Pasic; linux-ker...@vger.kernel.org; > >

Re: [PATCH v8 1/1] crypto: add virtio-crypto driver

2017-01-12 Thread Michael S. Tsirkin
On Thu, Jan 12, 2017 at 03:10:25PM +0100, Christian Borntraeger wrote: > On 01/10/2017 01:56 PM, Christian Borntraeger wrote: > > On 01/10/2017 01:36 PM, Gonglei (Arei) wrote: > >> Hi, > >> > >>> > >>> On 12/15/2016 03:03 AM, Gonglei wrote: > >>> [...] > + > +static struct crypto_alg virt

[PATCH 1/6] virtio: wrap find_vqs

2017-03-29 Thread Michael S. Tsirkin
We are going to add more parameters to find_vqs, let's wrap the call so we don't need to tweak all drivers every time. Signed-off-by: Michael S. Tsirkin --- drivers/block/virtio_blk.c | 3 +-- drivers/char/virtio_console.c | 6 +++--- drivers/cry

Re: [PATCH 1/6] virtio: wrap find_vqs

2017-03-30 Thread Michael S. Tsirkin
On Thu, Mar 30, 2017 at 02:00:08PM +0800, Jason Wang wrote: > > > On 2017年03月30日 04:48, Michael S. Tsirkin wrote: > > We are going to add more parameters to find_vqs, let's wrap the call so > > we don't need to tweak all drivers every time. > >

Re: [PATCH 1/6] virtio: wrap find_vqs

2017-03-31 Thread Michael S. Tsirkin
On Fri, Mar 31, 2017 at 12:04:55PM +0800, Jason Wang wrote: > > > On 2017年03月30日 22:32, Michael S. Tsirkin wrote: > > On Thu, Mar 30, 2017 at 02:00:08PM +0800, Jason Wang wrote: > > > > > > On 2017年03月30日 04:48, Michael S. Tsirkin wrote: > > > > We

[PATCH v2 08/24] virtio_crypto: correct tags for config space fields

2020-08-03 Thread Michael S. Tsirkin
Since crypto is a modern-only device, tag config space fields as having little endian-ness. Signed-off-by: Michael S. Tsirkin --- include/uapi/linux/virtio_crypto.h | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/include/uapi/linux

[PATCH v3 08/38] virtio_crypto: correct tags for config space fields

2020-08-05 Thread Michael S. Tsirkin
Since crypto is a modern-only device, tag config space fields as having little endian-ness. Signed-off-by: Michael S. Tsirkin Reviewed-by: Cornelia Huck --- include/uapi/linux/virtio_crypto.h | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a

[PATCH v3 32/38] virtio_crypto: convert to LE accessors

2020-08-05 Thread Michael S. Tsirkin
Virtio crypto is modern-only. Use LE accessors for config space. Signed-off-by: Michael S. Tsirkin --- drivers/crypto/virtio/virtio_crypto_core.c | 46 +++--- 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/drivers/crypto/virtio/virtio_crypto_core.c b/drivers

Re: [PATCH v2 2/2] crypto: virtio: Fix use-after-free in virtio_crypto_skcipher_finalize_req()

2020-05-31 Thread Michael S. Tsirkin
On Tue, May 26, 2020 at 02:11:37PM +, Sasha Levin wrote: > <20200123101000.GB24255@Red> > References: <20200526031956.1897-3-longpe...@huawei.com> > <20200123101000.GB24255@Red> > > Hi > > [This is an automated email] > > This commit has been processed because it contains a "Fixes:" tag > fi

Re: [PATCH v2 4/6] crypto: virtio: convert to new crypto engine API

2018-02-14 Thread Michael S. Tsirkin
On Fri, Jan 26, 2018 at 08:15:32PM +0100, Corentin Labbe wrote: > This patch convert the driver to the new crypto engine API. > > Signed-off-by: Corentin Labbe Acked-by: Michael S. Tsirkin Pls queue when/if rest of changes go in. > --- > drivers/crypto/virtio/virtio_crypt

Re: [PATCH v2 06/35] crypto: Use kmemdup rather than duplicating its implementation

2019-07-03 Thread Michael S. Tsirkin
ty, leads to smaller code and also reduce the chances of mistakes. > Suggestion to use kmemdup rather than using kmalloc/kzalloc + memcpy. > > Signed-off-by: Fuqian Huang virtio part: Acked-by: Michael S. Tsirkin > --- > Changes in v2: > - Fix a typo in commit message (m

Re: [PATCH v2 net-next 0/2] virtio_net: Expand affinity to arbitrary numbers of cpu and vq

2018-08-12 Thread Michael S. Tsirkin
t; when the number of cpus != the number of virtual queues. > > Patch 1 allows vqs to take an affinity mask with more than 1 cpu. > Patch 2 generalizes the algorithm in virtnet_set_affinity beyond > the case where #cpus == #vqs. Acked-by: Michael S. Tsirkin Let's see where doe

Re: [PATCH] crypto: virtio-crypto: Handle dataq logic with tasklet

2023-12-01 Thread Michael S. Tsirkin
On Mon, Nov 20, 2023 at 11:49:45AM +, Gonglei (Arei) wrote: > Doing ipsec produces a spinlock recursion warning. > This is due to crypto_finalize_request() being called in the upper half. > Move virtual data queue processing of virtio-crypto driver to tasklet. > > Fixes: dbaf0624ffa57 ("crypto

Re: [PATCH 5.10 000/286] 5.10.209-rc1 review

2024-01-28 Thread Michael S. Tsirkin
On Fri, Jan 26, 2024 at 03:55:02PM -0800, Guenter Roeck wrote: > On 1/26/24 14:35, Nathan Chancellor wrote: > > (slimming up the CC list, I don't think this is too relevant to the > > wider stable community) > > > > On Fri, Jan 26, 2024 at 01:01:15PM -0800, Guenter Roeck wrote: > > > On 1/26/24 12

Re: [PATCH 5.10 000/286] 5.10.209-rc1 review

2024-01-29 Thread Michael S. Tsirkin
On Fri, Jan 26, 2024 at 03:55:02PM -0800, Guenter Roeck wrote: > On 1/26/24 14:35, Nathan Chancellor wrote: > > (slimming up the CC list, I don't think this is too relevant to the > > wider stable community) > > > > On Fri, Jan 26, 2024 at 01:01:15PM -0800, Guenter Roeck wrote: > > > On 1/26/24 12

Re: [PATCH] crypto: virtio/akcipher - Fix stack overflow on memcpy

2024-01-31 Thread Michael S. Tsirkin
rtio-crypto: implement RSA algorithm") > Link: > https://lore.kernel.org/all/0a194a79-e3a3-45e7-be98-83abd3e1c...@roeck-us.net/ > Signed-off-by: zhenwei pi Cc: sta...@vger.kernel.org Acked-by: Michael S. Tsirkin > --- > drivers/crypto/virtio/virtio_crypto_akcipher_algs.c | 5 +++-- &

Re: virtcrypto_dataq_callback calls crypto_finalize_request() from irq context

2024-02-22 Thread Michael S. Tsirkin
On Thu, Nov 02, 2023 at 01:25:31PM +, Gonglei (Arei) wrote: > > > > -Original Message- > > From: Michael S. Tsirkin [mailto:m...@redhat.com] > > Sent: Thursday, November 2, 2023 9:17 PM > > To: Gonglei (Arei) > > Cc: Halil Pasic ; Herbert X

[PATCH] hwrng: document the quality field

2018-09-25 Thread Michael S. Tsirkin
* current_quality / 1000 but that has performance costs, so probably isn't a good option. Fixes: 0f734e6e768 ("hwrng: add per-device entropy derating") Reported-by: "Dr. David Alan Gilbert" Signed-off-by: Michael S. Tsirkin --- drivers/char/hw_random/core.c | 4 +

Re: [PATCH] char: hw_random: Fix missing check during driver release

2018-12-26 Thread Michael S. Tsirkin
On Wed, Dec 26, 2018 at 11:23:31AM -0600, Aditya Pakki wrote: > devres_release can return -ENOENT if the device is not freed. The fix > throws a warning consistent with other invocations. > > Signed-off-by: Aditya Pakki Well why not Acked-by: Michael S. Tsirkin > ---